From: Denys Dmytriyenko <denis@denix.org>
To: Alexander Kanavin <alexander.kanavin@linux.intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 4/5] u-boot: Drop PV altogether
Date: Wed, 19 Oct 2016 16:27:50 -0400 [thread overview]
Message-ID: <20161019202750.GU2494@denix.org> (raw)
In-Reply-To: <78ae8d38-99d6-56d0-5e28-786bf2c13d43@linux.intel.com>
On Wed, Oct 19, 2016 at 03:09:26PM +0300, Alexander Kanavin wrote:
> On 10/19/2016 12:55 AM, Marek Vasut wrote:
> >Drop the explicit setting of PV altogether. This changes the U-Boot
> >version string from "v20yy.mm" to "20yy.mm" , so layer setting the
> >PREFERRED_VERSION_u-boot must be updated by removing the leading 'v'.
>
> Don't drop the 'v'. Package versions need to monotonically increase,
> and also package version needs to match git tags so that upstream
> version check can work.
>
> Basically, rename the recipe filenames so that 'v' is in their
> versions, and then you can drop the PV from the recipes. All in a
> single patch.
This has been discussed at OEDEM - we are trying to unify u-boot recipes.
Looking at the layer index, it seems there's 50/50 split of using vXXXX.YY vs.
just XXXX.YY versions:
https://layers.openembedded.org/layerindex/branch/master/recipes/?q=u-boot
So, I'd vote to drop "v" prefix and bump PE to get it sorted out...
--
Denys
next prev parent reply other threads:[~2016-10-19 20:27 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-18 21:55 [PATCH 1/5] u-boot: Factor out common bits Marek Vasut
2016-10-18 21:55 ` [PATCH 2/5] u-boot: Update to 2016.09.01 release Marek Vasut
2016-10-19 12:06 ` Alexander Kanavin
2016-10-19 20:10 ` Denys Dmytriyenko
2016-10-18 21:55 ` [PATCH 3/5] u-boot: Drop the +git part from PV Marek Vasut
2016-10-19 20:15 ` Denys Dmytriyenko
2016-10-18 21:55 ` [PATCH 4/5] u-boot: Drop PV altogether Marek Vasut
2016-10-19 12:09 ` Alexander Kanavin
2016-10-19 20:27 ` Denys Dmytriyenko [this message]
2016-10-19 20:10 ` Denys Dmytriyenko
2016-10-19 21:00 ` Marek Vasut
2016-10-19 21:15 ` Denys Dmytriyenko
2016-10-19 21:39 ` Burton, Ross
2016-10-19 22:53 ` Marek Vasut
2016-10-20 10:13 ` Burton, Ross
2016-10-20 13:36 ` Marek Vasut
2016-10-18 21:55 ` [PATCH 5/5] u-boot: Drop redundant comment Marek Vasut
2016-10-19 20:11 ` Denys Dmytriyenko
2016-10-19 20:13 ` [PATCH 1/5] u-boot: Factor out common bits Denys Dmytriyenko
2016-10-19 20:39 ` Marek Vasut
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=20161019202750.GU2494@denix.org \
--to=denis@denix.org \
--cc=alexander.kanavin@linux.intel.com \
--cc=openembedded-core@lists.openembedded.org \
/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