X.sh in profile.xml

Ag. D. Hatzimanikas a.hatzim at gmail.com
Sun Jul 29 21:30:10 PDT 2007


On Sun, Jul 29, at 06:53 David Jensen wrote:
> Ag. D. Hatzimanikas wrote:
> >
> > With this chance, I also updated the X.sh script (/X11R6/X11R7).
> >   
> I was not aware we were making this change.  

Oh yes, sometimes I think I am like a bull in a glass shop.
I have to be a little more elegant, I guess, and I will do it now.

> It works for me, /usr/X11R7 is what I use.  
> We should put it to discussion?  Personally +1, but the 
> xorg default instructions should match.
> 

The fact is that /usr/X11R6 is outdated and should be replaced anyway, 
even with a convention like "/usr/X11R7" which looks that is in common
use and follows some kind of tradition, or by just using replaceable tags,
like we do in the same page (profile.xml) for the i18n.sh script.

Personally I feel better with "/usr/X11R7" because I think replaceable
tags can be a little confusing sometimes and in this special case we
have to use them four times, which is a little too much.

The text above the script should be enough to make the reader to
understand what is the purpose of this script and what s(h)e should in
two different cases (Xorg+Xfree).

Attached is a diff for review and suggestions.

-- 
Ag.
-------------- next part --------------
Index: postlfs/config/profile.xml
===================================================================
--- postlfs/config/profile.xml	(revision 6988)
+++ postlfs/config/profile.xml	(working copy)
@@ -281,15 +281,21 @@
         <primary sortas="e-etc-profile.d-X.sh">/etc/profile.d/X.sh</primary>
       </indexterm>
 
-      <para>If <application>X</application> is installed, the <envar>PATH</envar>
-      and <envar>PKG_CONFIG_PATH</envar> variables are also updated.</para>
+      <para>If <application>X</application> is installed in
+      <filename class="directory">/usr/X11R7</filename> directory, the
+      <envar>PATH</envar> and <envar>PKG_CONFIG_PATH</envar> variables are also
+      updated. <emphasis>Note</emphasis>: Consult the <xref linkend="xorg7"/> or
+      the <xref linkend="xfree86"/> pages of the Book, and substitute /usr/X11R7
+      with the choosen <envar>XORG_PREFIX</envar> for <application>Xorg</application>,
+      and the <envar>ProjectRoot</envar> in the case of <application>XFree86</application>.
+      </para>
 
 <screen role="root"><userinput>cat > /etc/profile.d/X.sh << "EOF"
-<literal>if [ -x /usr/X11R6/bin/X ]; then
-        pathappend /usr/X11R6/bin
+<literal>if [ -x /usr/X11R7/bin/X ]; then
+        pathappend /usr/X11R7/bin
 fi
-if [ -d /usr/X11R6/lib/pkgconfig ] ; then
-        pathappend /usr/X11R6/lib/pkgconfig PKG_CONFIG_PATH
+if [ -d /usr/X11R7/lib/pkgconfig ] ; then
+        pathappend /usr/X11R7/lib/pkgconfig PKG_CONFIG_PATH
 fi</literal>
 EOF</userinput></screen>
 


More information about the blfs-dev mailing list