From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v5] support/scripts/pkg-stats: add latest upstream version information
Date: Thu, 3 Jan 2019 21:33:16 +0100 [thread overview]
Message-ID: <20190103213316.637c464b@windsurf> (raw)
In-Reply-To: <CA+fik52n+izpqRGSSP1QFMoPCCd1y0Mub714-1cHWFYotXuhNg@mail.gmail.com>
Hello Brandon,
Thanks for the feedback.
On Thu, 3 Jan 2019 13:57:46 -0600, Brandon Maier wrote:
> From https://bootlin.com/~thomas/stats.html, there's a few packages
> that are up-to-date, but get marked "version-needs-update" because
> either the Buildroot or release-monitoring version string starts with
> a "v". For example, the vmtouch and libmodbus packages. It would be
> nice to change the version comparison to ignore a leading "v". E.g.
>
> def version_uptodate(pkg):
> latest = remove_prefix(pkg.latest_version[1], 'v')
> current = remove_prefix(pkg.current_version, 'v')
> return latest == current
I'm aware of this problem, but I'm not sure we should handle this with
a hack in the pkg-stats script itself. Indeed sometimes the version
prefix is just "v", but sometimes it's the entire name of the package,
sometimes something slightly different.
Back when I initially started working on this script, I had a series
that changed all the packages that used vX.Y.Z as a version to use just
X.Y.Z, and similarly for other packages in similar but slightly
different situations. I found (with some effort) the old branch I had
with those changes:
https://git.bootlin.com/users/thomas-petazzoni/buildroot/log/?h=fix-versions
It seems like I never posted them. I thought there was some discussion
on the list about this issue, but I can't find it. Perhaps I should
update this series, submit it for good, and see what the feedback is ?
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
next prev parent reply other threads:[~2019-01-03 20:33 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-03 8:38 [Buildroot] [PATCH v5] support/scripts/pkg-stats: add latest upstream version information Thomas Petazzoni
2019-01-03 8:40 ` Thomas Petazzoni
2019-01-03 19:57 ` Brandon Maier
2019-01-03 20:33 ` Thomas Petazzoni [this message]
2019-01-03 21:02 ` [Buildroot] [External] " Brandon Maier
2019-01-03 21:16 ` Thomas Petazzoni
2019-01-03 23:29 ` Brandon Maier
2019-01-03 20:38 ` [Buildroot] " Yann E. MORIN
[not found] ` <CANQCQpZHYrkUVrYUxqANd9xzicMz-9apDWYeUWYmHLqG6YA8wg@mail.gmail.com>
2019-01-03 20:23 ` [Buildroot] [External] " Matthew Weber
2019-01-04 2:12 ` [Buildroot] " Ricardo Martincoski
2019-01-04 14:54 ` Brandon Maier
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=20190103213316.637c464b@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 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.