Hi,
I've been bitten again by this when upgrading from debian 12 to debian 13 our
DHCP relay server.
This time I looked a bit more in detail debian/isc-dhcp-relay.postinst does :
db_get isc-dhcp-relay/options || true
OPTIONS="${RET}"
TMPFILE=`mktemp -q /tmp/dhcp.config.XXXXXX`
sed -e "s/^[[:space:]]*OPTIONS[[:space:]]*=.*/OPTIONS=\"${OPTIONS}\"/" \
<${INITCONFFILE} >${TMPFILE}
cp ${TMPFILE} ${INITCONFFILE}
rm ${TMPFILE}
So this unconditionnally overwrite from db_get to /etc/default/isc-dhcp-relay
without telling the user.
This means that any direct edit done over the time to
/etc/default/isc-dhcp-relay will be lost and reset to the initial install
choices (if any) when upgrading or reinstalling.
I don't think debian should overwrite what the user has written to
/etc/default/isc-dhcp-relay, I'm not familiar with debian policy but usually
when a config file is changed by the upgrade process debian is carefull to warn
the user and propose to keep the changes or install the packager version.
Thanks,
Laurent
