The wording might need improvement but both of your examples actually do
provide SRC_REV but in the inline form with SRC_URI (rev/tag).

The documentation should probably say something like SRC_REV needs to be
provided but it can either be done by setting the variable explicitly or
by using the rev/tag option with SRC_URI.

:rjs

On 2/14/20 10:35 PM, [email protected] wrote:
>   yes, yes, more obsessive-compulsive nitpickery, but YP dev tasks
> manual, section 3.3.5, reads:
>
> "Another way of specifying source is from an SCM. For Git
> repositories, you must specify SRCREV and you should specify PV to
> include the revision with SRCPV."
>
>   you *must* specify SRCREV? i don't think that's true -- a couple
> examples from meta-oe ... here's hiredis_0.13.1.bb:
>
> SRC_URI = 
> "git://github.com/redis/hiredis;protocol=git;rev=f58dd249d6ed47a7e835463c3b04722972281dbb
>  \
>
>   and here's sshfs-fuse_2.8.bb:
>
> SRC_URI = 
> "git://github.com/libfuse/sshfs;tag=b2fa7593586b141298e6159f40f521d2b0f4f894 \
>
>   neither of which specify SRCREV (clearly since they don't need to
> given the SRC_URI options in use).
>
> rday
>
> p.s. bitbake's git.py fetcher code makes no mention of the "rev"
> option in its opening docstring. just an observation if anyone wants
> to tweak that.
>
>
> 

-- 
-----
Rudolf J Streif
CEO/CTO ibeeto
+1.855.442.3386 x700

Attachment: signature.asc
Description: OpenPGP digital signature

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

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

Reply via email to