Re: DEP1: Clarifying policies and workflows for Non Maintainer Uploads (NMUs)
  Home FAQ Contact Sign in
linux.debian.project only
 
Advanced search
POPULAR GROUPS

more...

linux.debian.project Profile…
 Up
Re: DEP1: Clarifying policies and workflows for Non Maintainer Uploads (NMUs)         


Author: Charles Plessy
Date: May 25, 2008 17:20

Hi Bas, and Lucas,

Le Sun, May 25, 2008 at 08:50:45AM +0200, Bas Wijnen a écrit :
>
> In some cases, the maintainer might allow direct commit to the package's
> VCS repository. We felt that it was not a good idea to include this in
> the DEP, because:

Actually, this leaves open the question whether the diff of the NMU
should be done against the current package in the Debian archive or the
work in progress in the maintainers VCS repository). One is clearer to
the users, and one is more useful for the maintainers. Which one do you
recommend?

Have a nice day,

--
Charles Plessy,
Wako, Saitama, Japan

--
To UNSUBSCRIBE, email to debian-project-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
13 Comments
Re: DEP1: Clarifying policies and workflows for Non Maintainer Uploads (NMUs)         


Author: Lucas Nussbaum
Date: May 26, 2008 00:00

On 26/05/08 at 09:17 +0900, Charles Plessy wrote:
> Hi Bas, and Lucas,
>
> Le Sun, May 25, 2008 at 08:50:45AM +0200, Bas Wijnen a écrit :
>>
>> In some cases, the maintainer might allow direct commit to the package's
>> VCS repository. We felt that it was not a good idea to include this in
>> the DEP, because:
>
> Actually, this leaves open the question whether the diff of the NMU
> should be done against the current package in the Debian archive or the
> work in progress in the maintainers VCS repository). One is clearer to
> the users, and one is more useful for the maintainers. Which one do you
> recommend?

Let 1.0-1 be the version in the archive, 1.0-2 the version being
worked on in the VCS repository.
Show full article (1.31Kb)
no comments
Re: DEP1: Clarifying policies and workflows for Non Maintainer Uploads (NMUs)         


Author: Charles Plessy
Date: May 26, 2008 01:20

Le Mon, May 26, 2008 at 10:42:22AM +0200, Lucas Nussbaum a écrit :
> Let 1.0-1 be the version in the archive, 1.0-2 the version being
> worked on in the VCS repository.
>
> An NMUer is very likely to base his NMU on 1.0-1, because he will try to
> do only the relevant changes to fix the bug he is interested in. In that
> case, the only logical choice is to use 1.0-1 as the basis for the diff.
> And it's actually more useful for the maintainer, don't you think so?

It depends on how important are the VCS and package histories for the
maintainer and Debian. In order to acknowledge the NMU, it would be
necessary to revert the current work, apply the NMU patch, merge the
reverted work and resolve the conflicts. This is why the last time one
of my package had a NMU, I just ignored it instead of acknowledging it
(the upload I made would have closed the bug as well).

Have a nice day,

--
Charles Plessy
http://charles.plessy.org
Wakō, Saitama, Japan
Show full article (1.11Kb)
no comments
Re: DEP1: Clarifying policies and workflows for Non Maintainer Uploads (NMUs)         


Author: Lars Wirzenius
Date: May 26, 2008 01:50

ma, 2008-05-26 kello 17:12 +0900, Charles Plessy kirjoitti:
> It depends on how important are the VCS and package histories for the
> maintainer and Debian. In order to acknowledge the NMU, it would be
> necessary to revert the current work, apply the NMU patch, merge the
> reverted work and resolve the conflicts. This is why the last time one
> of my package had a NMU, I just ignored it instead of acknowledging it
> (the upload I made would have closed the bug as well).

If the NMU is minimal, surely you should not have to undo all your own
changes, but just merge in the NMU.

(This is independent of whether things are done manually or with a
version control system.)

--
To UNSUBSCRIBE, email to debian-project-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
no comments
Re: DEP1: Clarifying policies and workflows for Non Maintainer Uploads (NMUs)         


Author: Mark Brown
Date: May 26, 2008 04:00

On Mon, May 26, 2008 at 05:12:23PM +0900, Charles Plessy wrote:
> It depends on how important are the VCS and package histories for the
> maintainer and Debian. In order to acknowledge the NMU, it would be
> necessary to revert the current work, apply the NMU patch, merge the
> reverted work and resolve the conflicts. This is why the last time one
> of my package had a NMU, I just ignored it instead of acknowledging it
> (the upload I made would have closed the bug as well).

As Lars said, if the NMU fix is minimal then it should be trivial to
just apply it directly to the VCS version (apart from the changelog
there should be a reasonable chance of patch being able to cope by
itself).

It's also important to consider why the changes that are in the VCS have
not been uploaded to the archive - it may be that it's simply a case of
nobody having got round to it yet but it could also be due to there
being some issues which need to be resolved.

--
To UNSUBSCRIBE, email to debian-project-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
no comments
Re: DEP1: Clarifying policies and workflows for Non Maintainer Uploads (NMUs)         


