[Pkg-electronics-devel] Bug#694015: Bug#694015: geda-gaf: diff for NMU version 1:1.6.2-4.3

[أحمد المحمودي]

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-electronics-devel/attachments/20130120/96ed5ffa/attachment.pgp>