Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 0/4] Sanetize packages version
Date: Sun, 23 Jun 2019 18:33:08 +0200	[thread overview]
Message-ID: <20190623183308.3bfbc24d@windsurf> (raw)
In-Reply-To: <da51251f-4903-fd0c-0e49-7cb0d9f5ea3c@mind.be>

Hello,

On Thu, 20 Jun 2019 22:32:29 +0200
Arnout Vandecappelle <arnout@mind.be> wrote:

> > - android-tools:        4.2.2+git20130218
> > The downloaded version is fetch from launchpad, the RM version
> > corresponds to the android version because sources for this tool are not
> > distributed alone.  
> 
>  I wonder why we're using this launchpad version instead of the upstream
> version. Maybe the original contributor (a certain Thomas P) remembers that?

The upstream android-tools from Google doesn't have any reasonable
build system: it only has a bunch of Android.mk, which requires the
entire Android stuff to be pulled in to be used. The Debian/Ubuntu
version has an alternate build system with a few makefiles that get the
job done.

> > - bandwidthd:           2.0.1-auto-r11
> > The downloaded version is not the original bandwithd but a fork that
> > uses automake, this fork appends -auto-rXX to the actual version.  
> 
>  We should keep the suffix then.
> 
>  Actually, there's another fork under NethServer that seems a little more
> active. It's also used by OpenWrt.
> 
>  But the sourceforge repo that hasn't seen activity in 15 years that
> release-monitoring refers to is definitely not OK :-)

Perhaps we should register a separate bandwidthd-auto project in
release-monitoring.org that tracks the bandwidthd fork we're using, and
create a bandwidthd -> bandwidthd-auto mapping in RM.

> > - imx-alsa-plugins:     rel_imx_4.9.x_1.0.0_ga
> > - imx-mkimage:          rel_imx_4.14.78_1.0.0_ga
> > I The 4.9.x seems to be related to freescale but I'm not sure about the
> > *_ga suffix.  
> 
>  The 4.9.x / 4.14.78 is the kernel version; the 1.0.0 is the freescale release
> on top of that. Obviously these packages really have nothing to do with the
> kernel version, but they distribute everything as a single SDK and the kernel
> version is the anchor.

Most of these vendor-specific packages often have very weird version
numbering, I'm not sure it will be easy for RM to track these. To me,
such packages are not really the priority of things we want to
check/monitor with RM.

> > - luarocks fetched packages: luasql-0.2.2.1-1
> > A large amount of lua packages have a -1 suffix. This suffix is not
> > present on release-monitoring, so I think we should remove it.
> > The thing is: the `luarocks-package` function seems to use the
> > <pkg>_VERSION variable.
> > Remove suffix for those packages will probably require to modify a bit
> > the way luarocks packages are handle.  
> 
>  Hm, good point... The -1 is the packaging version. Sometimes it needs to be
> repackaged and you get a -2. And for some reason, sometimes it's -0.
> 
>  I don't see a convenient way to work around this...

The only solution that I can see is changing the luarocks infra to use
something else than <pkg>_VERSION, like <pkg>_LUAROCKS_VERSION, and
have packages provide this variable.

> > If we achieve to modify these packages, the only packages left would be
> > those with a commit id as version.  
> 
>  Commit ID is no problem.

Agreed.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

  reply	other threads:[~2019-06-23 16:33 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-06-12  6:42 [Buildroot] [PATCH 0/4] Sanetize packages version Victor Huesca
2019-06-12  6:42 ` [Buildroot] [PATCH 1/4] package: remove 'v' prefix from github-fetched packages Victor Huesca
2019-06-19 20:50   ` Arnout Vandecappelle
2019-06-20  6:31     ` Thomas Petazzoni
2019-06-20 19:33       ` Arnout Vandecappelle
2019-06-20 19:46         ` Thomas Petazzoni
2019-06-20 21:07           ` Arnout Vandecappelle
2019-06-20 11:51     ` Victor Huesca
2019-06-20 19:31       ` Arnout Vandecappelle
2019-06-12  6:42 ` [Buildroot] [PATCH 2/4] package: remove 'v' prefix from tarball-fetched packages Victor Huesca
2019-06-19 21:06   ` Arnout Vandecappelle
2019-06-20  6:13     ` Thomas Petazzoni
2019-06-20  9:23       ` Victor Huesca
2019-06-20 19:43         ` Arnout Vandecappelle
2019-06-12  6:42 ` [Buildroot] [PATCH 3/4] package: remove non-conventional prefix/suffix from github-fetched packages Victor Huesca
2019-06-19 21:30   ` Arnout Vandecappelle
2019-06-20 12:42     ` Victor Huesca
2019-06-20 19:50       ` Arnout Vandecappelle
2019-06-21  2:15   ` Carlos Santos
2019-06-12  6:42 ` [Buildroot] [PATCH 4/4] package: remove non-conventional prefix/suffix from tarball-fetched packages Victor Huesca
2019-06-20 21:27   ` Arnout Vandecappelle
2019-06-12  7:26 ` [Buildroot] [PATCH 0/4] Sanetize packages version Thomas Petazzoni
2019-06-12  8:39   ` Victor Huesca
2019-06-12 14:50     ` Thomas Petazzoni
2019-06-19 15:35       ` Victor Huesca
2019-06-20 20:32         ` Arnout Vandecappelle
2019-06-23 16:33           ` Thomas Petazzoni [this message]
2019-06-12  8:51   ` Arnout Vandecappelle
2019-06-18 12:11     ` Victor Huesca

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=20190623183308.3bfbc24d@windsurf \
    --to=thomas.petazzoni@bootlin.com \
    --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