update-maintainer doesn't work for fakesyncs

Bug #376623 reported by Bryce Harrington on 2009-05-14
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-dev-tools (Ubuntu)
Low
Unassigned

Bug Description

Binary package hint: ubuntu-dev-tools

If you're doing a fakesync, the version should be something like '1.2.3-1build1'. update-maintainer doesn't recognize this as a valid Ubuntu version number and so fails to run.

Something like this is needed in the script:

if ! head -1 $DEBIANDIR/changelog | egrep -q '(*ubuntu.*|*build.*)'
then
        echo "Latest changelog entry has no Ubuntu version number." >&2
        exit 1
fi

Changed in ubuntu-dev-tools (Ubuntu):
assignee: nobody → Nathan Handler (nhandler)
importance: Undecided → Low
status: New → In Progress
Nathan Handler (nhandler) wrote :

Fix commited to trunk in revision 397. This will be fixed in ubuntu-dev-tools version 0.74

Changed in ubuntu-dev-tools (Ubuntu):
assignee: Nathan Handler (nhandler) → nobody
status: In Progress → Fix Committed
Michael Bienia (geser) wrote :

Does the maintainer need to be updated for -XbuildY?
As far as I know we only need to update the maintainer if we did changes to the package, so it's correct that update-maintainer only updates the maintainer field for -XubuntuY.
As a fakesync is just a different .orig.tar.gz (different md5sum) but with no ubuntu changes, we don't need to update the maintainer field.
After this change update-maintainer will also change the maintainer field for mere rebuilds. Which is not a problem in itself, but also not really necessary.

Nathan Handler (nhandler) wrote :

Michael, you are correct. I just looked at [1], which confirms that rebuilds like this should not have the maintainer field updated. I have taken care of reverting revision 397, which had modified update-maintainer.

[1] https://wiki.ubuntu.com/UbuntuDevelopment/NBS

Changed in ubuntu-dev-tools (Ubuntu):
status: Fix Committed → Won't Fix
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers