On 12/09/16 20:51, Gianfranco Costamagna wrote:

Gianfranco, as per Policy ยง4.4, please put your own name in the
changelog trailer when making an upload.

well, it seems unfair wrt Peter,
IMO what the changelog should look like in a case like this is

imapcopy (1.04-2.1) unstable; urgency=medium
  * Non-maintainer upload
  [Peter Michael Green]
  * Fix build with fpc 3.0.0 (Closes: 818448)

 -- Gianfranco Costamagna<locutusofb...@debian.org>   <date of upload>

This way it correctly identifies both who authored the change and who decided 
it was ready to upload to Unstable.

You should only upload with someone else's name in the chagelog trailer if they 
have explicitly asked for sponsorship.

who actually did *all* the work
<--snip-->
BTW on top of the changes, me and a friend of mine, tested the basic
functions of the imapcopy tool.
(not sure if the testing was complete or not, since it was just a FTBFS
fix and it was somewhere trivial).
the fix was referring network operations, it might break somewhere in some 
particular
networks, but I think this isn't the case.
Maybe i'm overcautious but I will only NMU if I have tested at least the basic functioning of the application and ideally have tested it in a way that I know will hit the codepath I changed.

Figuring out how to test a given application, preparing a suitable test environment etc is often more work than writing the actual patch. That is why in this case I merely subimitted a patch to the bug report and did not upload as a NMU myself.

IMO your actions testing that the fix was indeed ready for upload are as valuable as my figuring out what needed to be changed to make the package build.

Reply via email to