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 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.