On Sun, Jan 20, 2013 at 10:53:27PM +0200, أحمد المحمودي wrote: > On Sun, Jan 20, 2013 at 02:09:33PM +0100, gregor herrmann wrote: > > On Sun, 20 Jan 2013 14:02:31 +0200, أحمد المحمودي wrote: > > > > > On Sat, Jan 19, 2013 at 05:19:17PM +0100, gregor herrmann wrote: > > > > +# Replace documentation directory with symlink > > > > +docdir=/usr/share/doc/geda > > > > +target=geda-doc > > > > + > > > > +if [ -d $docdir ] && [ ! -L $docdir ]; then > > > > + if rmdir $docdir 2>/dev/null; then > > > > + ln -sf $target $docdir > > > > + fi > > > > +fi > > > ---end quoted text--- > > > > > > Are you sure about this fix ? > > > I think target should be /usr/share/doc/geda-doc rather than just > > > 'geda-doc' > > > > I admit that it looks a bit surprising, but it's like in other > > packages, and I've tested > > (a) this ln syntax manually [0] > > (b) and installing the new package in a chroot after making sure that > > /usr/share/doc/geda is a directory, and it was a symlink > > afterwards. > ---end quoted text--- > > For this to run correctly, it must be that the postinst's CWD is > /usr/share/doc, is that guaranteed ? ---end quoted text---
Oh, nevermind the noise, you are right indeed. It would work correctly, regardless of the CWD ! -- أحمد المحمودي (Ahmed El-Mahmoudy) Digital design engineer GPG KeyID: 0xEDDDA1B7 GPG Fingerprint: 8206 A196 2084 7E6D 0DF8 B176 BC19 6A94 EDDD A1B7
signature.asc
Description: Digital signature