* [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V
@ 2020-04-23 16:31 Jan Kiszka
2020-04-23 16:43 ` Henning Schild
` (2 more replies)
0 siblings, 3 replies; 13+ messages in thread
From: Jan Kiszka @ 2020-04-23 16:31 UTC (permalink / raw)
To: isar-users; +Cc: Henning Schild
From: Jan Kiszka <jan.kiszka@siemens.com>
This allows to append to the latest upstream version when using an
unpinned source version. The format of the placeholder is
"<orig-version>".
Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
---
Changes in v3:
- fix regression when placeholder was not used
meta/classes/debianize.bbclass | 13 +++++++++++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/meta/classes/debianize.bbclass b/meta/classes/debianize.bbclass
index da43a4e2..e77be232 100644
--- a/meta/classes/debianize.bbclass
+++ b/meta/classes/debianize.bbclass
@@ -11,6 +11,15 @@ DESCRIPTION ?= "must not be empty"
MAINTAINER ?= "Unknown maintainer <unknown@example.com>"
deb_add_changelog() {
+ changelog_v="${CHANGELOG_V}"
+ if [ -f ${S}/debian/changelog ]; then
+ if [ ! -f ${WORKDIR}/changelog.orig ]; then
+ cp ${S}/debian/changelog ${WORKDIR}/changelog.orig
+ fi
+ orig_version=$(dpkg-parsechangelog -l ${WORKDIR}/changelog.orig -S Version)
+ changelog_v=$(echo "${changelog_v}" | sed 's/<orig-version>/'${orig_version}'/')
+ fi
+
timestamp=$(find ${S}/ -type f -not -path "${S}/debian/*" -printf "%T@\n"|sort -n -r|head -n 1)
if [ -n "${timestamp}" ]; then
date=$(LANG=C date -R -d @${timestamp})
@@ -18,7 +27,7 @@ deb_add_changelog() {
date=$(LANG=C date -R)
fi
cat <<EOF > ${S}/debian/changelog
-${PN} (${CHANGELOG_V}) UNRELEASED; urgency=low
+${PN} (${changelog_v}) UNRELEASED; urgency=low
* generated by Isar
@@ -26,7 +35,7 @@ ${PN} (${CHANGELOG_V}) UNRELEASED; urgency=low
EOF
if [ -f ${WORKDIR}/changelog ]; then
if head -1 "${WORKDIR}"/changelog | \
- grep -q -e "^${PN} (${CHANGELOG_V})"
+ grep -q -e "^${PN} (${changelog_v})"
then
# entry for our version already there, use unmodified
rm ${S}/debian/changelog
--
2.16.4
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V
2020-04-23 16:31 [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V Jan Kiszka
@ 2020-04-23 16:43 ` Henning Schild
2020-04-23 16:50 ` Jan Kiszka
2020-04-23 16:47 ` Henning Schild
2020-05-07 19:45 ` Baurzhan Ismagulov
2 siblings, 1 reply; 13+ messages in thread
From: Henning Schild @ 2020-04-23 16:43 UTC (permalink / raw)
To: Jan Kiszka; +Cc: isar-users
On Thu, 23 Apr 2020 18:31:10 +0200
Jan Kiszka <jan.kiszka@siemens.com> wrote:
> From: Jan Kiszka <jan.kiszka@siemens.com>
>
> This allows to append to the latest upstream version when using an
> unpinned source version. The format of the placeholder is
> "<orig-version>".
>
> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
> ---
>
> Changes in v3:
> - fix regression when placeholder was not used
>
> meta/classes/debianize.bbclass | 13 +++++++++++--
> 1 file changed, 11 insertions(+), 2 deletions(-)
>
> diff --git a/meta/classes/debianize.bbclass
> b/meta/classes/debianize.bbclass index da43a4e2..e77be232 100644
> --- a/meta/classes/debianize.bbclass
> +++ b/meta/classes/debianize.bbclass
> @@ -11,6 +11,15 @@ DESCRIPTION ?= "must not be empty"
> MAINTAINER ?= "Unknown maintainer <unknown@example.com>"
>
> deb_add_changelog() {
> + changelog_v="${CHANGELOG_V}"
> + if [ -f ${S}/debian/changelog ]; then
> + if [ ! -f ${WORKDIR}/changelog.orig ]; then
> + cp ${S}/debian/changelog
> ${WORKDIR}/changelog.orig
> + fi
> + orig_version=$(dpkg-parsechangelog -l
> ${WORKDIR}/changelog.orig -S Version)
> + changelog_v=$(echo "${changelog_v}" | sed
> 's/<orig-version>/'${orig_version}'/')
> + fi
> +
> timestamp=$(find ${S}/ -type f -not -path "${S}/debian/*"
> -printf "%T@\n"|sort -n -r|head -n 1) if [ -n "${timestamp}" ]; then
> date=$(LANG=C date -R -d @${timestamp})
> @@ -18,7 +27,7 @@ deb_add_changelog() {
> date=$(LANG=C date -R)
> fi
> cat <<EOF > ${S}/debian/changelog
> -${PN} (${CHANGELOG_V}) UNRELEASED; urgency=low
> +${PN} (${changelog_v}) UNRELEASED; urgency=low
>
> * generated by Isar
>
> @@ -26,7 +35,7 @@ ${PN} (${CHANGELOG_V}) UNRELEASED; urgency=low
> EOF
> if [ -f ${WORKDIR}/changelog ]; then
> if head -1 "${WORKDIR}"/changelog | \
> - grep -q -e "^${PN} (${CHANGELOG_V})"
> + grep -q -e "^${PN} (${changelog_v})"
Might want to switch to dpkg-parsechangelog while at it.
Henning
> then
> # entry for our version already there, use
> unmodified rm ${S}/debian/changelog
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V
2020-04-23 16:31 [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V Jan Kiszka
2020-04-23 16:43 ` Henning Schild
@ 2020-04-23 16:47 ` Henning Schild
2020-05-07 19:45 ` Baurzhan Ismagulov
2 siblings, 0 replies; 13+ messages in thread
From: Henning Schild @ 2020-04-23 16:47 UTC (permalink / raw)
To: Jan Kiszka; +Cc: isar-users
This looks nice and clean, consider putting this into the hello rebuild
example.
Henning
On Thu, 23 Apr 2020 18:31:10 +0200
Jan Kiszka <jan.kiszka@siemens.com> wrote:
> From: Jan Kiszka <jan.kiszka@siemens.com>
>
> This allows to append to the latest upstream version when using an
> unpinned source version. The format of the placeholder is
> "<orig-version>".
>
> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
> ---
>
> Changes in v3:
> - fix regression when placeholder was not used
>
> meta/classes/debianize.bbclass | 13 +++++++++++--
> 1 file changed, 11 insertions(+), 2 deletions(-)
>
> diff --git a/meta/classes/debianize.bbclass
> b/meta/classes/debianize.bbclass index da43a4e2..e77be232 100644
> --- a/meta/classes/debianize.bbclass
> +++ b/meta/classes/debianize.bbclass
> @@ -11,6 +11,15 @@ DESCRIPTION ?= "must not be empty"
> MAINTAINER ?= "Unknown maintainer <unknown@example.com>"
>
> deb_add_changelog() {
> + changelog_v="${CHANGELOG_V}"
> + if [ -f ${S}/debian/changelog ]; then
> + if [ ! -f ${WORKDIR}/changelog.orig ]; then
> + cp ${S}/debian/changelog
> ${WORKDIR}/changelog.orig
> + fi
> + orig_version=$(dpkg-parsechangelog -l
> ${WORKDIR}/changelog.orig -S Version)
> + changelog_v=$(echo "${changelog_v}" | sed
> 's/<orig-version>/'${orig_version}'/')
> + fi
> +
> timestamp=$(find ${S}/ -type f -not -path "${S}/debian/*"
> -printf "%T@\n"|sort -n -r|head -n 1) if [ -n "${timestamp}" ]; then
> date=$(LANG=C date -R -d @${timestamp})
> @@ -18,7 +27,7 @@ deb_add_changelog() {
> date=$(LANG=C date -R)
> fi
> cat <<EOF > ${S}/debian/changelog
> -${PN} (${CHANGELOG_V}) UNRELEASED; urgency=low
> +${PN} (${changelog_v}) UNRELEASED; urgency=low
>
> * generated by Isar
>
> @@ -26,7 +35,7 @@ ${PN} (${CHANGELOG_V}) UNRELEASED; urgency=low
> EOF
> if [ -f ${WORKDIR}/changelog ]; then
> if head -1 "${WORKDIR}"/changelog | \
> - grep -q -e "^${PN} (${CHANGELOG_V})"
> + grep -q -e "^${PN} (${changelog_v})"
> then
> # entry for our version already there, use
> unmodified rm ${S}/debian/changelog
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V
2020-04-23 16:43 ` Henning Schild
@ 2020-04-23 16:50 ` Jan Kiszka
2020-04-23 17:32 ` Henning Schild
0 siblings, 1 reply; 13+ messages in thread
From: Jan Kiszka @ 2020-04-23 16:50 UTC (permalink / raw)
To: Henning Schild; +Cc: isar-users
On 23.04.20 18:43, Henning Schild wrote:
> On Thu, 23 Apr 2020 18:31:10 +0200
> Jan Kiszka <jan.kiszka@siemens.com> wrote:
>
>> From: Jan Kiszka <jan.kiszka@siemens.com>
>>
>> This allows to append to the latest upstream version when using an
>> unpinned source version. The format of the placeholder is
>> "<orig-version>".
>>
>> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
>> ---
>>
>> Changes in v3:
>> - fix regression when placeholder was not used
>>
>> meta/classes/debianize.bbclass | 13 +++++++++++--
>> 1 file changed, 11 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/classes/debianize.bbclass
>> b/meta/classes/debianize.bbclass index da43a4e2..e77be232 100644
>> --- a/meta/classes/debianize.bbclass
>> +++ b/meta/classes/debianize.bbclass
>> @@ -11,6 +11,15 @@ DESCRIPTION ?= "must not be empty"
>> MAINTAINER ?= "Unknown maintainer <unknown@example.com>"
>>
>> deb_add_changelog() {
>> + changelog_v="${CHANGELOG_V}"
>> + if [ -f ${S}/debian/changelog ]; then
>> + if [ ! -f ${WORKDIR}/changelog.orig ]; then
>> + cp ${S}/debian/changelog
>> ${WORKDIR}/changelog.orig
>> + fi
>> + orig_version=$(dpkg-parsechangelog -l
>> ${WORKDIR}/changelog.orig -S Version)
>> + changelog_v=$(echo "${changelog_v}" | sed
>> 's/<orig-version>/'${orig_version}'/')
>> + fi
>> +
>> timestamp=$(find ${S}/ -type f -not -path "${S}/debian/*"
>> -printf "%T@\n"|sort -n -r|head -n 1) if [ -n "${timestamp}" ]; then
>> date=$(LANG=C date -R -d @${timestamp})
>> @@ -18,7 +27,7 @@ deb_add_changelog() {
>> date=$(LANG=C date -R)
>> fi
>> cat <<EOF > ${S}/debian/changelog
>> -${PN} (${CHANGELOG_V}) UNRELEASED; urgency=low
>> +${PN} (${changelog_v}) UNRELEASED; urgency=low
>>
>> * generated by Isar
>>
>> @@ -26,7 +35,7 @@ ${PN} (${CHANGELOG_V}) UNRELEASED; urgency=low
>> EOF
>> if [ -f ${WORKDIR}/changelog ]; then
>> if head -1 "${WORKDIR}"/changelog | \
>> - grep -q -e "^${PN} (${CHANGELOG_V})"
>> + grep -q -e "^${PN} (${changelog_v})"
>
> Might want to switch to dpkg-parsechangelog while at it.
>
Should that still match both PN and the version? Would actually become
more complex, though conceptually cleaner.
Jan
--
Siemens AG, Corporate Technology, CT RDA IOT SES-DE
Corporate Competence Center Embedded Linux
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V
2020-04-23 16:50 ` Jan Kiszka
@ 2020-04-23 17:32 ` Henning Schild
2020-04-24 7:59 ` Jan Kiszka
0 siblings, 1 reply; 13+ messages in thread
From: Henning Schild @ 2020-04-23 17:32 UTC (permalink / raw)
To: Jan Kiszka; +Cc: isar-users
On Thu, 23 Apr 2020 18:50:54 +0200
Jan Kiszka <jan.kiszka@siemens.com> wrote:
> On 23.04.20 18:43, Henning Schild wrote:
> > On Thu, 23 Apr 2020 18:31:10 +0200
> > Jan Kiszka <jan.kiszka@siemens.com> wrote:
> >
> >> From: Jan Kiszka <jan.kiszka@siemens.com>
> >>
> >> This allows to append to the latest upstream version when using an
> >> unpinned source version. The format of the placeholder is
> >> "<orig-version>".
> >>
> >> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
> >> ---
> >>
> >> Changes in v3:
> >> - fix regression when placeholder was not used
> >>
> >> meta/classes/debianize.bbclass | 13 +++++++++++--
> >> 1 file changed, 11 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/meta/classes/debianize.bbclass
> >> b/meta/classes/debianize.bbclass index da43a4e2..e77be232 100644
> >> --- a/meta/classes/debianize.bbclass
> >> +++ b/meta/classes/debianize.bbclass
> >> @@ -11,6 +11,15 @@ DESCRIPTION ?= "must not be empty"
> >> MAINTAINER ?= "Unknown maintainer <unknown@example.com>"
> >>
> >> deb_add_changelog() {
> >> + changelog_v="${CHANGELOG_V}"
> >> + if [ -f ${S}/debian/changelog ]; then
> >> + if [ ! -f ${WORKDIR}/changelog.orig ]; then
> >> + cp ${S}/debian/changelog
> >> ${WORKDIR}/changelog.orig
> >> + fi
> >> + orig_version=$(dpkg-parsechangelog -l
> >> ${WORKDIR}/changelog.orig -S Version)
> >> + changelog_v=$(echo "${changelog_v}" | sed
> >> 's/<orig-version>/'${orig_version}'/')
> >> + fi
> >> +
> >> timestamp=$(find ${S}/ -type f -not -path "${S}/debian/*"
> >> -printf "%T@\n"|sort -n -r|head -n 1) if [ -n "${timestamp}" ];
> >> then date=$(LANG=C date -R -d @${timestamp})
> >> @@ -18,7 +27,7 @@ deb_add_changelog() {
> >> date=$(LANG=C date -R)
> >> fi
> >> cat <<EOF > ${S}/debian/changelog
> >> -${PN} (${CHANGELOG_V}) UNRELEASED; urgency=low
> >> +${PN} (${changelog_v}) UNRELEASED; urgency=low
> >>
> >> * generated by Isar
> >>
> >> @@ -26,7 +35,7 @@ ${PN} (${CHANGELOG_V}) UNRELEASED; urgency=low
> >> EOF
> >> if [ -f ${WORKDIR}/changelog ]; then
> >> if head -1 "${WORKDIR}"/changelog | \
> >> - grep -q -e "^${PN} (${CHANGELOG_V})"
> >> + grep -q -e "^${PN} (${changelog_v})"
> >
> > Might want to switch to dpkg-parsechangelog while at it.
> >
>
> Should that still match both PN and the version? Would actually
> become more complex, though conceptually cleaner.
Good catch, i think the PN was just in there because i decided to
"parse" myself. Prepending a changelog entry with another PN would be
pretty broken.
What i cared about here was "only" the case where someone already
created an entry with that PV, maybe because they wanted a nicer
"commit message", another author, prio, date ...
If we wanted to double-check, other places would need to do that as
well. I would drop it and wait for the first "fool" to change PN during
a rebuild.
Henning
> Jan
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V
2020-04-23 17:32 ` Henning Schild
@ 2020-04-24 7:59 ` Jan Kiszka
0 siblings, 0 replies; 13+ messages in thread
From: Jan Kiszka @ 2020-04-24 7:59 UTC (permalink / raw)
To: Henning Schild; +Cc: isar-users
On 23.04.20 19:32, Henning Schild wrote:
> On Thu, 23 Apr 2020 18:50:54 +0200
> Jan Kiszka <jan.kiszka@siemens.com> wrote:
>
>> On 23.04.20 18:43, Henning Schild wrote:
>>> On Thu, 23 Apr 2020 18:31:10 +0200
>>> Jan Kiszka <jan.kiszka@siemens.com> wrote:
>>>
>>>> From: Jan Kiszka <jan.kiszka@siemens.com>
>>>>
>>>> This allows to append to the latest upstream version when using an
>>>> unpinned source version. The format of the placeholder is
>>>> "<orig-version>".
>>>>
>>>> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
>>>> ---
>>>>
>>>> Changes in v3:
>>>> - fix regression when placeholder was not used
>>>>
>>>> meta/classes/debianize.bbclass | 13 +++++++++++--
>>>> 1 file changed, 11 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/meta/classes/debianize.bbclass
>>>> b/meta/classes/debianize.bbclass index da43a4e2..e77be232 100644
>>>> --- a/meta/classes/debianize.bbclass
>>>> +++ b/meta/classes/debianize.bbclass
>>>> @@ -11,6 +11,15 @@ DESCRIPTION ?= "must not be empty"
>>>> MAINTAINER ?= "Unknown maintainer <unknown@example.com>"
>>>>
>>>> deb_add_changelog() {
>>>> + changelog_v="${CHANGELOG_V}"
>>>> + if [ -f ${S}/debian/changelog ]; then
>>>> + if [ ! -f ${WORKDIR}/changelog.orig ]; then
>>>> + cp ${S}/debian/changelog
>>>> ${WORKDIR}/changelog.orig
>>>> + fi
>>>> + orig_version=$(dpkg-parsechangelog -l
>>>> ${WORKDIR}/changelog.orig -S Version)
>>>> + changelog_v=$(echo "${changelog_v}" | sed
>>>> 's/<orig-version>/'${orig_version}'/')
>>>> + fi
>>>> +
>>>> timestamp=$(find ${S}/ -type f -not -path "${S}/debian/*"
>>>> -printf "%T@\n"|sort -n -r|head -n 1) if [ -n "${timestamp}" ];
>>>> then date=$(LANG=C date -R -d @${timestamp})
>>>> @@ -18,7 +27,7 @@ deb_add_changelog() {
>>>> date=$(LANG=C date -R)
>>>> fi
>>>> cat <<EOF > ${S}/debian/changelog
>>>> -${PN} (${CHANGELOG_V}) UNRELEASED; urgency=low
>>>> +${PN} (${changelog_v}) UNRELEASED; urgency=low
>>>>
>>>> * generated by Isar
>>>>
>>>> @@ -26,7 +35,7 @@ ${PN} (${CHANGELOG_V}) UNRELEASED; urgency=low
>>>> EOF
>>>> if [ -f ${WORKDIR}/changelog ]; then
>>>> if head -1 "${WORKDIR}"/changelog | \
>>>> - grep -q -e "^${PN} (${CHANGELOG_V})"
>>>> + grep -q -e "^${PN} (${changelog_v})"
>>>
>>> Might want to switch to dpkg-parsechangelog while at it.
>>>
>>
>> Should that still match both PN and the version? Would actually
>> become more complex, though conceptually cleaner.
>
> Good catch, i think the PN was just in there because i decided to
> "parse" myself. Prepending a changelog entry with another PN would be
> pretty broken.
>
> What i cared about here was "only" the case where someone already
> created an entry with that PV, maybe because they wanted a nicer
> "commit message", another author, prio, date ...
>
> If we wanted to double-check, other places would need to do that as
> well. I would drop it and wait for the first "fool" to change PN during
> a rebuild.
>
OK. I will write a cleanup patch for that aspect on top of this one.
Jan
--
Siemens AG, Corporate Technology, CT RDA IOT SES-DE
Corporate Competence Center Embedded Linux
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V
2020-04-23 16:31 [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V Jan Kiszka
2020-04-23 16:43 ` Henning Schild
2020-04-23 16:47 ` Henning Schild
@ 2020-05-07 19:45 ` Baurzhan Ismagulov
2020-05-07 20:02 ` Henning Schild
2 siblings, 1 reply; 13+ messages in thread
From: Baurzhan Ismagulov @ 2020-05-07 19:45 UTC (permalink / raw)
To: isar-users
On Thu, Apr 23, 2020 at 06:31:10PM +0200, Jan Kiszka wrote:
> This allows to append to the latest upstream version when using an
> unpinned source version. The format of the placeholder is
> "<orig-version>".
Applied to next, thanks.
With kind regards,
Baurzhan.
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V
2020-05-07 19:45 ` Baurzhan Ismagulov
@ 2020-05-07 20:02 ` Henning Schild
2020-05-07 20:28 ` Baurzhan Ismagulov
2020-05-08 6:02 ` Jan Kiszka
0 siblings, 2 replies; 13+ messages in thread
From: Henning Schild @ 2020-05-07 20:02 UTC (permalink / raw)
To: Baurzhan Ismagulov; +Cc: isar-users
Am Thu, 7 May 2020 21:45:02 +0200
schrieb Baurzhan Ismagulov <ibr@radix50.net>:
> On Thu, Apr 23, 2020 at 06:31:10PM +0200, Jan Kiszka wrote:
> > This allows to append to the latest upstream version when using an
> > unpinned source version. The format of the placeholder is
> > "<orig-version>".
>
> Applied to next, thanks.
The discussion was not finished, IMHO.
Jan went for another patch on top "meta-isar: hello: Use <orig-version>
in CHANGELOG_V" which raised more questions on this series.
The patch on top should have been included in a v4, which would now be
stuck.
Not sure how to deal with the premature merge now.
Henning
> With kind regards,
> Baurzhan.
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V
2020-05-07 20:02 ` Henning Schild
@ 2020-05-07 20:28 ` Baurzhan Ismagulov
2020-05-08 5:45 ` Henning Schild
2020-05-08 6:02 ` Jan Kiszka
1 sibling, 1 reply; 13+ messages in thread
From: Baurzhan Ismagulov @ 2020-05-07 20:28 UTC (permalink / raw)
To: isar-users
On Thu, May 07, 2020 at 10:02:15PM +0200, Henning Schild wrote:
> The discussion was not finished, IMHO.
>
> Jan went for another patch on top "meta-isar: hello: Use <orig-version>
> in CHANGELOG_V" which raised more questions on this series.
>
> The patch on top should have been included in a v4, which would now be
> stuck.
Oops, I haven't seen the intention to do that, have I overlooked anything? I've
stated the reason in the next mail -- sorry for one-side decision. If there is
consensus on a working approach, I'd of course welcome that.
> Not sure how to deal with the premature merge now.
The final solution can be applied on top, or I can revert the patches if that
would help.
With kind regards,
Baurzhan.
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V
2020-05-07 20:28 ` Baurzhan Ismagulov
@ 2020-05-08 5:45 ` Henning Schild
2020-05-08 6:02 ` Baurzhan Ismagulov
0 siblings, 1 reply; 13+ messages in thread
From: Henning Schild @ 2020-05-08 5:45 UTC (permalink / raw)
To: Baurzhan Ismagulov; +Cc: isar-users
Am Thu, 7 May 2020 22:28:28 +0200
schrieb Baurzhan Ismagulov <ibr@radix50.net>:
> On Thu, May 07, 2020 at 10:02:15PM +0200, Henning Schild wrote:
> > The discussion was not finished, IMHO.
> >
> > Jan went for another patch on top "meta-isar: hello: Use
> > <orig-version> in CHANGELOG_V" which raised more questions on this
> > series.
> >
> > The patch on top should have been included in a v4, which would now
> > be stuck.
>
> Oops, I haven't seen the intention to do that, have I overlooked
> anything? I've stated the reason in the next mail -- sorry for
> one-side decision. If there is consensus on a working approach, I'd
> of course welcome that.
It was hard to see. The follow up kind of questions this series and the
discussion is still ongoing.
> > Not sure how to deal with the premature merge now.
>
> The final solution can be applied on top, or I can revert the patches
> if that would help.
The feature will potentially vanish or work differently. That should be
clear for users. Since isar master is slow many users are on next. Next
does not seem to ever rebase.
So i say we need to finish the discussion eventually. Until then
potentially revert the commits to prevent users jumping on a feature
that will be harder to repair once its out there.
Let us see what Jan has to say. Maybe the discussion only seems open
because the points i raised are not valid in his opinion.
Henning
> With kind regards,
> Baurzhan.
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V
2020-05-07 20:02 ` Henning Schild
2020-05-07 20:28 ` Baurzhan Ismagulov
@ 2020-05-08 6:02 ` Jan Kiszka
2020-05-08 6:06 ` Baurzhan Ismagulov
1 sibling, 1 reply; 13+ messages in thread
From: Jan Kiszka @ 2020-05-08 6:02 UTC (permalink / raw)
To: [ext] Henning Schild, Baurzhan Ismagulov; +Cc: isar-users
On 07.05.20 22:02, [ext] Henning Schild wrote:
> Am Thu, 7 May 2020 21:45:02 +0200
> schrieb Baurzhan Ismagulov <ibr@radix50.net>:
>
>> On Thu, Apr 23, 2020 at 06:31:10PM +0200, Jan Kiszka wrote:
>>> This allows to append to the latest upstream version when using an
>>> unpinned source version. The format of the placeholder is
>>> "<orig-version>".
>>
>> Applied to next, thanks.
>
> The discussion was not finished, IMHO.
>
Regarding this patch, we are through IMHO. It makes sense to track at
least stable updates of debian packages, and it will be used for that in
production.
> Jan went for another patch on top "meta-isar: hello: Use <orig-version>
> in CHANGELOG_V" which raised more questions on this series.
>
> The patch on top should have been included in a v4, which would now be
> stuck.
Nope, nothing like that planned.
>
> Not sure how to deal with the premature merge now.
>
Keep it.
Jan
--
Siemens AG, Corporate Technology, CT RDA IOT SES-DE
Corporate Competence Center Embedded Linux
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V
2020-05-08 5:45 ` Henning Schild
@ 2020-05-08 6:02 ` Baurzhan Ismagulov
0 siblings, 0 replies; 13+ messages in thread
From: Baurzhan Ismagulov @ 2020-05-08 6:02 UTC (permalink / raw)
To: isar-users
On Fri, May 08, 2020 at 07:45:05AM +0200, Henning Schild wrote:
> > Oops, I haven't seen the intention to do that, have I overlooked
> > anything? I've stated the reason in the next mail -- sorry for
> > one-side decision. If there is consensus on a working approach, I'd
> > of course welcome that.
>
> It was hard to see. The follow up kind of questions this series and the
> discussion is still ongoing.
>
> > > Not sure how to deal with the premature merge now.
> >
> > The final solution can be applied on top, or I can revert the patches
> > if that would help.
>
> The feature will potentially vanish or work differently. That should be
> clear for users. Since isar master is slow many users are on next. Next
> does not seem to ever rebase.
> So i say we need to finish the discussion eventually. Until then
> potentially revert the commits to prevent users jumping on a feature
> that will be harder to repair once its out there.
>
> Let us see what Jan has to say. Maybe the discussion only seems open
> because the points i raised are not valid in his opinion.
I see, thanks for the explanations. IMHO, your points are perfectly valid and
addressing them would be a good structural improvement. OTOH, Jan's use case
doesn't go away, and his change works as is. But I agree we should finish the
discussion.
With kind regards,
Baurzhan.
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V
2020-05-08 6:02 ` Jan Kiszka
@ 2020-05-08 6:06 ` Baurzhan Ismagulov
0 siblings, 0 replies; 13+ messages in thread
From: Baurzhan Ismagulov @ 2020-05-08 6:06 UTC (permalink / raw)
To: isar-users
On Fri, May 08, 2020 at 08:02:05AM +0200, Jan Kiszka wrote:
> Keep it.
Makes sense for me.
With kind regards,
Baurzhan.
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2020-05-08 6:06 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-23 16:31 [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V Jan Kiszka
2020-04-23 16:43 ` Henning Schild
2020-04-23 16:50 ` Jan Kiszka
2020-04-23 17:32 ` Henning Schild
2020-04-24 7:59 ` Jan Kiszka
2020-04-23 16:47 ` Henning Schild
2020-05-07 19:45 ` Baurzhan Ismagulov
2020-05-07 20:02 ` Henning Schild
2020-05-07 20:28 ` Baurzhan Ismagulov
2020-05-08 5:45 ` Henning Schild
2020-05-08 6:02 ` Baurzhan Ismagulov
2020-05-08 6:02 ` Jan Kiszka
2020-05-08 6:06 ` Baurzhan Ismagulov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox