2006-08-10 13:37:51 |
Christian Reis |
description |
On Wed, 2006-08-09 at 10:36 +0200, Gauvain Pocentek wrote:
Bart Martens a écrit :
> > Hi Gauvain,
> >
> > Can you explain why my name was replaced by your name in the changelog?
> > Did this happen by mistake?
> >
> > https://launchpad.net/distros/ubuntu/edgy/+source/discover/+changelog
> > http://packages.debian.org/changelogs/pool/main/d/discover/current/changelog
> >
> > Regards,
> >
> > Bart Martens
> >
> >
>
> Hi again Bart,
>
> I've just downloaded the sources to fix the discover changelog, but your
> name appears in the source package. Looking at the .diff.gz on launchpad
> [1], everything seems ok. It's certainly a Launchpad bug in the
> changelog display. You can report the bug at [2] (I can do it if it's
> easier for you).
>
> [1] http://librarian.launchpad.net/3782208/discover_2.0.7-3ubuntu1.diff.gz
> [2] https://launchpad.net/products/launchpad/+filebug
>
> HTH,
> Gauvain
>
>
The changelog currently displayed in launchpad:
discover (2.0.7-3ubuntu1) edgy; urgency=low
* Merge from debian unstable:
- discover.dtd: Change device->model default to 'default'
- Miscellaneous changes to allow default entries for vendors
discover (2.0.7-3) unstable; urgency=low
* QA upload.
* NMU ACK. Closes: #305042. Thanks to Jeroen and David, see below.
* sysdeps/stub/stubs.c: Fix FTBFS on GNU/kFreeBSD (and any system using
sysdep/stubs/). Closes: #364863. Patch by Robert Millan
<rmh@aybabtu.com>, thanks.
* doc/discover-modprobe.8, debian/discover.postrm: Removed "/usr/local" from
paths in man page, and added removal of /var/lib/discover in postrm.
Closes: #328497. Thanks to Lars Wirzenius <liw@iki.fi>.
* debian/discover.postinst: Enter invoke-rc.d. Closes: #367739.
* doc/device-types.xml, doc/discover.1, doc/guide.html, doc/guide.txt: Fixed
typo. Closes: #311442.
* debian/rules: Fixed debian-rules-missing-required-target binary-indep.
* debian/changelog: Fixed syntax-error-in-debian-changelog.
-- Gauvain Pocentek <gauvainpocentek@ubuntu.com> Mon, 31 Jul 2006 01:44:51 +0100
discover (2.0.7-2.1ubuntu1) dapper; urgency=low
* Resynchronise with Debian.
-- Martin Meredith <mez@ubuntu.com> Wed, 2 Nov 2005 21:10:47 +0000
But, as Gauvain correctly noted, the real changelog is:
discover (2.0.7-3ubuntu1) edgy; urgency=low
* Merge from debian unstable:
- discover.dtd: Change device->model default to 'default'
- Miscellaneous changes to allow default entries for vendors
discover (2.0.7-3) unstable; urgency=low
* QA upload.
* NMU ACK. Closes: #305042. Thanks to Jeroen and David, see below.
* sysdeps/stub/stubs.c: Fix FTBFS on GNU/kFreeBSD (and any system using
sysdep/stubs/). Closes: #364863. Patch by Robert Millan
<rmh@aybabtu.com>, thanks.
* doc/discover-modprobe.8, debian/discover.postrm: Removed "/usr/local" from
paths in man page, and added removal of /var/lib/discover in postrm.
Closes: #328497. Thanks to Lars Wirzenius <liw@iki.fi>.
* debian/discover.postinst: Enter invoke-rc.d. Closes: #367739.
* doc/device-types.xml, doc/discover.1, doc/guide.html, doc/guide.txt: Fixed
typo. Closes: #311442.
* debian/rules: Fixed debian-rules-missing-required-target binary-indep.
* debian/changelog: Fixed syntax-error-in-debian-changelog.
-- Gauvain Pocentek <gauvainpocentek@ubuntu.com> Mon, 31 Jul 2006 01:44:51 +0100
discover (2.0.7-2.1ubuntu1) dapper; urgency=low
* Resynchronise with Debian.
-- Martin Meredith <mez@ubuntu.com> Wed, 2 Nov 2005 21:10:47 +0000
So my request is to display the debian/changelog in launchpad without replacing the Debian maintainer by the Ubuntu maintainer.
Thanks,
Bart Martens
bartm@knars.be
|
The changelog at
https://launchpad.net/distros/ubuntu/edgy/+source/discover/+changelog
disagrees with the changelog at:
http://packages.debian.org/changelogs/pool/main/d/discover/current/changelog
crediting Gauvain for a set of changes that Bart did. The reason this happens is because:
a) This package discover=2.0.7-3 was manually synced into Ubuntu as discover=2.0.7-3ubuntu1
b) The changes file includes information for both versions. The changes file doesn't credit the original author, mainly because the format is hard to change, according to James.
c) We use the information from the changes file to generate the information displayed on +changelog.
The stated Ubuntu policy is to include all modification changelogs in the changes file, which means that we will need to find another way of generating the +changelog data to avoid the problem with missing/wrong credit.
It is likely that the best way to do this is to store the debian/changelog file in the librarian, and to use that when displaying the +changelog.
We could as a first step just link to the librarian file. For bonus points, we could read() the librarian content and then linkify it. Note that linkification should only be done for packages which we know about -- we could do that by grabbing all valid package versions and then only linking versions we knew about.
|
|