[Piuparts-devel] Bug#850917: Bug#850917: Please export /var/lib/dpkg/alternatives content after installation

[Holger Levsen]

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 811 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/piuparts-devel/attachments/20170115/59466493/attachment-0001.sig>