All of lore.kernel.org
 help / color / mirror / Atom feed
* [oe-core][PATCH v2 1/2] devtools: Initial recipe for repo 2.17.3
@ 2021-11-05 15:31 Jasper Orschulko
  2021-11-05 15:31 ` [oe-core][PATCH v2 2/2] base.bbclass: Add sysroot deps for repo fetcher Jasper Orschulko
  2021-11-05 16:11 ` [bitbake-devel] [oe-core][PATCH v2 1/2] devtools: Initial recipe for repo 2.17.3 Peter Kjellerstedt
  0 siblings, 2 replies; 5+ messages in thread
From: Jasper Orschulko @ 2021-11-05 15:31 UTC (permalink / raw)
  To: openembedded-core
  Cc: martin, Daniel.Baumgart, bitbake-devel, Jasper Orschulko

From: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>

Add a recipe for repo, prerequisite for the repo fetcher.

Signed-off-by: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
---
 .../repo/files/0001-python3-shebang.patch     | 21 ++++++++++++
 .../0001-Set-REPO_REV-to-v2.17.3.patch        | 33 +++++++++++++++++++
 meta/recipes-devtools/repo/repo.inc           | 25 ++++++++++++++
 meta/recipes-devtools/repo/repo_2.17.3.bb     |  7 ++++
 4 files changed, 86 insertions(+)
 create mode 100644 meta/recipes-devtools/repo/files/0001-python3-shebang.patch
 create mode 100644 meta/recipes-devtools/repo/repo-2.17.3/0001-Set-REPO_REV-to-v2.17.3.patch
 create mode 100644 meta/recipes-devtools/repo/repo.inc
 create mode 100644 meta/recipes-devtools/repo/repo_2.17.3.bb

diff --git a/meta/recipes-devtools/repo/files/0001-python3-shebang.patch b/meta/recipes-devtools/repo/files/0001-python3-shebang.patch
new file mode 100644
index 0000000000..09ccf58264
--- /dev/null
+++ b/meta/recipes-devtools/repo/files/0001-python3-shebang.patch
@@ -0,0 +1,21 @@
+From b8e84b202cd302a7c99288d3835dc9c63071f8f2 Mon Sep 17 00:00:00 2001
+From: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
+Date: Tue, 14 Sep 2021 16:46:51 +0200
+Subject: [PATCH] python3 shebang
+
+---
+ repo | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/repo b/repo
+index b13e34c..205e0e5 100755
+--- a/repo
++++ b/repo
+@@ -1,4 +1,4 @@
+-#!/usr/bin/env python
++#!/usr/bin/env python3
+ # -*- coding:utf-8 -*-
+ #
+ # Copyright (C) 2008 The Android Open Source Project
+--
+2.33.0
diff --git a/meta/recipes-devtools/repo/repo-2.17.3/0001-Set-REPO_REV-to-v2.17.3.patch b/meta/recipes-devtools/repo/repo-2.17.3/0001-Set-REPO_REV-to-v2.17.3.patch
new file mode 100644
index 0000000000..4d76bfc5d2
--- /dev/null
+++ b/meta/recipes-devtools/repo/repo-2.17.3/0001-Set-REPO_REV-to-v2.17.3.patch
@@ -0,0 +1,33 @@
+From bdd2a528da59c28db8ae2986834926de7cebf3ab Mon Sep 17 00:00:00 2001
+From: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
+Date: Thu, 4 Nov 2021 16:55:12 +0100
+Subject: [PATCH] Set REPO_REV to v2.17.3
+
+repo is an unusual tool because it downloads all of its own Python modules
+using GPG-signed git tags, and stores those files as part of the project
+that it is working with.
+
+So in order to have a reproducible repo installation within the project
+folders, we hardcode the REPO_REV variable to this recipes PV.
+
+Upstream-Status: Inappropriate [configuration]
+Signed-off-by: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
+---
+ repo | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/repo b/repo
+index 4cddbf1..cf5f6b1 100755
+--- a/repo
++++ b/repo
+@@ -142,7 +142,7 @@ if __name__ == '__main__':
+ REPO_URL = os.environ.get('REPO_URL', None)
+ if not REPO_URL:
+   REPO_URL = 'https://gerrit.googlesource.com/git-repo'
+-REPO_REV = os.environ.get('REPO_REV')
++REPO_REV = '11b30b91df1f0e03b53da970ec2588e85817bacc'
+ if not REPO_REV:
+   REPO_REV = 'stable'
+ # URL to file bug reports for repo tool issues.
+--
+2.33.1
diff --git a/meta/recipes-devtools/repo/repo.inc b/meta/recipes-devtools/repo/repo.inc
new file mode 100644
index 0000000000..60b32e4d74
--- /dev/null
+++ b/meta/recipes-devtools/repo/repo.inc
@@ -0,0 +1,25 @@
+# SPDX-License-Identifier: MIT
+# Copyright (C) 2021 iris-GmbH infrared & intelligent sensors
+
+SUMMARY = "Tool for managing many Git repositories"
+DESCRIPTION = "Repo is a tool built on top of Git. Repo helps manage many Git repositories, does the uploads to revision control systems, and automates parts of the development workflow."
+HOMEPAGE = "https://android.googlesource.com/tools/repo"
+SECTION = "console/utils"
+
+LICENSE = "Apache-2.0"
+
+SRC_URI = "git://git@gerrit.googlesource.com/git-repo.git;protocol=https;branch=main"
+MIRRORS = "git://git@gerrit.googlesource.com/git-repo.git git://github.com/GerritCodeReview/git-repo.git \n"
+
+SRC_URI += "file://0001-python3-shebang.patch"
+
+S = "${WORKDIR}/git"
+
+RDEPENDS_${PN} = "python3"
+
+do_install() {
+    install -d ${D}${bindir}
+    install -m 755 ${WORKDIR}/git/repo ${D}${bindir}
+}
+
+BBCLASSEXTEND = "native nativesdk"
diff --git a/meta/recipes-devtools/repo/repo_2.17.3.bb b/meta/recipes-devtools/repo/repo_2.17.3.bb
new file mode 100644
index 0000000000..c26264b9e9
--- /dev/null
+++ b/meta/recipes-devtools/repo/repo_2.17.3.bb
@@ -0,0 +1,7 @@
+# SPDX-License-Identifier: MIT
+# Copyright (C) 2021 iris-GmbH infrared & intelligent sensors
+
+require recipes-devtools/repo/repo.inc
+
+SRCREV = "11b30b91df1f0e03b53da970ec2588e85817bacc"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=3b83ef96387f14655fc854ddc3c6bd57"
-- 
2.33.1



^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [oe-core][PATCH v2 2/2] base.bbclass: Add sysroot deps for repo fetcher
  2021-11-05 15:31 [oe-core][PATCH v2 1/2] devtools: Initial recipe for repo 2.17.3 Jasper Orschulko
@ 2021-11-05 15:31 ` Jasper Orschulko
  2021-11-05 16:11 ` [bitbake-devel] [oe-core][PATCH v2 1/2] devtools: Initial recipe for repo 2.17.3 Peter Kjellerstedt
  1 sibling, 0 replies; 5+ messages in thread
From: Jasper Orschulko @ 2021-11-05 15:31 UTC (permalink / raw)
  To: openembedded-core
  Cc: martin, Daniel.Baumgart, bitbake-devel, Jasper Orschulko

From: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>

Add git-native and repo-native as prerequisite for the repo fetcher.

Signed-off-by: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
---
 meta/classes/base.bbclass | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
index a65fcc6c1d..3298bd1952 100644
--- a/meta/classes/base.bbclass
+++ b/meta/classes/base.bbclass
@@ -665,6 +665,11 @@ python () {
         elif uri.scheme == "npm":
             d.appendVarFlag('do_fetch', 'depends', ' nodejs-native:do_populate_sysroot')
 
+        elif uri.scheme == "repo":
+            needsrcrev = True
+            d.appendVarFlag('do_fetch', 'depends', ' repo-native:do_populate_sysroot')
+            d.appendVarFlag('do_fetch', 'depends', ' git-native:do_populate_sysroot')
+
         # *.lz4 should DEPEND on lz4-native for unpacking
         if path.endswith('.lz4'):
             d.appendVarFlag('do_unpack', 'depends', ' lz4-native:do_populate_sysroot')
-- 
2.33.1



^ permalink raw reply related	[flat|nested] 5+ messages in thread

* RE: [bitbake-devel] [oe-core][PATCH v2 1/2] devtools: Initial recipe for repo 2.17.3
  2021-11-05 15:31 [oe-core][PATCH v2 1/2] devtools: Initial recipe for repo 2.17.3 Jasper Orschulko
  2021-11-05 15:31 ` [oe-core][PATCH v2 2/2] base.bbclass: Add sysroot deps for repo fetcher Jasper Orschulko
@ 2021-11-05 16:11 ` Peter Kjellerstedt
  2021-11-05 16:32   ` Jasper Orschulko
  1 sibling, 1 reply; 5+ messages in thread
From: Peter Kjellerstedt @ 2021-11-05 16:11 UTC (permalink / raw)
  To: jasper, openembedded-core
  Cc: martin, Daniel.Baumgart, bitbake-devel, Jasper Orschulko

> -----Original Message-----
> From: bitbake-devel@lists.openembedded.org <bitbake-devel@lists.openembedded.org> On Behalf Of Jasper Orschulko via lists.openembedded.org
> Sent: den 5 november 2021 16:32
> To: openembedded-core@lists.openembedded.org
> Cc: martin@mko.dev; Daniel.Baumgart@iris-sensing.com; bitbake-devel@lists.openembedded.org; Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> Subject: [bitbake-devel] [oe-core][PATCH v2 1/2] devtools: Initial recipe for repo 2.17.3

Change the Git subject to:

repo: Add recipe for 2.17.3

> 
> From: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> 
> Add a recipe for repo, prerequisite for the repo fetcher.
> 
> Signed-off-by: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> ---
>  .../repo/files/0001-python3-shebang.patch     | 21 ++++++++++++
>  .../0001-Set-REPO_REV-to-v2.17.3.patch        | 33 +++++++++++++++++++
>  meta/recipes-devtools/repo/repo.inc           | 25 ++++++++++++++
>  meta/recipes-devtools/repo/repo_2.17.3.bb     |  7 ++++
>  4 files changed, 86 insertions(+)
>  create mode 100644 meta/recipes-devtools/repo/files/0001-python3-shebang.patch
>  create mode 100644 meta/recipes-devtools/repo/repo-2.17.3/0001-Set-REPO_REV-to-v2.17.3.patch
>  create mode 100644 meta/recipes-devtools/repo/repo.inc
>  create mode 100644 meta/recipes-devtools/repo/repo_2.17.3.bb
> 
> diff --git a/meta/recipes-devtools/repo/files/0001-python3-shebang.patch > b/meta/recipes-devtools/repo/files/0001-python3-shebang.patch
> new file mode 100644
> index 0000000000..09ccf58264
> --- /dev/null
> +++ b/meta/recipes-devtools/repo/files/0001-python3-shebang.patch
> @@ -0,0 +1,21 @@
> +From b8e84b202cd302a7c99288d3835dc9c63071f8f2 Mon Sep 17 00:00:00 2001
> +From: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> +Date: Tue, 14 Sep 2021 16:46:51 +0200
> +Subject: [PATCH] python3 shebang
> +
> +---
> + repo | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/repo b/repo
> +index b13e34c..205e0e5 100755
> +--- a/repo
> ++++ b/repo
> +@@ -1,4 +1,4 @@
> +-#!/usr/bin/env python
> ++#!/usr/bin/env python3
> + # -*- coding:utf-8 -*-
> + #
> + # Copyright (C) 2008 The Android Open Source Project
> +--
> +2.33.0
> diff --git a/meta/recipes-devtools/repo/repo-2.17.3/0001-Set-REPO_REV-to-v2.17.3.patch b/meta/recipes-devtools/repo/repo-2.17.3/0001-Set-REPO_REV-to-v2.17.3.patch
> new file mode 100644
> index 0000000000..4d76bfc5d2
> --- /dev/null
> +++ b/meta/recipes-devtools/repo/repo-2.17.3/0001-Set-REPO_REV-to-v2.17.3.patch
> @@ -0,0 +1,33 @@
> +From bdd2a528da59c28db8ae2986834926de7cebf3ab Mon Sep 17 00:00:00 2001
> +From: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> +Date: Thu, 4 Nov 2021 16:55:12 +0100
> +Subject: [PATCH] Set REPO_REV to v2.17.3
> +
> +repo is an unusual tool because it downloads all of its own Python modules
> +using GPG-signed git tags, and stores those files as part of the project
> +that it is working with.
> +
> +So in order to have a reproducible repo installation within the project
> +folders, we hardcode the REPO_REV variable to this recipes PV.

Change "this recipes PV" to "a SHA1 that corresponds to the version 
of the recipe."

> +
> +Upstream-Status: Inappropriate [configuration]
> +Signed-off-by: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> +---
> + repo | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/repo b/repo
> +index 4cddbf1..cf5f6b1 100755
> +--- a/repo
> ++++ b/repo
> +@@ -142,7 +142,7 @@ if __name__ == '__main__':
> + REPO_URL = os.environ.get('REPO_URL', None)
> + if not REPO_URL:
> +   REPO_URL = 'https://gerrit.googlesource.com/git-repo'
> +-REPO_REV = os.environ.get('REPO_REV')
> ++REPO_REV = '11b30b91df1f0e03b53da970ec2588e85817bacc'
> + if not REPO_REV:
> +   REPO_REV = 'stable'
> + # URL to file bug reports for repo tool issues.
> +--
> +2.33.1
> diff --git a/meta/recipes-devtools/repo/repo.inc b/meta/recipes-devtools/repo/repo.inc
> new file mode 100644
> index 0000000000..60b32e4d74
> --- /dev/null
> +++ b/meta/recipes-devtools/repo/repo.inc
> @@ -0,0 +1,25 @@
> +# SPDX-License-Identifier: MIT
> +# Copyright (C) 2021 iris-GmbH infrared & intelligent sensors
> +
> +SUMMARY = "Tool for managing many Git repositories"
> +DESCRIPTION = "Repo is a tool built on top of Git. Repo helps manage many Git repositories, does the uploads to revision control systems, and automates parts of the development workflow."
> +HOMEPAGE = "https://android.googlesource.com/tools/repo"
> +SECTION = "console/utils"
> +
> +LICENSE = "Apache-2.0"
> +
> +SRC_URI = "git://git@gerrit.googlesource.com/git-repo.git;protocol=https;branch=main"

I do not think you need to specify "git@" for this URL.

> +MIRRORS = "git://git@gerrit.googlesource.com/git-repo.git git://github.com/GerritCodeReview/git-repo.git \n"

It is probably better to use += and add to MIRRORS.

> +
> +SRC_URI += "file://0001-python3-shebang.patch"

I would rewrite the above lines as:

SRC_URI = "git://gerrit.googlesource.com/git-repo.git;protocol=https;branch=main"
SRC_URI += "file://0001-python3-shebang.patch"

MIRRORS += "git://gerrit.googlesource.com/git-repo.git git://github.com/GerritCodeReview/git-repo.git \n"

> +
> +S = "${WORKDIR}/git"
> +
> +RDEPENDS_${PN} = "python3"

Change to RDEPENDS:${PN} and move it to between do_install() and 
BBCLASSEXTEND.

> +
> +do_install() {
> +    install -d ${D}${bindir}
> +    install -m 755 ${WORKDIR}/git/repo ${D}${bindir}

Use tabs to indent shell code in OE-Core. You can also remove "-m 755" 
as that is the default for install.

> +}
> +
> +BBCLASSEXTEND = "native nativesdk"
> diff --git a/meta/recipes-devtools/repo/repo_2.17.3.bb b/meta/recipes-devtools/repo/repo_2.17.3.bb
> new file mode 100644
> index 0000000000..c26264b9e9
> --- /dev/null
> +++ b/meta/recipes-devtools/repo/repo_2.17.3.bb
> @@ -0,0 +1,7 @@
> +# SPDX-License-Identifier: MIT
> +# Copyright (C) 2021 iris-GmbH infrared & intelligent sensors
> +
> +require recipes-devtools/repo/repo.inc

Merge the repo.inc with the recipe as there is only one user of it.

> +
> +SRCREV = "11b30b91df1f0e03b53da970ec2588e85817bacc"
> +LIC_FILES_CHKSUM = "file://LICENSE;md5=3b83ef96387f14655fc854ddc3c6bd57"
> --
> 2.33.1

//Peter



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [bitbake-devel] [oe-core][PATCH v2 1/2] devtools: Initial recipe for repo 2.17.3
  2021-11-05 16:11 ` [bitbake-devel] [oe-core][PATCH v2 1/2] devtools: Initial recipe for repo 2.17.3 Peter Kjellerstedt
@ 2021-11-05 16:32   ` Jasper Orschulko
  2021-11-05 17:07     ` Peter Kjellerstedt
  0 siblings, 1 reply; 5+ messages in thread
From: Jasper Orschulko @ 2021-11-05 16:32 UTC (permalink / raw)
  To: openembedded-core, peter.kjellerstedt, jasper
  Cc: martin, Daniel Baumgart, bitbake-devel

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Hi Peter,

> Change to RDEPENDS:${PN} and move it to between do_install() and 
> BBCLASSEXTEND.

What does this do? The Yocto manual only mentions RDEPENDS_${PN}.

ack to the rest.

- -- 
With best regards

Jasper Orschulko
DevOps Engineer

Tel. +49 30 58 58 14 265
Fax +49 30 58 58 14 999
Jasper.Orschulko@iris-sensing.com

• • • • • • • • • • • • • • • • • • • • • • • • • •

iris-GmbH
infrared & intelligent sensors
Schnellerstraße 1-5 | 12439 Berlin

https://iris-sensing.com/





On Fri, 2021-11-05 at 16:11 +0000, Peter Kjellerstedt wrote:
> > -----Original Message-----
> > From:
> > bitbake-devel@lists.openembedded.org <bitbake-devel@lists.openembedded.org
> > > On Behalf Of Jasper Orschulko via lists.openembedded.org
> > Sent: den 5 november 2021 16:32
> > To: openembedded-core@lists.openembedded.org
> > Cc: martin@mko.dev; Daniel.Baumgart@iris-sensing.com;
> > bitbake-devel@lists.openembedded.org; Jasper Orschulko
> > <Jasper.Orschulko@iris-sensing.com>
> > Subject: [bitbake-devel] [oe-core][PATCH v2 1/2] devtools: Initial
> > recipe for repo 2.17.3
> 
> Change the Git subject to:
> 
> repo: Add recipe for 2.17.3
> 
> > 
> > From: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> > 
> > Add a recipe for repo, prerequisite for the repo fetcher.
> > 
> > Signed-off-by: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> > ---
> >  .../repo/files/0001-python3-shebang.patch     | 21 ++++++++++++
> >  .../0001-Set-REPO_REV-to-v2.17.3.patch        | 33
> > +++++++++++++++++++
> >  meta/recipes-devtools/repo/repo.inc           | 25 ++++++++++++++
> >  meta/recipes-devtools/repo/repo_2.17.3.bb     |  7 ++++
> >  4 files changed, 86 insertions(+)
> >  create mode 100644 meta/recipes-devtools/repo/files/0001-python3-
> > shebang.patch
> >  create mode 100644 meta/recipes-devtools/repo/repo-2.17.3/0001-Set-
> > REPO_REV-to-v2.17.3.patch
> >  create mode 100644 meta/recipes-devtools/repo/repo.inc
> >  create mode 100644 meta/recipes-devtools/repo/repo_2.17.3.bb
> > 
> > diff --git a/meta/recipes-devtools/repo/files/0001-python3-
> > shebang.patch > b/meta/recipes-devtools/repo/files/0001-python3-
> > shebang.patch
> > new file mode 100644
> > index 0000000000..09ccf58264
> > --- /dev/null
> > +++ b/meta/recipes-devtools/repo/files/0001-python3-shebang.patch
> > @@ -0,0 +1,21 @@
> > +From b8e84b202cd302a7c99288d3835dc9c63071f8f2 Mon Sep 17 00:00:00
> > 2001
> > +From: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> > +Date: Tue, 14 Sep 2021 16:46:51 +0200
> > +Subject: [PATCH] python3 shebang
> > +
> > +---
> > + repo | 2 +-
> > + 1 file changed, 1 insertion(+), 1 deletion(-)
> > +
> > +diff --git a/repo b/repo
> > +index b13e34c..205e0e5 100755
> > +--- a/repo
> > ++++ b/repo
> > +@@ -1,4 +1,4 @@
> > +-#!/usr/bin/env python
> > ++#!/usr/bin/env python3
> > + # -*- coding:utf-8 -*-
> > + #
> > + # Copyright (C) 2008 The Android Open Source Project
> > +--
> > +2.33.0
> > diff --git a/meta/recipes-devtools/repo/repo-2.17.3/0001-Set-
> > REPO_REV-to-v2.17.3.patch b/meta/recipes-devtools/repo/repo-
> > 2.17.3/0001-Set-REPO_REV-to-v2.17.3.patch
> > new file mode 100644
> > index 0000000000..4d76bfc5d2
> > --- /dev/null
> > +++ b/meta/recipes-devtools/repo/repo-2.17.3/0001-Set-REPO_REV-to-
> > v2.17.3.patch
> > @@ -0,0 +1,33 @@
> > +From bdd2a528da59c28db8ae2986834926de7cebf3ab Mon Sep 17 00:00:00
> > 2001
> > +From: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> > +Date: Thu, 4 Nov 2021 16:55:12 +0100
> > +Subject: [PATCH] Set REPO_REV to v2.17.3
> > +
> > +repo is an unusual tool because it downloads all of its own Python
> > modules
> > +using GPG-signed git tags, and stores those files as part of the
> > project
> > +that it is working with.
> > +
> > +So in order to have a reproducible repo installation within the
> > project
> > +folders, we hardcode the REPO_REV variable to this recipes PV.
> 
> Change "this recipes PV" to "a SHA1 that corresponds to the version 
> of the recipe."
> 
> > +
> > +Upstream-Status: Inappropriate [configuration]
> > +Signed-off-by: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> > +---
> > + repo | 2 +-
> > + 1 file changed, 1 insertion(+), 1 deletion(-)
> > +
> > +diff --git a/repo b/repo
> > +index 4cddbf1..cf5f6b1 100755
> > +--- a/repo
> > ++++ b/repo
> > +@@ -142,7 +142,7 @@ if __name__ == '__main__':
> > + REPO_URL = os.environ.get('REPO_URL', None)
> > + if not REPO_URL:
> > +   REPO_URL = 'https://gerrit.googlesource.com/git-repo'
> > +-REPO_REV = os.environ.get('REPO_REV')
> > ++REPO_REV = '11b30b91df1f0e03b53da970ec2588e85817bacc'
> > + if not REPO_REV:
> > +   REPO_REV = 'stable'
> > + # URL to file bug reports for repo tool issues.
> > +--
> > +2.33.1
> > diff --git a/meta/recipes-devtools/repo/repo.inc b/meta/recipes-
> > devtools/repo/repo.inc
> > new file mode 100644
> > index 0000000000..60b32e4d74
> > --- /dev/null
> > +++ b/meta/recipes-devtools/repo/repo.inc
> > @@ -0,0 +1,25 @@
> > +# SPDX-License-Identifier: MIT
> > +# Copyright (C) 2021 iris-GmbH infrared & intelligent sensors
> > +
> > +SUMMARY = "Tool for managing many Git repositories"
> > +DESCRIPTION = "Repo is a tool built on top of Git. Repo helps manage
> > many Git repositories, does the uploads to revision control systems,
> > and automates parts of the development workflow."
> > +HOMEPAGE = "https://android.googlesource.com/tools/repo"
> > +SECTION = "console/utils"
> > +
> > +LICENSE = "Apache-2.0"
> > +
> > +SRC_URI = "git://git@gerrit.googlesource.com/git-
> > repo.git;protocol=https;branch=main"
> 
> I do not think you need to specify "git@" for this URL.
> 
> > +MIRRORS = "git://git@gerrit.googlesource.com/git-repo.git
> > git://github.com/GerritCodeReview/git-repo.git \n"
> 
> It is probably better to use += and add to MIRRORS.
> 
> > +
> > +SRC_URI += "file://0001-python3-shebang.patch"
> 
> I would rewrite the above lines as:
> 
> SRC_URI = "git://gerrit.googlesource.com/git-
> repo.git;protocol=https;branch=main"
> SRC_URI += "file://0001-python3-shebang.patch"
> 
> MIRRORS += "git://gerrit.googlesource.com/git-repo.git
> git://github.com/GerritCodeReview/git-repo.git \n"
> 
> > +
> > +S = "${WORKDIR}/git"
> > +
> > +RDEPENDS_${PN} = "python3"
> 
> Change to RDEPENDS:${PN} and move it to between do_install() and 
> BBCLASSEXTEND.
> 
> > +
> > +do_install() {
> > +    install -d ${D}${bindir}
> > +    install -m 755 ${WORKDIR}/git/repo ${D}${bindir}
> 
> Use tabs to indent shell code in OE-Core. You can also remove "-m 755" 
> as that is the default for install.
> 
> > +}
> > +
> > +BBCLASSEXTEND = "native nativesdk"
> > diff --git a/meta/recipes-devtools/repo/repo_2.17.3.bb
> > b/meta/recipes-devtools/repo/repo_2.17.3.bb
> > new file mode 100644
> > index 0000000000..c26264b9e9
> > --- /dev/null
> > +++ b/meta/recipes-devtools/repo/repo_2.17.3.bb
> > @@ -0,0 +1,7 @@
> > +# SPDX-License-Identifier: MIT
> > +# Copyright (C) 2021 iris-GmbH infrared & intelligent sensors
> > +
> > +require recipes-devtools/repo/repo.inc
> 
> Merge the repo.inc with the recipe as there is only one user of it.
> 
> > +
> > +SRCREV = "11b30b91df1f0e03b53da970ec2588e85817bacc"
> > +LIC_FILES_CHKSUM =
> > "file://LICENSE;md5=3b83ef96387f14655fc854ddc3c6bd57"
> > --
> > 2.33.1
> 
> //Peter
> 
-----BEGIN PGP SIGNATURE-----

iQEzBAEBCAAdFiEE4WyPMIC5Ap4+Ooo1Ygqew07VMNUFAmGFXJAACgkQYgqew07V
MNVx1wf+J5fXl6SlC2C4N6F48fnxW0bK4v+xaKeUjlQJ6oN2i6SDAgHKix54SOhz
z8RbepU3GHikSk2/GXn/cX6gNzG8sxBfAbhT0MeTjiJEfbhuyqYWo3W+i5t8NPuL
76Gm5ya4ljnATItAhIgBC1WyGgqbpMCajB82gLw1XEC6mlD4fUoma2SBIWfGh94y
jbU6ixUKF0ulmSyK2q20yzXjq6jd++ShDaYeSPx5P1s0C1NeWoTM8llaaagnJ3R1
Cnj/uFVff7a46i4yd4pbK6o4oj8sNoBcBpRkX9szHBF4xALF9v6sAAD4qkYtGwq0
CwN3psutY1op4hHS4OFCJg8Vl8AHpw==
=bwdZ
-----END PGP SIGNATURE-----

^ permalink raw reply	[flat|nested] 5+ messages in thread

* RE: [bitbake-devel] [oe-core][PATCH v2 1/2] devtools: Initial recipe for repo 2.17.3
  2021-11-05 16:32   ` Jasper Orschulko
@ 2021-11-05 17:07     ` Peter Kjellerstedt
  0 siblings, 0 replies; 5+ messages in thread
From: Peter Kjellerstedt @ 2021-11-05 17:07 UTC (permalink / raw)
  To: Jasper Orschulko, openembedded-core, jasper
  Cc: martin, Daniel Baumgart, bitbake-devel

> -----Original Message-----
> From: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> Sent: den 5 november 2021 17:32
> To: openembedded-core@lists.openembedded.org; Peter Kjellerstedt
> <peter.kjellerstedt@axis.com>; jasper@fancydomain.eu
> Cc: martin@mko.dev; Daniel Baumgart <Daniel.Baumgart@iris-sensing.com>;
> bitbake-devel@lists.openembedded.org
> Subject: Re: [bitbake-devel] [oe-core][PATCH v2 1/2] devtools: Initial
> recipe for repo 2.17.3
> 
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA256
> 
> Hi Peter,
> 
> > Change to RDEPENDS:${PN} and move it to between do_install() and
> > BBCLASSEXTEND.
> 
> What does this do? The Yocto manual only mentions RDEPENDS_${PN}.

The syntax for bitbake variable overrides were changed with the 
Honister release, see: 

https://docs.yoctoproject.org/3.4/migration-guides/migration-3.4.html#override-syntax-changes

//Peter

> 
> ack to the rest.
> 
> - --
> With best regards
> 
> Jasper Orschulko
> DevOps Engineer
> 
> Tel. +49 30 58 58 14 265
> Fax +49 30 58 58 14 999
> Jasper.Orschulko@iris-sensing.com
> 
> • • • • • • • • • • • • • • • • • • • • • • • • • •
> 
> iris-GmbH
> infrared & intelligent sensors
> Schnellerstraße 1-5 | 12439 Berlin
> 
> https://iris-sensing.com/
> 
> 
> 
> 
> 
> On Fri, 2021-11-05 at 16:11 +0000, Peter Kjellerstedt wrote:
> > > -----Original Message-----
> > > From:
> > > bitbake-devel@lists.openembedded.org <bitbake-
> devel@lists.openembedded.org
> > > > On Behalf Of Jasper Orschulko via lists.openembedded.org
> > > Sent: den 5 november 2021 16:32
> > > To: openembedded-core@lists.openembedded.org
> > > Cc: martin@mko.dev; Daniel.Baumgart@iris-sensing.com;
> > > bitbake-devel@lists.openembedded.org; Jasper Orschulko
> > > <Jasper.Orschulko@iris-sensing.com>
> > > Subject: [bitbake-devel] [oe-core][PATCH v2 1/2] devtools: Initial
> > > recipe for repo 2.17.3
> >
> > Change the Git subject to:
> >
> > repo: Add recipe for 2.17.3
> >
> > >
> > > From: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> > >
> > > Add a recipe for repo, prerequisite for the repo fetcher.
> > >
> > > Signed-off-by: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> > > ---
> > >  .../repo/files/0001-python3-shebang.patch     | 21 ++++++++++++
> > >  .../0001-Set-REPO_REV-to-v2.17.3.patch        | 33
> > > +++++++++++++++++++
> > >  meta/recipes-devtools/repo/repo.inc           | 25 ++++++++++++++
> > >  meta/recipes-devtools/repo/repo_2.17.3.bb     |  7 ++++
> > >  4 files changed, 86 insertions(+)
> > >  create mode 100644 meta/recipes-devtools/repo/files/0001-python3-
> > > shebang.patch
> > >  create mode 100644 meta/recipes-devtools/repo/repo-2.17.3/0001-Set-
> > > REPO_REV-to-v2.17.3.patch
> > >  create mode 100644 meta/recipes-devtools/repo/repo.inc
> > >  create mode 100644 meta/recipes-devtools/repo/repo_2.17.3.bb
> > >
> > > diff --git a/meta/recipes-devtools/repo/files/0001-python3-
> > > shebang.patch > b/meta/recipes-devtools/repo/files/0001-python3-
> > > shebang.patch
> > > new file mode 100644
> > > index 0000000000..09ccf58264
> > > --- /dev/null
> > > +++ b/meta/recipes-devtools/repo/files/0001-python3-shebang.patch
> > > @@ -0,0 +1,21 @@
> > > +From b8e84b202cd302a7c99288d3835dc9c63071f8f2 Mon Sep 17 00:00:00
> > > 2001
> > > +From: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> > > +Date: Tue, 14 Sep 2021 16:46:51 +0200
> > > +Subject: [PATCH] python3 shebang
> > > +
> > > +---
> > > + repo | 2 +-
> > > + 1 file changed, 1 insertion(+), 1 deletion(-)
> > > +
> > > +diff --git a/repo b/repo
> > > +index b13e34c..205e0e5 100755
> > > +--- a/repo
> > > ++++ b/repo
> > > +@@ -1,4 +1,4 @@
> > > +-#!/usr/bin/env python
> > > ++#!/usr/bin/env python3
> > > + # -*- coding:utf-8 -*-
> > > + #
> > > + # Copyright (C) 2008 The Android Open Source Project
> > > +--
> > > +2.33.0
> > > diff --git a/meta/recipes-devtools/repo/repo-2.17.3/0001-Set-
> > > REPO_REV-to-v2.17.3.patch b/meta/recipes-devtools/repo/repo-
> > > 2.17.3/0001-Set-REPO_REV-to-v2.17.3.patch
> > > new file mode 100644
> > > index 0000000000..4d76bfc5d2
> > > --- /dev/null
> > > +++ b/meta/recipes-devtools/repo/repo-2.17.3/0001-Set-REPO_REV-to-
> > > v2.17.3.patch
> > > @@ -0,0 +1,33 @@
> > > +From bdd2a528da59c28db8ae2986834926de7cebf3ab Mon Sep 17 00:00:00
> > > 2001
> > > +From: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> > > +Date: Thu, 4 Nov 2021 16:55:12 +0100
> > > +Subject: [PATCH] Set REPO_REV to v2.17.3
> > > +
> > > +repo is an unusual tool because it downloads all of its own Python
> > > modules
> > > +using GPG-signed git tags, and stores those files as part of the
> > > project
> > > +that it is working with.
> > > +
> > > +So in order to have a reproducible repo installation within the
> > > project
> > > +folders, we hardcode the REPO_REV variable to this recipes PV.
> >
> > Change "this recipes PV" to "a SHA1 that corresponds to the version
> > of the recipe."
> >
> > > +
> > > +Upstream-Status: Inappropriate [configuration]
> > > +Signed-off-by: Jasper Orschulko <Jasper.Orschulko@iris-sensing.com>
> > > +---
> > > + repo | 2 +-
> > > + 1 file changed, 1 insertion(+), 1 deletion(-)
> > > +
> > > +diff --git a/repo b/repo
> > > +index 4cddbf1..cf5f6b1 100755
> > > +--- a/repo
> > > ++++ b/repo
> > > +@@ -142,7 +142,7 @@ if __name__ == '__main__':
> > > + REPO_URL = os.environ.get('REPO_URL', None)
> > > + if not REPO_URL:
> > > +   REPO_URL = 'https://gerrit.googlesource.com/git-repo'
> > > +-REPO_REV = os.environ.get('REPO_REV')
> > > ++REPO_REV = '11b30b91df1f0e03b53da970ec2588e85817bacc'
> > > + if not REPO_REV:
> > > +   REPO_REV = 'stable'
> > > + # URL to file bug reports for repo tool issues.
> > > +--
> > > +2.33.1
> > > diff --git a/meta/recipes-devtools/repo/repo.inc b/meta/recipes-
> > > devtools/repo/repo.inc
> > > new file mode 100644
> > > index 0000000000..60b32e4d74
> > > --- /dev/null
> > > +++ b/meta/recipes-devtools/repo/repo.inc
> > > @@ -0,0 +1,25 @@
> > > +# SPDX-License-Identifier: MIT
> > > +# Copyright (C) 2021 iris-GmbH infrared & intelligent sensors
> > > +
> > > +SUMMARY = "Tool for managing many Git repositories"
> > > +DESCRIPTION = "Repo is a tool built on top of Git. Repo helps manage
> > > many Git repositories, does the uploads to revision control systems,
> > > and automates parts of the development workflow."
> > > +HOMEPAGE = "https://android.googlesource.com/tools/repo"
> > > +SECTION = "console/utils"
> > > +
> > > +LICENSE = "Apache-2.0"
> > > +
> > > +SRC_URI = "git://git@gerrit.googlesource.com/git-
> > > repo.git;protocol=https;branch=main"
> >
> > I do not think you need to specify "git@" for this URL.
> >
> > > +MIRRORS = "git://git@gerrit.googlesource.com/git-repo.git
> > > git://github.com/GerritCodeReview/git-repo.git \n"
> >
> > It is probably better to use += and add to MIRRORS.
> >
> > > +
> > > +SRC_URI += "file://0001-python3-shebang.patch"
> >
> > I would rewrite the above lines as:
> >
> > SRC_URI = "git://gerrit.googlesource.com/git-
> > repo.git;protocol=https;branch=main"
> > SRC_URI += "file://0001-python3-shebang.patch"
> >
> > MIRRORS += "git://gerrit.googlesource.com/git-repo.git
> > git://github.com/GerritCodeReview/git-repo.git \n"
> >
> > > +
> > > +S = "${WORKDIR}/git"
> > > +
> > > +RDEPENDS_${PN} = "python3"
> >
> > Change to RDEPENDS:${PN} and move it to between do_install() and
> > BBCLASSEXTEND.
> >
> > > +
> > > +do_install() {
> > > +    install -d ${D}${bindir}
> > > +    install -m 755 ${WORKDIR}/git/repo ${D}${bindir}
> >
> > Use tabs to indent shell code in OE-Core. You can also remove "-m 755"
> > as that is the default for install.
> >
> > > +}
> > > +
> > > +BBCLASSEXTEND = "native nativesdk"
> > > diff --git a/meta/recipes-devtools/repo/repo_2.17.3.bb
> > > b/meta/recipes-devtools/repo/repo_2.17.3.bb
> > > new file mode 100644
> > > index 0000000000..c26264b9e9
> > > --- /dev/null
> > > +++ b/meta/recipes-devtools/repo/repo_2.17.3.bb
> > > @@ -0,0 +1,7 @@
> > > +# SPDX-License-Identifier: MIT
> > > +# Copyright (C) 2021 iris-GmbH infrared & intelligent sensors
> > > +
> > > +require recipes-devtools/repo/repo.inc
> >
> > Merge the repo.inc with the recipe as there is only one user of it.
> >
> > > +
> > > +SRCREV = "11b30b91df1f0e03b53da970ec2588e85817bacc"
> > > +LIC_FILES_CHKSUM =
> > > "file://LICENSE;md5=3b83ef96387f14655fc854ddc3c6bd57"
> > > --
> > > 2.33.1
> >
> > //Peter
> >
> -----BEGIN PGP SIGNATURE-----
> 
> iQEzBAEBCAAdFiEE4WyPMIC5Ap4+Ooo1Ygqew07VMNUFAmGFXJAACgkQYgqew07V
> MNVx1wf+J5fXl6SlC2C4N6F48fnxW0bK4v+xaKeUjlQJ6oN2i6SDAgHKix54SOhz
> z8RbepU3GHikSk2/GXn/cX6gNzG8sxBfAbhT0MeTjiJEfbhuyqYWo3W+i5t8NPuL
> 76Gm5ya4ljnATItAhIgBC1WyGgqbpMCajB82gLw1XEC6mlD4fUoma2SBIWfGh94y
> jbU6ixUKF0ulmSyK2q20yzXjq6jd++ShDaYeSPx5P1s0C1NeWoTM8llaaagnJ3R1
> Cnj/uFVff7a46i4yd4pbK6o4oj8sNoBcBpRkX9szHBF4xALF9v6sAAD4qkYtGwq0
> CwN3psutY1op4hHS4OFCJg8Vl8AHpw==
> =bwdZ
> -----END PGP SIGNATURE-----

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2021-11-05 17:07 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-05 15:31 [oe-core][PATCH v2 1/2] devtools: Initial recipe for repo 2.17.3 Jasper Orschulko
2021-11-05 15:31 ` [oe-core][PATCH v2 2/2] base.bbclass: Add sysroot deps for repo fetcher Jasper Orschulko
2021-11-05 16:11 ` [bitbake-devel] [oe-core][PATCH v2 1/2] devtools: Initial recipe for repo 2.17.3 Peter Kjellerstedt
2021-11-05 16:32   ` Jasper Orschulko
2021-11-05 17:07     ` Peter Kjellerstedt

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.