Author: Henrique de Moraes Holschuh
Date: May 26, 2008 05:30

On Mon, 26 May 2008, Mark Brown wrote:
> On Mon, May 26, 2008 at 05:12:23PM +0900, Charles Plessy wrote:
> As Lars said, if the NMU fix is minimal then it should be trivial to
> just apply it directly to the VCS version (apart from the changelog
> there should be a reasonable chance of patch being able to cope by
> itself).

When you base anything on an unreleased version, you have a lot more
potential to introduce issues. If you fix the current package with an
NMU, you have a known set of issues to deal with.

So, unless you *REALLY* know what you're doing, I'd say it is damn best
to stay the hell alway from anything in the VCS that has not been
uploaded yet in an NMU.
> It's also important to consider why the changes that are in the VCS have
> not been uploaded to the archive - it may be that it's simply a case of
> nobody having got round to it yet but it could also be due to there
> being some issues which need to be resolved.

Exactly. So, just don't do it as a *rule*, and leave the "upload from
VCS" as the exception.
Show full article (1.40Kb)
no comments
Re: DEP1: Clarifying policies and workflows for Non Maintainer Uploads (NMUs)         


Author: Cyril Brulebois
Date: May 26, 2008 08:30

On 26/05/2008, Charles Plessy wrote:
> It depends on how important are the VCS and package histories for the
> maintainer and Debian. In order to acknowledge the NMU, it would be
> necessary to revert the current work, apply the NMU patch, merge the
> reverted work and resolve the conflicts.

It looks to me like the wording of the 3rd paragraph of 5.11.2 is a bit
(too) strong: one must include the patch. It might be relaxed a bit so
that the maintainer is still allowed to implement the changes the way
s/he intends, rather than having to include the very patch sent to the
BTS.
> This is why the last time one of my package had a NMU, I just ignored
> it instead of acknowledging it (the upload I made would have closed
> the bug as well).

In which case, I'd probably dosomething like ACK'ing the NMU, adding a
“thanks to $nmuer, although the final implementation differs [details]”.

Mraw,
KiBi.

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Show full article (1.10Kb)
no comments
Re: DEP1: Clarifying policies and workflows for Non Maintainer Uploads (NMUs)         


Author: Bas Wijnen
Date: May 27, 2008 10:50

On Mon, May 26, 2008 at 11:56:12AM +0200, Cyril Brulebois wrote:
> On 26/05/2008, Charles Plessy wrote:
>> It depends on how important are the VCS and package histories for the
>> maintainer and Debian. In order to acknowledge the NMU, it would be
>> necessary to revert the current work, apply the NMU patch, merge the
>> reverted work and resolve the conflicts.
>
> It looks to me like the wording of the 3rd paragraph of 5.11.2 is a bit
> (too) strong: one must include the patch. It might be relaxed a bit so
> that the maintainer is still allowed to implement the changes the way
> s/he intends, rather than having to include the very patch sent to the
> BTS.

The proposal is to use the DELAYED queue as the default way to do an
NMU. This means in particular that the code is already finished when
the mail about the NMU is sent to the BTS. So there is no reason to
allow changes to the patch after this mail; if you need them, cancel the
NMU and upload an other one instead (sending the new patch to the BTS).
Show full article (1.93Kb)
no comments
Re: DEP1: Clarifying policies and workflows for Non Maintainer Uploads (NMUs)         


Author: Cyril Brulebois
Date: May 27, 2008 11:10

On 27/05/2008, Bas Wijnen wrote:
> The proposal is to use the DELAYED queue as the default way to do an
> NMU. This means in particular that the code is already finished when
> the mail about the NMU is sent to the BTS. So there is no reason to
> allow changes to the patch after this mail; if you need them, cancel
> the NMU and upload an other one instead (sending the new patch to the
> BTS).

I'm talking about ACK'ing a previously-uploaded-and-accepted NMU in a
future upload.

Quoting Charles: “In order to acknowledge the NMU, it would be necessary
to revert the current work, apply the NMU patch, merge the reverted work
and resolve the conflicts.”
Show full article (1.48Kb)
no comments
Re: DEP1: Clarifying policies and workflows for Non Maintainer Uploads (NMUs)         


Author: Bas Wijnen
Date: May 27, 2008 11:40

On Tue, May 27, 2008 at 08:01:27PM +0200, Cyril Brulebois wrote:
> On 27/05/2008, Bas Wijnen wrote:
>> The proposal is to use the DELAYED queue as the default way to do an
>> NMU. This means in particular that the code is already finished when
>> the mail about the NMU is sent to the BTS. So there is no reason to
>> allow changes to the patch after this mail; if you need them, cancel
>> the NMU and upload an other one instead (sending the new patch to the
>> BTS).
>
> I'm talking about ACK'ing a previously-uploaded-and-accepted NMU in a
> future upload.

Oh, sorry, I didn't look up your reference and thought you were talking
about including the patch in the BTS.
> Quoting Charles: “In order to acknowledge the NMU, it would be necessary
> to revert the current work, apply the NMU patch, merge the reverted work
> and resolve the conflicts.”
>
> I think I wrote...
Show full article (2.30Kb)
no comments
1 2