Hi Michael, On Wed, Jan 11, 2017 at 11:34:52PM +0100, Michael Stapelberg wrote: > Attached you can find a first stab at implementing this feature. I > introduced a new protocol message to transfer base64-encoded arbitrary > binary data.
the patch looks very nice. thanks for adding all the documentation already. > This can easily be used to transfer other files in the > same spirit, should that become necessary in the future (it also seems > like the clean thing to do, even if we’re just talking about a single > file). nice! > The files are then made available at > /<section>/aux/<package>_<version>/<filename>, e.g. > /sid/aux/libva1_1.7.3-2/alternatives.tar.gz. fine in principle, except this is part of the three concerns I have about the patch: - "aux" is maybe a bit too short (as a directory name), maybe not :) - the option --auxdir is good, but in the current patches this is not seperated from --record-alternatives-in-auxdir or whatever that option to enable alternatives collection should be called. Not sure about the name for that option but you'll get the idea… - please also add a debian/changelog entry… -- cheers, Holger
signature.asc
Description: Digital signature