* [PATCH] meta/classes/debianize: Add copyright file to debian folder
@ 2024-02-28 5:44 Srinuvasan A
2024-02-29 8:10 ` MOESSBAUER, Felix
` (2 more replies)
0 siblings, 3 replies; 10+ messages in thread
From: Srinuvasan A @ 2024-02-28 5:44 UTC (permalink / raw)
To: isar-users; +Cc: Srinuvasan A
From: Srinuvasan A <srinuvasan_a@mentor.com>
In downstream layer we uses copyright files for most of the packages,
this is very important for our OSS clearing.
User can add the copyright to the required recipe, then the generated
packages from the recipes contains the debian/copyright file.
Added example copyright file in example-raw recipe.
Signed-off-by: Srinuvasan A <srinuvasan_a@mentor.com>
---
.../recipes-app/example-raw/example-raw_0.3.bb | 3 ++-
.../example-raw/files/default-copyright | 15 +++++++++++++++
meta/classes/debianize.bbclass | 4 ++++
3 files changed, 21 insertions(+), 1 deletion(-)
create mode 100644 meta-isar/recipes-app/example-raw/files/default-copyright
diff --git a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
index ffa14340..35f4b3d9 100644
--- a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
+++ b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
@@ -9,7 +9,8 @@ DEBIAN_DEPENDS = "adduser, apt (>= 0.4.2)"
SRC_URI = "file://README \
file://postinst \
- file://rules"
+ file://rules \
+ file://default-copyright"
inherit dpkg-raw
diff --git a/meta-isar/recipes-app/example-raw/files/default-copyright b/meta-isar/recipes-app/example-raw/files/default-copyright
new file mode 100644
index 00000000..9c04255a
--- /dev/null
+++ b/meta-isar/recipes-app/example-raw/files/default-copyright
@@ -0,0 +1,15 @@
+Format: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
+Upstream-Name: ${PN}
+Upstream-Contact: Mentor Embedded <embedded_support@mentor.com>
+Source: https://github.com/MentorEmbedded/industrial-core
+
+Files: *
+Copyright: 2022, Siemens
+License: Siemens
+ .
+ This material contains trade secrets or otherwise confidential information
+ owned by Siemens Industry Software Inc.or its affiliates (collectively,
+ "Siemens"), or its licensors. Access to and use of this information is strictly
+ limited as set forth in the Customer's applicable agreements with Siemens.
+ .
+ Unpublished work. Copyright 2023 Siemens
diff --git a/meta/classes/debianize.bbclass b/meta/classes/debianize.bbclass
index 16f3638d..6039e869 100644
--- a/meta/classes/debianize.bbclass
+++ b/meta/classes/debianize.bbclass
@@ -121,6 +121,10 @@ deb_debianize() {
else
deb_create_rules
fi
+ # Add the copyright if unpacked sources does not contain copyright file
+ if [ ! -f ${S}/debian/copyright ] && [ -f ${WORKDIR}/default-copyright ]; then
+ install -v -m 644 ${WORKDIR}/default-copyright ${S}/debian/copyright
+ fi
# prepend a changelog-entry unless an existing changelog file already
# contains an entry with CHANGELOG_V
deb_add_changelog
--
2.34.1
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] meta/classes/debianize: Add copyright file to debian folder
2024-02-28 5:44 [PATCH] meta/classes/debianize: Add copyright file to debian folder Srinuvasan A
@ 2024-02-29 8:10 ` MOESSBAUER, Felix
2024-02-29 19:04 ` Florian Bezdeka
2024-03-14 9:12 ` Cedric Hombourger
2 siblings, 0 replies; 10+ messages in thread
From: MOESSBAUER, Felix @ 2024-02-29 8:10 UTC (permalink / raw)
To: isar-users, Arjunan, Srinu; +Cc: srinuvasan_a
On Wed, 2024-02-28 at 11:14 +0530, 'Srinuvasan A' via isar-users wrote:
> From: Srinuvasan A <srinuvasan_a@mentor.com>
>
> In downstream layer we uses copyright files for most of the packages,
> this is very important for our OSS clearing.
>
> User can add the copyright to the required recipe, then the generated
> packages from the recipes contains the debian/copyright file.
>
> Added example copyright file in example-raw recipe.
>
> Signed-off-by: Srinuvasan A <srinuvasan_a@mentor.com>
> ---
> .../recipes-app/example-raw/example-raw_0.3.bb | 3 ++-
> .../example-raw/files/default-copyright | 15
> +++++++++++++++
> meta/classes/debianize.bbclass | 4 ++++
> 3 files changed, 21 insertions(+), 1 deletion(-)
> create mode 100644 meta-isar/recipes-app/example-raw/files/default-
> copyright
>
> diff --git a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> index ffa14340..35f4b3d9 100644
> --- a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> +++ b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> @@ -9,7 +9,8 @@ DEBIAN_DEPENDS = "adduser, apt (>= 0.4.2)"
>
> SRC_URI = "file://README \
> file://postinst \
> - file://rules"
> + file://rules \
> + file://default-copyright"
>
> inherit dpkg-raw
>
> diff --git a/meta-isar/recipes-app/example-raw/files/default-
> copyright b/meta-isar/recipes-app/example-raw/files/default-copyright
> new file mode 100644
> index 00000000..9c04255a
> --- /dev/null
> +++ b/meta-isar/recipes-app/example-raw/files/default-copyright
> @@ -0,0 +1,15 @@
> +Format:
> https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
> +Upstream-Name: ${PN}
> +Upstream-Contact: Mentor Embedded <embedded_support@mentor.com>
> +Source: https://github.com/MentorEmbedded/industrial-core
> +
> +Files: *
> +Copyright: 2022, Siemens
> +License: Siemens
> + .
> + This material contains trade secrets or otherwise confidential
> information
> + owned by Siemens Industry Software Inc.or its affiliates
> (collectively,
> + "Siemens"), or its licensors. Access to and use of this information
> is strictly
> + limited as set forth in the Customer's applicable agreements with
> Siemens.
NACK!
Hi Srinuvasan,
please note, that ISAR is an OSS project and there we cannot simply set
Siemens internal defaults (also not in an example). Why not add the
copyright file (if available) in a similar way as we add the preinst /
postinst files?
Best regards,
Felix
> + .
> + Unpublished work. Copyright 2023 Siemens
> diff --git a/meta/classes/debianize.bbclass
> b/meta/classes/debianize.bbclass
> index 16f3638d..6039e869 100644
> --- a/meta/classes/debianize.bbclass
> +++ b/meta/classes/debianize.bbclass
> @@ -121,6 +121,10 @@ deb_debianize() {
> else
> deb_create_rules
> fi
> + # Add the copyright if unpacked sources does not contain
> copyright file
> + if [ ! -f ${S}/debian/copyright ] && [ -f ${WORKDIR}/default-
> copyright ]; then
> + install -v -m 644 ${WORKDIR}/default-copyright
> ${S}/debian/copyright
> + fi
> # prepend a changelog-entry unless an existing changelog file
> already
> # contains an entry with CHANGELOG_V
> deb_add_changelog
> --
> 2.34.1
>
--
Siemens AG, Technology
Linux Expert Center
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] meta/classes/debianize: Add copyright file to debian folder
2024-02-28 5:44 [PATCH] meta/classes/debianize: Add copyright file to debian folder Srinuvasan A
2024-02-29 8:10 ` MOESSBAUER, Felix
@ 2024-02-29 19:04 ` Florian Bezdeka
2024-03-14 9:12 ` Cedric Hombourger
2 siblings, 0 replies; 10+ messages in thread
From: Florian Bezdeka @ 2024-02-29 19:04 UTC (permalink / raw)
To: Srinuvasan A, isar-users; +Cc: Srinuvasan A
On Wed, 2024-02-28 at 11:14 +0530, 'Srinuvasan A' via isar-users wrote:
> From: Srinuvasan A <srinuvasan_a@mentor.com>
We can / should ignore this one as the following mail / patch has the
same content but a different sender?!?
Please send a new version (git format-patch -vX+1) in such cases. Helps
us to review patches a lot. Thanks!
Best regards,
Florian
>
> In downstream layer we uses copyright files for most of the packages,
> this is very important for our OSS clearing.
>
> User can add the copyright to the required recipe, then the generated
> packages from the recipes contains the debian/copyright file.
>
> Added example copyright file in example-raw recipe.
>
> Signed-off-by: Srinuvasan A <srinuvasan_a@mentor.com>
> ---
> .../recipes-app/example-raw/example-raw_0.3.bb | 3 ++-
> .../example-raw/files/default-copyright | 15 +++++++++++++++
> meta/classes/debianize.bbclass | 4 ++++
> 3 files changed, 21 insertions(+), 1 deletion(-)
> create mode 100644 meta-isar/recipes-app/example-raw/files/default-copyright
>
> diff --git a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> index ffa14340..35f4b3d9 100644
> --- a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> +++ b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> @@ -9,7 +9,8 @@ DEBIAN_DEPENDS = "adduser, apt (>= 0.4.2)"
>
> SRC_URI = "file://README \
> file://postinst \
> - file://rules"
> + file://rules \
> + file://default-copyright"
>
> inherit dpkg-raw
>
> diff --git a/meta-isar/recipes-app/example-raw/files/default-copyright b/meta-isar/recipes-app/example-raw/files/default-copyright
> new file mode 100644
> index 00000000..9c04255a
> --- /dev/null
> +++ b/meta-isar/recipes-app/example-raw/files/default-copyright
> @@ -0,0 +1,15 @@
> +Format: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
> +Upstream-Name: ${PN}
> +Upstream-Contact: Mentor Embedded <embedded_support@mentor.com>
> +Source: https://github.com/MentorEmbedded/industrial-core
> +
> +Files: *
> +Copyright: 2022, Siemens
> +License: Siemens
> + .
> + This material contains trade secrets or otherwise confidential information
> + owned by Siemens Industry Software Inc.or its affiliates (collectively,
> + "Siemens"), or its licensors. Access to and use of this information is strictly
> + limited as set forth in the Customer's applicable agreements with Siemens.
> + .
> + Unpublished work. Copyright 2023 Siemens
> diff --git a/meta/classes/debianize.bbclass b/meta/classes/debianize.bbclass
> index 16f3638d..6039e869 100644
> --- a/meta/classes/debianize.bbclass
> +++ b/meta/classes/debianize.bbclass
> @@ -121,6 +121,10 @@ deb_debianize() {
> else
> deb_create_rules
> fi
> + # Add the copyright if unpacked sources does not contain copyright file
> + if [ ! -f ${S}/debian/copyright ] && [ -f ${WORKDIR}/default-copyright ]; then
> + install -v -m 644 ${WORKDIR}/default-copyright ${S}/debian/copyright
> + fi
> # prepend a changelog-entry unless an existing changelog file already
> # contains an entry with CHANGELOG_V
> deb_add_changelog
> --
> 2.34.1
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] meta/classes/debianize: Add copyright file to debian folder
2024-02-28 5:44 [PATCH] meta/classes/debianize: Add copyright file to debian folder Srinuvasan A
2024-02-29 8:10 ` MOESSBAUER, Felix
2024-02-29 19:04 ` Florian Bezdeka
@ 2024-03-14 9:12 ` Cedric Hombourger
2024-03-14 12:09 ` srinu
2 siblings, 1 reply; 10+ messages in thread
From: Cedric Hombourger @ 2024-03-14 9:12 UTC (permalink / raw)
To: isar-users
[-- Attachment #1.1: Type: text/plain, Size: 4068 bytes --]
Le mercredi 28 février 2024 à 06:45:03 UTC+1, Srinuvasan A a écrit :
From: Srinuvasan A <srinuv...@mentor.com>
In downstream layer we uses copyright files for most of the packages,
this is very important for our OSS clearing.
User can add the copyright to the required recipe, then the generated
packages from the recipes contains the debian/copyright file.
Added example copyright file in example-raw recipe.
Signed-off-by: Srinuvasan A <srinuv...@mentor.com>
please use your @siemens.com address moving forward
please explicitly cc the main contributors of the "Isar extensions" from
our downstream project when upstreaming them (especially when you change
them)
they will get a chance to review (e.g. I am not subscribed to isar-users
and only check the archives once a week or so)
---
.../recipes-app/example-raw/example-raw_0.3.bb | 3 ++-
.../example-raw/files/default-copyright | 15 +++++++++++++++
meta/classes/debianize.bbclass | 4 ++++
3 files changed, 21 insertions(+), 1 deletion(-)
create mode 100644
meta-isar/recipes-app/example-raw/files/default-copyright
diff --git a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
index ffa14340..35f4b3d9 100644
--- a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
+++ b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
@@ -9,7 +9,8 @@ DEBIAN_DEPENDS = "adduser, apt (>= 0.4.2)"
SRC_URI = "file://README \
file://postinst \
- file://rules"
+ file://rules \
+ file://default-copyright"
the license helper in our downstream project was introduced to make it a
one-liner for dpkg-raw packages to get a debian/copyright especially when
they use "standard" licenses
IMO, debianize.bbclass should check ${LICENSE} if it matches a well-known
OSS license, add a default debian/copyright file (matching that license)
We should add default copyright texts for MIT, Apache-2, GPLv2, GPLv2+,
GPLv3, etc.
We should use templates (.tmpl) for all OSS licenses we provide a generic
debian/copyright file for
inherit dpkg-raw
diff --git a/meta-isar/recipes-app/example-raw/files/default-copyright
b/meta-isar/recipes-app/example-raw/files/default-copyright
new file mode 100644
index 00000000..9c04255a
--- /dev/null
+++ b/meta-isar/recipes-app/example-raw/files/default-copyright
@@ -0,0 +1,15 @@
+Format: https://www.de
<https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/>
bian.org/doc/packaging-manuals/copyright-format/1.0/
<https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/>
+Upstream-Name: ${PN}
${BPN}
+Upstream-Contact: Mentor Embedded <embedded...@mentor.com>
${MAINTAINER}
+Source: https://github.com/MentorEmbedded/industrial-core
${HOMEPAGE}
+
+Files: *
+Copyright: 2022, Siemens
Copyright: ${MAINTAINER}
+License: Siemens
Need one template per OSS license, we keep this license in-house
+ .
+ This material contains trade secrets or otherwise confidential
information
+ owned by Siemens Industry Software Inc.or its affiliates (collectively,
+ "Siemens"), or its licensors. Access to and use of this information is
strictly
+ limited as set forth in the Customer's applicable agreements with
Siemens.
+ .
+ Unpublished work. Copyright 2023 Siemens
diff --git a/meta/classes/debianize.bbclass
b/meta/classes/debianize.bbclass
index 16f3638d..6039e869 100644
--- a/meta/classes/debianize.bbclass
+++ b/meta/classes/debianize.bbclass
@@ -121,6 +121,10 @@ deb_debianize() {
else
deb_create_rules
fi
+ # Add the copyright if unpacked sources does not contain copyright file
+ if [ ! -f ${S}/debian/copyright ] && [ -f ${WORKDIR}/default-copyright ];
then
+ install -v -m 644 ${WORKDIR}/default-copyright ${S}/debian/copyright
+ fi
# prepend a changelog-entry unless an existing changelog file already
# contains an entry with CHANGELOG_V
deb_add_changelog
--
2.34.1
[-- Attachment #1.2: Type: text/html, Size: 6436 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] meta/classes/debianize: Add copyright file to debian folder
2024-03-14 9:12 ` Cedric Hombourger
@ 2024-03-14 12:09 ` srinu
0 siblings, 0 replies; 10+ messages in thread
From: srinu @ 2024-03-14 12:09 UTC (permalink / raw)
To: isar-users
[-- Attachment #1.1: Type: text/plain, Size: 4390 bytes --]
On Thursday, March 14, 2024 at 2:42:24 PM UTC+5:30 Cedric Hombourger wrote:
Le mercredi 28 février 2024 à 06:45:03 UTC+1, Srinuvasan A a écrit :
From: Srinuvasan A <srinuv...@mentor.com>
In downstream layer we uses copyright files for most of the packages,
this is very important for our OSS clearing.
User can add the copyright to the required recipe, then the generated
packages from the recipes contains the debian/copyright file.
Added example copyright file in example-raw recipe.
Signed-off-by: Srinuvasan A <srinuv...@mentor.com>
please use your @siemens.com address moving forward
Yes after got the special User ID and password am using the siemens
account, in this thread itself i changed to siemens ID.
please explicitly cc the main contributors of the "Isar extensions" from
our downstream project when upstreaming them (especially when you change
them)
they will get a chance to review (e.g. I am not subscribed to isar-users
and only check the archives once a week or so)
Sure, i will add the respective persons for the review.
---
.../recipes-app/example-raw/example-raw_0.3.bb | 3 ++-
.../example-raw/files/default-copyright | 15 +++++++++++++++
meta/classes/debianize.bbclass | 4 ++++
3 files changed, 21 insertions(+), 1 deletion(-)
create mode 100644
meta-isar/recipes-app/example-raw/files/default-copyright
diff --git a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
index ffa14340..35f4b3d9 100644
--- a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
+++ b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
@@ -9,7 +9,8 @@ DEBIAN_DEPENDS = "adduser, apt (>= 0.4.2)"
SRC_URI = "file://README \
file://postinst \
- file://rules"
+ file://rules \
+ file://default-copyright"
the license helper in our downstream project was introduced to make it a
one-liner for dpkg-raw packages to get a debian/copyright especially when
they use "standard" licenses
IMO, debianize.bbclass should check ${LICENSE} if it matches a well-known
OSS license, add a default debian/copyright file (matching that license)
We should add default copyright texts for MIT, Apache-2, GPLv2, GPLv2+,
GPLv3, etc.
We should use templates (.tmpl) for all OSS licenses we provide a generic
debian/copyright file for
Let me check this one
inherit dpkg-raw
diff --git a/meta-isar/recipes-app/example-raw/files/default-copyright
b/meta-isar/recipes-app/example-raw/files/default-copyright
new file mode 100644
index 00000000..9c04255a
--- /dev/null
+++ b/meta-isar/recipes-app/example-raw/files/default-copyright
@@ -0,0 +1,15 @@
+Format: https://www.de
<https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/>
bian.org/doc/packaging-manuals/copyright-format/1.0/
<https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/>
+Upstream-Name: ${PN}
${BPN}
+Upstream-Contact: Mentor Embedded <embedded...@mentor.com>
${MAINTAINER}
+Source: https://github.com/MentorEmbedded/industrial-core
${HOMEPAGE}
+
+Files: *
+Copyright: 2022, Siemens
Copyright: ${MAINTAINER}
+License: Siemens
Need one template per OSS license, we keep this license in-house
+ .
+ This material contains trade secrets or otherwise confidential
information
+ owned by Siemens Industry Software Inc.or its affiliates (collectively,
+ "Siemens"), or its licensors. Access to and use of this information is
strictly
+ limited as set forth in the Customer's applicable agreements with
Siemens.
+ .
+ Unpublished work. Copyright 2023 Siemens
diff --git a/meta/classes/debianize.bbclass
b/meta/classes/debianize.bbclass
index 16f3638d..6039e869 100644
--- a/meta/classes/debianize.bbclass
+++ b/meta/classes/debianize.bbclass
@@ -121,6 +121,10 @@ deb_debianize() {
else
deb_create_rules
fi
+ # Add the copyright if unpacked sources does not contain copyright file
+ if [ ! -f ${S}/debian/copyright ] && [ -f ${WORKDIR}/default-copyright ];
then
+ install -v -m 644 ${WORKDIR}/default-copyright ${S}/debian/copyright
+ fi
# prepend a changelog-entry unless an existing changelog file already
# contains an entry with CHANGELOG_V
deb_add_changelog
--
2.34.1
[-- Attachment #1.2: Type: text/html, Size: 7489 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] meta/classes/debianize: Add copyright file to debian folder
@ 2024-02-28 7:22 srinuvasan.a
2024-02-29 1:10 ` Jan Kiszka
2024-02-29 19:14 ` Florian Bezdeka
0 siblings, 2 replies; 10+ messages in thread
From: srinuvasan.a @ 2024-02-28 7:22 UTC (permalink / raw)
To: isar-users; +Cc: Srinuvasan A
From: Srinuvasan A <srinuvasan.a@siemens.com>
In downstream layer we uses copyright files for most of the packages,
this is very important for our OSS clearing.
User can add the copyright to the required recipe, then the generated
packages from the recipes contains the debian/copyright file.
Added example copyright file in example-raw recipe.
Signed-off-by: Srinuvasan A <srinuvasan.a@siemens.com>
---
.../recipes-app/example-raw/example-raw_0.3.bb | 3 ++-
.../example-raw/files/default-copyright.tmpl | 15 +++++++++++++++
meta/classes/debianize.bbclass | 4 ++++
3 files changed, 21 insertions(+), 1 deletion(-)
create mode 100644 meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
diff --git a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
index ffa14340..b4dfaa4d 100644
--- a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
+++ b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
@@ -9,7 +9,8 @@ DEBIAN_DEPENDS = "adduser, apt (>= 0.4.2)"
SRC_URI = "file://README \
file://postinst \
- file://rules"
+ file://rules \
+ file://default-copyright.tmpl"
inherit dpkg-raw
diff --git a/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl b/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
new file mode 100644
index 00000000..9c04255a
--- /dev/null
+++ b/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
@@ -0,0 +1,15 @@
+Format: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
+Upstream-Name: ${PN}
+Upstream-Contact: Mentor Embedded <embedded_support@mentor.com>
+Source: https://github.com/MentorEmbedded/industrial-core
+
+Files: *
+Copyright: 2022, Siemens
+License: Siemens
+ .
+ This material contains trade secrets or otherwise confidential information
+ owned by Siemens Industry Software Inc.or its affiliates (collectively,
+ "Siemens"), or its licensors. Access to and use of this information is strictly
+ limited as set forth in the Customer's applicable agreements with Siemens.
+ .
+ Unpublished work. Copyright 2023 Siemens
diff --git a/meta/classes/debianize.bbclass b/meta/classes/debianize.bbclass
index 7de98673..5b9cd9dc 100644
--- a/meta/classes/debianize.bbclass
+++ b/meta/classes/debianize.bbclass
@@ -112,6 +112,10 @@ deb_debianize() {
else
deb_create_rules
fi
+ # Add the copyright if unpacked sources does not contain copyright file
+ if [ ! -f ${S}/debian/copyright.tmpl ] && [ -f ${WORKDIR}/default-copyright.tmpl ]; then
+ install -v -m 644 ${WORKDIR}/default-copyright.tmpl ${S}/debian/copyright
+ fi
# prepend a changelog-entry unless an existing changelog file already
# contains an entry with CHANGELOG_V
deb_add_changelog
--
2.34.1
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] meta/classes/debianize: Add copyright file to debian folder
2024-02-28 7:22 srinuvasan.a
@ 2024-02-29 1:10 ` Jan Kiszka
2024-02-29 5:02 ` srinu
2024-02-29 19:14 ` Florian Bezdeka
1 sibling, 1 reply; 10+ messages in thread
From: Jan Kiszka @ 2024-02-29 1:10 UTC (permalink / raw)
To: srinuvasan.a, isar-users
On 28.02.24 08:22, srinuvasan.a via isar-users wrote:
> From: Srinuvasan A <srinuvasan.a@siemens.com>
>
> In downstream layer we uses copyright files for most of the packages,
> this is very important for our OSS clearing.
>
> User can add the copyright to the required recipe, then the generated
> packages from the recipes contains the debian/copyright file.
>
> Added example copyright file in example-raw recipe.
>
> Signed-off-by: Srinuvasan A <srinuvasan.a@siemens.com>
> ---
> .../recipes-app/example-raw/example-raw_0.3.bb | 3 ++-
> .../example-raw/files/default-copyright.tmpl | 15 +++++++++++++++
> meta/classes/debianize.bbclass | 4 ++++
> 3 files changed, 21 insertions(+), 1 deletion(-)
> create mode 100644 meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
>
> diff --git a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> index ffa14340..b4dfaa4d 100644
> --- a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> +++ b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> @@ -9,7 +9,8 @@ DEBIAN_DEPENDS = "adduser, apt (>= 0.4.2)"
>
> SRC_URI = "file://README \
> file://postinst \
> - file://rules"
> + file://rules \
> + file://default-copyright.tmpl"
>
> inherit dpkg-raw
>
> diff --git a/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl b/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
> new file mode 100644
> index 00000000..9c04255a
> --- /dev/null
> +++ b/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
> @@ -0,0 +1,15 @@
> +Format: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
> +Upstream-Name: ${PN}
> +Upstream-Contact: Mentor Embedded <embedded_support@mentor.com>
> +Source: https://github.com/MentorEmbedded/industrial-core
> +
> +Files: *
> +Copyright: 2022, Siemens
> +License: Siemens
> + .
> + This material contains trade secrets or otherwise confidential information
> + owned by Siemens Industry Software Inc.or its affiliates (collectively,
> + "Siemens"), or its licensors. Access to and use of this information is strictly
> + limited as set forth in the Customer's applicable agreements with Siemens.
> + .
> + Unpublished work. Copyright 2023 Siemens
Somehow I can't imagine you actually want to have this merged into an
open source project, do you? ;)
Jan
> diff --git a/meta/classes/debianize.bbclass b/meta/classes/debianize.bbclass
> index 7de98673..5b9cd9dc 100644
> --- a/meta/classes/debianize.bbclass
> +++ b/meta/classes/debianize.bbclass
> @@ -112,6 +112,10 @@ deb_debianize() {
> else
> deb_create_rules
> fi
> + # Add the copyright if unpacked sources does not contain copyright file
> + if [ ! -f ${S}/debian/copyright.tmpl ] && [ -f ${WORKDIR}/default-copyright.tmpl ]; then
> + install -v -m 644 ${WORKDIR}/default-copyright.tmpl ${S}/debian/copyright
> + fi
> # prepend a changelog-entry unless an existing changelog file already
> # contains an entry with CHANGELOG_V
> deb_add_changelog
--
Siemens AG, Technology
Linux Expert Center
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] meta/classes/debianize: Add copyright file to debian folder
2024-02-29 1:10 ` Jan Kiszka
@ 2024-02-29 5:02 ` srinu
2024-03-01 7:36 ` Jan Kiszka
0 siblings, 1 reply; 10+ messages in thread
From: srinu @ 2024-02-29 5:02 UTC (permalink / raw)
To: isar-users
[-- Attachment #1.1: Type: text/plain, Size: 3622 bytes --]
On Thursday, February 29, 2024 at 6:40:24 AM UTC+5:30 Jan Kiszka wrote:
On 28.02.24 08:22, srinuvasan.a via isar-users wrote:
> From: Srinuvasan A <srinuv...@siemens.com>
>
> In downstream layer we uses copyright files for most of the packages,
> this is very important for our OSS clearing.
>
> User can add the copyright to the required recipe, then the generated
> packages from the recipes contains the debian/copyright file.
>
> Added example copyright file in example-raw recipe.
>
> Signed-off-by: Srinuvasan A <srinuv...@siemens.com>
> ---
> .../recipes-app/example-raw/example-raw_0.3.bb | 3 ++-
> .../example-raw/files/default-copyright.tmpl | 15 +++++++++++++++
> meta/classes/debianize.bbclass | 4 ++++
> 3 files changed, 21 insertions(+), 1 deletion(-)
> create mode 100644
meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
>
> diff --git a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> index ffa14340..b4dfaa4d 100644
> --- a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> +++ b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> @@ -9,7 +9,8 @@ DEBIAN_DEPENDS = "adduser, apt (>= 0.4.2)"
>
> SRC_URI = "file://README \
> file://postinst \
> - file://rules"
> + file://rules \
> + file://default-copyright.tmpl"
>
> inherit dpkg-raw
>
> diff --git
a/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
b/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
> new file mode 100644
> index 00000000..9c04255a
> --- /dev/null
> +++ b/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
> @@ -0,0 +1,15 @@
> +Format:
https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
> +Upstream-Name: ${PN}
> +Upstream-Contact: Mentor Embedded <embedded...@mentor.com>
> +Source: https://github.com/MentorEmbedded/industrial-core
> +
> +Files: *
> +Copyright: 2022, Siemens
> +License: Siemens
> + .
> + This material contains trade secrets or otherwise confidential
information
> + owned by Siemens Industry Software Inc.or its affiliates (collectively,
> + "Siemens"), or its licensors. Access to and use of this information is
strictly
> + limited as set forth in the Customer's applicable agreements with
Siemens.
> + .
> + Unpublished work. Copyright 2023 Siemens
Somehow I can't imagine you actually want to have this merged into an
open source project, do you? ;)
Jan
This PR we proposed for our downstream layer, here we have n no.of
recipes which contains the LICENSE,
based on the LICENSE variable we are installing this copyright file
into the packages, and this would really
needed for our OSS clearing
Do you have any thoughts to address this one in ISAR?
> diff --git a/meta/classes/debianize.bbclass
b/meta/classes/debianize.bbclass
> index 7de98673..5b9cd9dc 100644
> --- a/meta/classes/debianize.bbclass
> +++ b/meta/classes/debianize.bbclass
> @@ -112,6 +112,10 @@ deb_debianize() {
> else
> deb_create_rules
> fi
> + # Add the copyright if unpacked sources does not contain copyright file
> + if [ ! -f ${S}/debian/copyright.tmpl ] && [ -f
${WORKDIR}/default-copyright.tmpl ]; then
> + install -v -m 644 ${WORKDIR}/default-copyright.tmpl
${S}/debian/copyright
> + fi
> # prepend a changelog-entry unless an existing changelog file already
> # contains an entry with CHANGELOG_V
> deb_add_changelog
--
Siemens AG, Technology
Linux Expert Center
[-- Attachment #1.2: Type: text/html, Size: 5321 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] meta/classes/debianize: Add copyright file to debian folder
2024-02-29 5:02 ` srinu
@ 2024-03-01 7:36 ` Jan Kiszka
0 siblings, 0 replies; 10+ messages in thread
From: Jan Kiszka @ 2024-03-01 7:36 UTC (permalink / raw)
To: srinu, isar-users
On 29.02.24 06:02, 'srinu' via isar-users wrote:
>
>
> On Thursday, February 29, 2024 at 6:40:24 AM UTC+5:30 Jan Kiszka wrote:
>
> On 28.02.24 08:22, srinuvasan.a via isar-users wrote:
> > From: Srinuvasan A <srinuv...@siemens.com>
> >
> > In downstream layer we uses copyright files for most of the packages,
> > this is very important for our OSS clearing.
> >
> > User can add the copyright to the required recipe, then the generated
> > packages from the recipes contains the debian/copyright file.
> >
> > Added example copyright file in example-raw recipe.
> >
> > Signed-off-by: Srinuvasan A <srinuv...@siemens.com>
> > ---
> > .../recipes-app/example-raw/example-raw_0.3.bb
> <http://example-raw_0.3.bb> | 3 ++-
> > .../example-raw/files/default-copyright.tmpl | 15 +++++++++++++++
> > meta/classes/debianize.bbclass | 4 ++++
> > 3 files changed, 21 insertions(+), 1 deletion(-)
> > create mode 100644
> meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
> >
> > diff --git a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> <http://example-raw_0.3.bb>
> b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> <http://example-raw_0.3.bb>
> > index ffa14340..b4dfaa4d 100644
> > --- a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> <http://example-raw_0.3.bb>
> > +++ b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> <http://example-raw_0.3.bb>
> > @@ -9,7 +9,8 @@ DEBIAN_DEPENDS = "adduser, apt (>= 0.4.2)"
> >
> > SRC_URI = "file://README \
> > file://postinst \
> > - file://rules"
> > + file://rules \
> > + file://default-copyright.tmpl"
> >
> > inherit dpkg-raw
> >
> > diff --git
> a/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
> b/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
> > new file mode 100644
> > index 00000000..9c04255a
> > --- /dev/null
> > +++ b/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
> > @@ -0,0 +1,15 @@
> > +Format:
> https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
> <https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/>
> > +Upstream-Name: ${PN}
> > +Upstream-Contact: Mentor Embedded <embedded...@mentor.com>
> > +Source: https://github.com/MentorEmbedded/industrial-core
> <https://github.com/MentorEmbedded/industrial-core>
> > +
> > +Files: *
> > +Copyright: 2022, Siemens
> > +License: Siemens
> > + .
> > + This material contains trade secrets or otherwise confidential
> information
> > + owned by Siemens Industry Software Inc.or its affiliates
> (collectively,
> > + "Siemens"), or its licensors. Access to and use of this
> information is strictly
> > + limited as set forth in the Customer's applicable agreements
> with Siemens.
> > + .
> > + Unpublished work. Copyright 2023 Siemens
>
> Somehow I can't imagine you actually want to have this merged into an
> open source project, do you? ;)
>
> Jan
>
>
> This PR we proposed for our downstream layer, here we have n no.of
> recipes which contains the LICENSE,
> based on the LICENSE variable we are installing this copyright file
> into the packages, and this would really
> needed for our OSS clearing
>
> Do you have any thoughts to address this one in ISAR?
>
There a two things to clearly tell apart here:
1) A mechanism to inject a default copyright file into an ad-hoc
debianized package
2) The default license text to use here, even if just as an example
Regarding 2), it is absolutely obvious for everyone thinking just a few
seconds that this can never be any proprietary, Siemens "infected" text.
If at all, this can only be the default license of the containing OSS
project, which would be either MIT or GPL here, more likely MIT.
Having 1) as additional feature of debianize.bbclass is possible, but
please first clarify which of the two versions you sent should be
reviewed. And as you have to fix the example license anyway, follow up
with a clearly marked v2 for that purpose.
Jan
--
Siemens AG, Technology
Linux Expert Center
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] meta/classes/debianize: Add copyright file to debian folder
2024-02-28 7:22 srinuvasan.a
2024-02-29 1:10 ` Jan Kiszka
@ 2024-02-29 19:14 ` Florian Bezdeka
1 sibling, 0 replies; 10+ messages in thread
From: Florian Bezdeka @ 2024-02-29 19:14 UTC (permalink / raw)
To: srinuvasan.a, isar-users; +Cc: felix.moessbauer
Hi,
On Wed, 2024-02-28 at 12:52 +0530, srinuvasan.a via isar-users wrote:
> From: Srinuvasan A <srinuvasan.a@siemens.com>
>
> In downstream layer we uses copyright files for most of the packages,
> this is very important for our OSS clearing.
>
> User can add the copyright to the required recipe, then the generated
> packages from the recipes contains the debian/copyright file.
>
> Added example copyright file in example-raw recipe.
>
> Signed-off-by: Srinuvasan A <srinuvasan.a@siemens.com>
> ---
> .../recipes-app/example-raw/example-raw_0.3.bb | 3 ++-
> .../example-raw/files/default-copyright.tmpl | 15 +++++++++++++++
> meta/classes/debianize.bbclass | 4 ++++
> 3 files changed, 21 insertions(+), 1 deletion(-)
> create mode 100644 meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
>
> diff --git a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> index ffa14340..b4dfaa4d 100644
> --- a/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> +++ b/meta-isar/recipes-app/example-raw/example-raw_0.3.bb
> @@ -9,7 +9,8 @@ DEBIAN_DEPENDS = "adduser, apt (>= 0.4.2)"
>
> SRC_URI = "file://README \
> file://postinst \
> - file://rules"
> + file://rules \
> + file://default-copyright.tmpl"
>
> inherit dpkg-raw
>
> diff --git a/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl b/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
> new file mode 100644
> index 00000000..9c04255a
> --- /dev/null
> +++ b/meta-isar/recipes-app/example-raw/files/default-copyright.tmpl
> @@ -0,0 +1,15 @@
> +Format: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
> +Upstream-Name: ${PN}
> +Upstream-Contact: Mentor Embedded <embedded_support@mentor.com>
> +Source: https://github.com/MentorEmbedded/industrial-core
Mentor or Siemens? And how did you make sure that "we" really hold the
copyright?
> +
> +Files: *
> +Copyright: 2022, Siemens
> +License: Siemens
Wrong year...
> + .
> + This material contains trade secrets or otherwise confidential information
> + owned by Siemens Industry Software Inc.or its affiliates (collectively,
> + "Siemens"), or its licensors. Access to and use of this information is strictly
> + limited as set forth in the Customer's applicable agreements with Siemens.
> + .
> + Unpublished work. Copyright 2023 Siemens
Nope. Now its published and in addition: wrong year...
> diff --git a/meta/classes/debianize.bbclass b/meta/classes/debianize.bbclass
> index 7de98673..5b9cd9dc 100644
> --- a/meta/classes/debianize.bbclass
> +++ b/meta/classes/debianize.bbclass
> @@ -112,6 +112,10 @@ deb_debianize() {
> else
> deb_create_rules
> fi
> + # Add the copyright if unpacked sources does not contain copyright file
> + if [ ! -f ${S}/debian/copyright.tmpl ] && [ -f ${WORKDIR}/default-copyright.tmpl ]; then
> + install -v -m 644 ${WORKDIR}/default-copyright.tmpl ${S}/debian/copyright
> + fi
This magic should be documented somewhere - after we agreed that this
is really the best way to add such a copyright file.
> # prepend a changelog-entry unless an existing changelog file already
> # contains an entry with CHANGELOG_V
> deb_add_changelog
> --
> 2.34.1
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2024-03-14 12:09 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-02-28 5:44 [PATCH] meta/classes/debianize: Add copyright file to debian folder Srinuvasan A
2024-02-29 8:10 ` MOESSBAUER, Felix
2024-02-29 19:04 ` Florian Bezdeka
2024-03-14 9:12 ` Cedric Hombourger
2024-03-14 12:09 ` srinu
2024-02-28 7:22 srinuvasan.a
2024-02-29 1:10 ` Jan Kiszka
2024-02-29 5:02 ` srinu
2024-03-01 7:36 ` Jan Kiszka
2024-02-29 19:14 ` Florian Bezdeka
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox