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 ? -- أحمد المحمودي (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
