classes/crate-fetch: Ensure crate fetcher is available

Message ID 20211119021249.113892-1-JPEWhacker@gmail.com
State Accepted, archived
Commit bc66c5dd65fb654af5cd91b47f9b4f7a5f53436e
Headers show
Series classes/crate-fetch: Ensure crate fetcher is available | expand

Commit Message

Joshua Watt Nov. 19, 2021, 2:12 a.m. UTC
Reworks the crate fetcher class to have it install the fetcher at recipe
finalization so that it is always available before SRC_URI is expanded.
In addition, override the value of SRCPV to also install the fetcher
when SRCPV is expanded so that AUTOREV works.

Signed-off-by: Joshua Watt <JPEWhacker@gmail.com>
---
 meta/classes/crate-fetch.bbclass | 21 ++++++++++++++++++---
 1 file changed, 18 insertions(+), 3 deletions(-)

Comments

Joshua Watt Nov. 19, 2021, 2:14 a.m. UTC | #1
On 11/18/21 8:12 PM, Joshua Watt wrote:
> Reworks the crate fetcher class to have it install the fetcher at recipe
> finalization so that it is always available before SRC_URI is expanded.
> In addition, override the value of SRCPV to also install the fetcher
> when SRCPV is expanded so that AUTOREV works.
[YOCTO #10867]
> Signed-off-by: Joshua Watt <JPEWhacker@gmail.com>
> ---
>   meta/classes/crate-fetch.bbclass | 21 ++++++++++++++++++---
>   1 file changed, 18 insertions(+), 3 deletions(-)
>
> diff --git a/meta/classes/crate-fetch.bbclass b/meta/classes/crate-fetch.bbclass
> index c0ed434a96..a7fa22b2a0 100644
> --- a/meta/classes/crate-fetch.bbclass
> +++ b/meta/classes/crate-fetch.bbclass
> @@ -7,7 +7,22 @@
>   # crate://<packagename>/<version>
>   #
>   
> -python () {
> -        import crate
> -        bb.fetch2.methods.append( crate.Crate() )
> +def import_crate(d):
> +    import crate
> +    if not getattr(crate, 'imported', False):
> +        bb.fetch2.methods.append(crate.Crate())
> +        crate.imported = True
> +
> +python crate_import_handler() {
> +    import_crate(d)
>   }
> +
> +addhandler crate_import_handler
> +crate_import_handler[eventmask] = "bb.event.RecipePreFinalise"
> +
> +def crate_get_srcrev(d):
> +    import_crate(d)
> +    return bb.fetch2.get_srcrev(d)
> +
> +# Override SRCPV to make sure it imports the fetcher first
> +SRCPV = "${@crate_get_srcrev(d)}"

Patch

diff --git a/meta/classes/crate-fetch.bbclass b/meta/classes/crate-fetch.bbclass
index c0ed434a96..a7fa22b2a0 100644
--- a/meta/classes/crate-fetch.bbclass
+++ b/meta/classes/crate-fetch.bbclass
@@ -7,7 +7,22 @@ 
 # crate://<packagename>/<version>
 #
 
-python () {
-        import crate
-        bb.fetch2.methods.append( crate.Crate() )
+def import_crate(d):
+    import crate
+    if not getattr(crate, 'imported', False):
+        bb.fetch2.methods.append(crate.Crate())
+        crate.imported = True
+
+python crate_import_handler() {
+    import_crate(d)
 }
+
+addhandler crate_import_handler
+crate_import_handler[eventmask] = "bb.event.RecipePreFinalise"
+
+def crate_get_srcrev(d):
+    import_crate(d)
+    return bb.fetch2.get_srcrev(d)
+
+# Override SRCPV to make sure it imports the fetcher first
+SRCPV = "${@crate_get_srcrev(d)}"