docs/88454: [patch] Porter's Handbook: Update sec. on Shared Libraries

[Simon L. Nielsen]

The following reply was made to PR docs/88454; it has been noted by GNATS.

From: "Simon L. Nielsen" <simon at FreeBSD.org>
To: Volker Stolz <vs at FreeBSD.org>
Cc: FreeBSD-gnats-submit at FreeBSD.org
Subject: Re: docs/88454: [patch] Porter's Handbook: Update sec. on Shared Libraries
Date: Thu, 3 Nov 2005 20:28:02 +0100

 --BXVAT5kNtrzKuDFl
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: inline
 Content-Transfer-Encoding: quoted-printable
 
 On 2005.11.03 15:26:04 +0100, Volker Stolz wrote:
 
 > --- shlibs.diff begins here ---
 > Index: book.sgml
 > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
 =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
 =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
 > RCS file: /home/dcvs/doc/en_US.ISO8859-1/books/porters-handbook/book.sgml=
 ,v
 > retrieving revision 1.625
 > diff -u -u -r1.625 book.sgml
 > --- book.sgml	2 Nov 2005 21:43:56 -0000	1.625
 > +++ book.sgml	3 Nov 2005 14:22:43 -0000
 > @@ -3869,6 +3869,16 @@
 >  	<programlisting>INSTALLS_SHLIB=3D yes
 >  LDCONFIG_DIRS=3D  %%PREFIX%%/lib/foo %%PREFIX%%/lib/bar</programlisting>
 > =20
 > +	<para>Remember that non-standard directories will not be passed to
 
 I would suggest "Note that" instead.  Just sounds better IMO.
 
 > +	  <literal>/sbin/ldconfig</literal> on a reboot! If any port really
 
 You should use &man.ldconfig.8; instead of <literal>/...</literal>.
 Also I would suggest simply saying "boot" instead of "reboot".  Also
 you should use double space before new sentence (IE. after the "!").
 
 > +	  needs this to work, install a startup-script as
 > +	  <literal>x11/kdelibs3</literal> does. Please double-check, often
 
 Use <filename role=3D"package">x11/kdelibs3</filename> and double space
 after ".".
 
 > +	  this is not necessary at all or can be avoided through
 > +	  <literal>-rpath</literal> or setting <makevar>LD_RUN_PATH</makevar>
 
 Since LD_LIBRARY_PATH is an environment variable you should use <envar>.
 
 > +	  during linking (see <literal>lang/moscow_ml</literal> for an example),
 
 <filename role=3D"package"> again.
 
 > +	  or through a shell-wrapper which sets <makevar>LD_LIBRARY_PATH</makev=
 ar>
 > +	  before invoking the binary, like <literal>www/mozilla</literal> does.=
 </para>
 
 <filename role=3D"package"> again.
 
 Hope this helps.
 
 --=20
 Simon L. Nielsen
 
 --BXVAT5kNtrzKuDFl
 Content-Type: application/pgp-signature
 Content-Disposition: inline
 
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v1.4.2 (FreeBSD)
 
 iD8DBQFDamTBh9pcDSc1mlERAmDgAJ0dzhdRLO7RaTmOM0JHqnS+QVULHwCaAxm7
 hcOxO6euHV9sSL4gZqnygss=
 =uDRv
 -----END PGP SIGNATURE-----
 
 --BXVAT5kNtrzKuDFl--