[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#820780: clarify syntax of ‘cancel’ command for queue control



On 26/12/16 21:36, Ben Finney wrote:
> Is this constraint – the argument to ‘cancel’ *must* be a base
> filename only – imposed by the upload queue processor? If so, the
> response:
> 
>> .commands file has invalid Commands line: cancel ../pytest-django_2.9.1-2.1_amd64.changes
>> debsign: .commands file appears to be invalid. see:
>> ftp://ftp.upload.debian.org/pub/UploadQueue/README
>> for valid format
> 
> is not very helpful, because the referenced document does not make
> that constraint plain.
> 
> 
> I'm casting this to ‘debian-devel’ for confirmation whether this is
> actually the problem. Can someone with knowledge of the upload queue
> processing clarify this?

On the archive side, arguments to 'cancel' must match this regex:
https://anonscm.debian.org/cgit/mirror/dak.git/tree/tools/debianqueued-0.9/debianqueued#n71

which is used here:
https://anonscm.debian.org/cgit/mirror/dak.git/tree/tools/debianqueued-0.9/debianqueued#n1294

Forward slash is not in the regex, so paths are disallowed.

James


Attachment: signature.asc
Description: OpenPGP digital signature


Reply to: