On Tue, Feb 25, 2020 at 02:39:28PM -0600, Joshua Watt wrote:
> On Tue, Feb 25, 2020 at 1:32 PM Joel A Cohen <[email protected]> wrote:
> >
> >
> >
> > On Tue, Feb 25, 2020 at 2:28 PM Martin Jansa <[email protected]> wrote:
> >>
> >> On Tue, Feb 25, 2020 at 02:10:15PM -0500, Joel A Cohen wrote:
> >>
> >>
> >> > PV_pn-recipe1 = "1.0.1+git${SRCPV}"  # I'm not sure if this would 
> >> > actually
> >> > work?
> >>
> >> Yes, this is how SRCPV is supposed to be used, but you can have this in
> >> the normal recipe without the override.
> >
> >
> > Everything else you said makes sense to me, but why would I want to append 
> > "+git${SRCPV}" to the actual recipe? Surely I only want to append the git 
> > suffix stuff if I'm using a "latest" version, and I want the PV to be set 
> > by the filename in the default case?
> 
> You should checkout devupstream.bbclass
> (https://www.yoctoproject.org/docs/current/mega-manual/mega-manual.html#ref-classes-devupstream)
> which allows you to do this inside a single recipe (or bbappend and
> existing recipe). You can also combine this with ${AUTOREV} in a
> common include file. I recently fixed this exact same problem with
> this combination in meta-webkit; see
> https://github.com/Igalia/meta-webkit/pull/136

Ah, nice touch with devupstream class and I see bleeding.inc concept lives on! 
:)

-- 
Denys
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#48607): https://lists.yoctoproject.org/g/yocto/message/48607
Mute This Topic: https://lists.yoctoproject.org/mt/71538656/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub  
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to