Quoting James Valleroy (2017-01-23 23:48:00) > On Tue, 13 Dec 2016 12:14:52 -0500 James Valleroy > <jvalle...@mailbox.org> wrote: >> The attached patch set implements this, by adding several debconf >> questions to set the server hostnames, base url, well-known paths, >> authentication type, and rights type. These questions all have low >> priority. I used augtool (from augeas-tools) to read/modify the >> config file, and ucf to merge user settings with updates to the >> packaged file. > > After sending this patch, I discovered this rule given in the Debconf > Programmer's Tutorial: > "Note that the config script is run before the package is unpacked. It > should only use commands that are in essential packages." > > But in the patch I submitted (part 0002), I used augtool in > debian/radicale.config to parse the ini-file config and read current > settings from disk. So that is a problem for the 0002 and 0003 patches > above.
Ah, that might explain why augeas hasn't been in more widespread use. > Patch 0001, which only adds ucf to improve merging changes to the > config, should still be ok. Ok, but patch 0001 alone does not really help address the requirement of Debian Policy ยง 10.7.4, further clarified in release notes like the most recent https://release.debian.org/stretch/rc_policy.txt: > Packages must not modify other packages' configuration files except by > an agreed upon APIs (eg, a /usr/sbin/update-foo command). For an old and long read about this issue, see bug#311188. - Jonas -- * Jonas Smedegaard - idealist & Internet-arkitekt * Tlf.: +45 40843136 Website: http://dr.jones.dk/ [x] quote me freely [ ] ask before reusing [ ] keep private
signature.asc
Description: signature
