On Sat, Mar 26, 2011 at 10:45:06AM +0100, Sven Eckelmann wrote:
> Josip Rodin wrote:
> > I appreciate the effort, but this link had been created by a version of the
> > package that only existed in unstable/backports, and 2.1.10+dfsg-2 is in
> > stable now, so the issue is transient - those users who already saw it
> > won't benefit from the fix, and those who didn't see it already won't see
> > it now. Right?
>
> Or we could say that people which try to upgrade from oldstable still have
> the
> problem. At least when they created a symlink similar to the current package.
> It is not only the random file (which is not created anymore), but also
> server.pem, server.key and ca.pem
OK, so when you said this I re-read the original report, but now I don't
see any logic in this...
You reported this error message:
ln: creating symbolic link `/etc/freeradius/certs/random' to `/dev/urandom':
File exists
This is the result of an action that does not exist at all in the current
postinst. You can see it in your own patch - we no longer link that location,
we substitute the setting in the text config file.
Furthermore, the patch changes ! -f into ! -e. This is needless because:
% touch /tmp/file
% ln -s /tmp/file /tmp/link
% test ! -f /tmp/file && echo true || echo false
false
% test ! -f /tmp/link && echo true || echo false
false
So, each of those tests whether certificate file/link is missing will behave
the same way. In fact, I remember testing this functionality extensively
while testing lenny -> squeeze upgrades.
So please explain to me once again what exactly are you trying to do :)
--
2. That which causes joy or happiness.
--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
