Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Luca Ceresoli <luca@lucaceresoli.net>
To: buildroot@busybox.net
Subject: [Buildroot] legal-info: which version to show in manifest for modified packages?
Date: Wed, 16 Apr 2014 10:31:27 +0200	[thread overview]
Message-ID: <534E3FDF.6090107@lucaceresoli.net> (raw)
In-Reply-To: <CAAXf6LWqZeuCLB3kZK-pzYh1JqY8OMmG68wFcUN9rYqRiNu69g@mail.gmail.com>

Hi Thomas,

Thomas De Schampheleire wrote:
> Hi,
>
> I'm pondering on the following: which 'version' should we show in the
> legal-info manifest for packages that have been modified for the
> project and are using a version-controlled source?
>
> For example, the linux and uboot packages allow to specify a custom
> version and custom location of the sources. In a given project, the
> version could be the git/hg/... revision of the repository containing
> these modified sources of linux/uboot/...
>
> When running 'make legal-info', these revision hashes would be stored
> in the manifest.csv file, but these numbers are meaningless outside of
> the project.

Not necessarily. They may be commits of a public repo, e.g.:

$ grep UBOOT_CUSTOM configs/cubieboard2_defconfig
BR2_TARGET_UBOOT_CUSTOM_GIT=y
BR2_TARGET_UBOOT_CUSTOM_REPO_URL="https://github.com/linux-sunxi/u-boot-sunxi.git"
BR2_TARGET_UBOOT_CUSTOM_REPO_VERSION="b4bca5e0651b460a4511582f07a935cb48f74948"
$

>
> Aside from the technical complications to integrate this in buildroot,
> would it instead be better to put the upstream version in the
> manifest, being the base version used to make the modifications on?

I don't think so.

First, if the manifest reports an upstream version, say "3.13.6", one
would think the target ships Linux version 3.13.6, unmodified. This
would be plain false.

Second, we would need a way to detect whether the VCS repository is
public or not (see example above). But I don't think this is feasible
in a reliable way.

Third, extracting the "upstream" version would be not straightforward.

-- 
Luca

      reply	other threads:[~2014-04-16  8:31 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-11 13:03 [Buildroot] legal-info: which version to show in manifest for modified packages? Thomas De Schampheleire
2014-04-16  8:31 ` Luca Ceresoli [this message]

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=534E3FDF.6090107@lucaceresoli.net \
    --to=luca@lucaceresoli.net \
    --cc=buildroot@busybox.net \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox