[PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V

[PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V

[Jan Kiszka]

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

Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V

[Henning Schild]

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

Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V

[Jan Kiszka]

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

Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V

[Henning Schild]

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
> 

Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V

[Jan Kiszka]

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

Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V

[Henning Schild]

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

Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V

[Baurzhan Ismagulov]

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.

Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V

[Henning Schild]

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.
> 

Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V

[Baurzhan Ismagulov]

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.

Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V

[Henning Schild]

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.
> 

Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V

[Baurzhan Ismagulov]

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.

Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V

[Jan Kiszka]

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

Re: [PATCH v3] debianize: Add placeholder for original version to CHANGELOG_V

[Baurzhan Ismagulov]

On Fri, May 08, 2020 at 08:02:05AM +0200, Jan Kiszka wrote:
> Keep it.

Makes sense for me.

With kind regards,
Baurzhan.