Hi Vincent,
On 16-09-15 22:13, Vincent Blut wrote:
> Le mer. 16 sept. 2015 à 19:57, Paul Gevers <elbrus@debian.org> a écrit :
>> Commit e88f1460f3e now prevents the upstream NEWS file to be installed
>> as upstream changelog. I rather have that stayed in. You probably should
>> call "dh_installchangelogs NEWS debian/NEWS" or something like that.
>
> Well, not really. In fact, neither upstream nor we install the
> upstream NEWS file by default.
If you would have tried the command from my previous e-mail, you would
have seen that this is not true. In the previous version of the package,
we install upstream NEWS file as /usr/share/doc/chrony/changelog.gz
$ debdiff chrony_1.31.1-1_amd64.changes chrony_1.31.1-2_amd64.changes
[The following lists of changes regard files as different if they have
different names, permissions or owners.]
Files in second .changes but not in first
-----------------------------------------
-rw-r--r-- root/root /usr/share/doc/chrony/NEWS.Debian.gz
Files in first .changes but not in second
-----------------------------------------
-rw-r--r-- root/root /usr/share/doc/chrony/changelog.gz
Control files: lines which differ (wdiff format)
------------------------------------------------
Installed-Size: [-725-] {+719+}
Version: [-1.31.1-1-] {+1.31.1-2+}
> But I agree that it would make sense to have it installed, so I’ll
> add it to debian/docs in a future upload (2.x serie) unless you
> absolutely want it for the release being discussed here.
As I would call this a regression, I rather have it fixed in our current
upload. And I think we made a good choice in the past to actually use
the NEWS file as changelog file.
Paul
Attachment:
signature.asc
Description: OpenPGP digital signature