All of lore.kernel.org
 help / color / mirror / Atom feed
From: Olof Johansson <olof.johansson@axis.com>
To: Khem Raj <raj.khem@gmail.com>, bitbake-devel@lists.openembedded.org
Subject: Re: [PATCH] BB_NO_NETWORK: Fallback to local source for git lsremote
Date: Tue, 9 Aug 2016 11:36:48 +0200	[thread overview]
Message-ID: <20160809093647.GP4674@axis.com> (raw)
In-Reply-To: <5D5A3DA9-5197-492B-8A63-E28A882D99F0@gmail.com>

On 16-08-08 10:22 -0700, Khem Raj wrote:
> you are confusing how a tool is specifying its functions with
> process that could emulate a different behavior. OE does not
> have any control over the internal processes of its users and
> it should not have.

No, I don't think I'm confusing anything. Are you confusing OE
and bitbake? We maintain and develop an in-house Poky based
distribution, and unlike OE, we know a lot about our
users/developers.

> > Tags are only a problem in git if people insist on
> > modifying/deleting them.
> 
> Fact is, its allowed rewrite tags in git, how people use it or not
> is irrelevant here.

Nope, Git is a protocol; git the tool can't do nothing about a
server refusing to (re)move a tag. Your view of what a git tags
is yours. Tags are meant to be static, otherwise they wouldn't
have been synced to all clones in such a silly namespace.

While I agreed with Tobias' patch ("if you use tags in your
builds, and use BB_NO_NETWORK, you are ok with falling back to
what you have locally"), RP had a point about this making
BB_NO_NETWORK have non-obvious side-effects, and also making it
harder to catch cases where tags aren't wanted (like in community
maintained layers).

But *we* still want to use human readable tags as the only
version string internally (for our layers), where we know we are
in control. So, introducing a variable that lets us opt-in to
this behavior sounds like something that could be accepted?

-- 
olofjn


  reply	other threads:[~2016-08-09  9:36 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-08  8:34 [PATCH] BB_NO_NETWORK: Fallback to local source for git lsremote Tobias Hagelborn
2016-08-08  9:55 ` Richard Purdie
2016-08-08 11:24   ` Jérémy Rosen
2016-08-08 15:35     ` Khem Raj
2016-08-08 15:58       ` Jérémy Rosen
2016-08-08 16:25         ` Khem Raj
2016-08-08 16:39       ` Olof Johansson
2016-08-08 17:22         ` Khem Raj
2016-08-09  9:36           ` Olof Johansson [this message]
2016-08-09 10:50             ` Barros Pena, Belen
2016-08-09 14:34             ` Khem Raj
2016-08-09 15:09               ` Olof Johansson
2016-08-09 15:54                 ` Khem Raj
2016-08-09 16:26                   ` Olof Johansson
2016-08-10  8:54                     ` Richard Purdie
2016-08-09  3:22       ` Paul Eggleton
2016-08-09  8:15         ` Jérémy Rosen
2016-08-09  9:49         ` Tobias Hagelborn
2016-08-08 12:54   ` Tobias Hagelborn
     [not found] <801c8b3c-a6f7-6b96-a527-d61c70a4a574@smile.fr>
2016-08-10  8:28 ` Jérémy Rosen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160809093647.GP4674@axis.com \
    --to=olof.johansson@axis.com \
    --cc=bitbake-devel@lists.openembedded.org \
    --cc=raj.khem@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.