A few comments below:
On 01/25/2013 01:48 AM, [email protected] wrote:
> Author: randy
> Date: Thu Jan 24 16:48:47 2013
> New Revision: 10973
>
> Log:
> Minor changes to various packages
>
> Modified:
> trunk/BOOK/general/genlib/libatasmart.xml
> trunk/BOOK/general/genlib/libical.xml
> trunk/BOOK/general/prog/php.xml
> trunk/BOOK/general/prog/tcl.xml
> trunk/BOOK/general/prog/tk.xml
> trunk/BOOK/general/sysutils/bluez.xml
> trunk/BOOK/gnome/core/libsecret.xml
> trunk/BOOK/multimedia/libdriv/gst-plugins-bad.xml
> trunk/BOOK/multimedia/videoutils/mplayer.xml
> trunk/BOOK/networking/netlibs/geoclue.xml
> trunk/BOOK/postlfs/filesystems/fuse.xml
> trunk/BOOK/xincludes/static-libraries.xml
>
> Modified: trunk/BOOK/general/genlib/libatasmart.xml
> ==============================================================================
> --- trunk/BOOK/general/genlib/libatasmart.xml Thu Jan 24 08:36:58 2013
> (r10972)
> +++ trunk/BOOK/general/genlib/libatasmart.xml Thu Jan 24 16:48:47 2013
> (r10973)
> @@ -94,7 +94,7 @@
> Now, as the <systemitem class="username">root</systemitem> user:
> </para>
>
> -<screen role="root"><userinput>make install</userinput></screen>
> +<screen role="root"><userinput>make
> docdir=/usr/share/doc/libatasmart-&libatasmart-version;
> install</userinput></screen>
>
> </sect2>
>
> @@ -136,7 +136,7 @@
> <term><command>skdump</command></term>
> <listitem>
> <para>
> - is an utility that reports on the status of the disk.
> + is a utility that reports on the status of the disk.
> </para>
> <indexterm zone="libatasmart skdump">
> <primary sortas="b-skdump">skdump</primary>
> @@ -148,7 +148,7 @@
> <term><command>sktest</command></term>
> <listitem>
> <para>
> - is an utility to issue disks tests.
> + is a utility to issue disks tests.
> </para>
> <indexterm zone="libatasmart sktest">
> <primary sortas="b-sktest">sktest</primary>
>
> Modified: trunk/BOOK/general/genlib/libical.xml
> ==============================================================================
> --- trunk/BOOK/general/genlib/libical.xml Thu Jan 24 08:36:58 2013
> (r10972)
> +++ trunk/BOOK/general/genlib/libical.xml Thu Jan 24 16:48:47 2013
> (r10973)
> @@ -56,10 +56,11 @@
> </listitem>
> </itemizedlist>
>
> + <!-- There is no support for the current version of BLFS BDB
> <bridgehead renderas="sect3">libical Dependencies</bridgehead>
>
> <bridgehead renderas="sect4">Optional</bridgehead>
> - <para role="optional"><xref linkend="db"/></para>
> + <para role="optional"><xref linkend="db"/></para> -->
>
> <para condition="html" role="usernotes">User Notes:
> <ulink url="&blfs-wiki;/libical"/></para>
> @@ -83,13 +84,15 @@
>
> </sect2>
>
> + <!-- There is no support for the current version of BLFS BDB
> <sect2 role="commands">
> <title>Command Explanations</title>
>
> - <para><option>--with-bdb4</option>: This option adds support for
> + <para><option>-with-bdb4</option>: This option adds support for
> <application>Berkeley DB 4.0</application>.</para>
>
> </sect2>
> + -->
>
> <sect2 role="content">
> <title>Contents</title>
>
> Modified: trunk/BOOK/general/prog/php.xml
> ==============================================================================
> --- trunk/BOOK/general/prog/php.xml Thu Jan 24 08:36:58 2013 (r10972)
> +++ trunk/BOOK/general/prog/php.xml Thu Jan 24 16:48:47 2013 (r10973)
> @@ -204,13 +204,13 @@
> install -v -m644 php.ini-production /etc/php.ini &&
>
> install -v -m755 -d /usr/share/doc/php-&php-version; &&
> -install -v -m644 CODING_STANDARDS EXTENSIONS INSTALL NEWS \
> - README* UPGRADING* php.gif \
> - /usr/share/doc/php-&php-version; &&
> -ln -v -sfn /usr/lib/php/doc/Archive_Tar/docs/Archive_Tar.txt \
> - /usr/share/doc/php-&php-version; &&
> -ln -v -sfn /usr/lib/php/doc/Structures_Graph/docs \
> - /usr/share/doc/php-&php-version;</userinput></screen>
> +install -v -m644 CODING_STANDARDS EXTENSIONS INSTALL NEWS \
> + README* UPGRADING* php.gif \
> + /usr/share/doc/php-&php-version; &&
> +ln -v -sfn /usr/lib/php/doc/Archive_Tar/docs/Archive_Tar.txt \
> + /usr/share/doc/php-&php-version; &&
> +ln -v -sfn /usr/lib/php/doc/Structures_Graph/docs \
> + /usr/share/doc/php-&php-version;</userinput></screen>
>
> <para>The pre-built HTML documentation is packaged in two forms: a
> tarball
> containing many individual files, useful for quick loading into your
> @@ -272,7 +272,7 @@
> <para><parameter>--with-readline</parameter>: This parameter
> enables command line <application>Readline</application>
> support.</para>
>
> - <para><parameter>--disable-libxml</parameter>: This parameter
> + <para><option>--disable-libxml</option>: This option
> allows building PHP without <application>libxml2</application>
> installed.</para>
>
I'd like to know how do you know when to use option or parameter. On
this one you are changing parameter to option, while below you are
changing option to parameter.
Just curious. I was always using template/template.xml for that case.
>
> Modified: trunk/BOOK/general/prog/tcl.xml
> ==============================================================================
> --- trunk/BOOK/general/prog/tcl.xml Thu Jan 24 08:36:58 2013 (r10972)
> +++ trunk/BOOK/general/prog/tcl.xml Thu Jan 24 16:48:47 2013 (r10973)
> @@ -161,8 +161,8 @@
> <title>Command Explanations</title>
>
> <para>
> - <option>$([ $(uname -m) = x86_64 ] && echo
> - --enable-64bit)</option>: This switch is used to enable 64 bit
> + <parameter>$([ $(uname -m) = x86_64 ] && echo
> + --enable-64bit)</parameter>: This switch is used to enable 64 bit
> support in <application>Tcl</application> on 64 bit operating
> systems.
> </para>
>
> Modified: trunk/BOOK/general/prog/tk.xml
> ==============================================================================
> --- trunk/BOOK/general/prog/tk.xml Thu Jan 24 08:36:58 2013 (r10972)
> +++ trunk/BOOK/general/prog/tk.xml Thu Jan 24 16:48:47 2013 (r10973)
> @@ -127,8 +127,8 @@
> <title>Command Explanations</title>
>
> <para>
> - <option>$([ $(uname -m) = x86_64 ] && echo
> - --enable-64bit)</option>: This switch is used to enable 64 bit
> + <parameter>$([ $(uname -m) = x86_64 ] && echo
> + --enable-64bit)</parameter>: This switch is used to enable 64 bit
> support in <application>Tk</application> on 64 bit operating
> systems.
> </para>
>
> Modified: trunk/BOOK/general/sysutils/bluez.xml
> ==============================================================================
> --- trunk/BOOK/general/sysutils/bluez.xml Thu Jan 24 08:36:58 2013
> (r10972)
> +++ trunk/BOOK/general/sysutils/bluez.xml Thu Jan 24 16:48:47 2013
> (r10973)
> @@ -82,7 +82,7 @@
> <para role="optional">
> <xref linkend="alsa-lib"/>,
> <xref linkend="check"/>,
> - <xref linkend="cups"/>,
> + <!-- <xref linkend="cups"/>, -->
Why did you comment out Cups? It is used for CUPS bluetooth backend.
> <xref linkend="gst-plugins-base"/>,
> <xref linkend="libsndfile"/> and
> <xref linkend="libusb-compat"/>
> @@ -163,7 +163,8 @@
>
> <screen role="root"><userinput>for CONFFILE in audio input network serial ;
> do
> install -v -m644 ${CONFFILE}/${CONFFILE}.conf
> /etc/bluetooth/${CONFFILE}.conf
> -done</userinput></screen>
> +done
> +unset CONFFILE</userinput></screen>
>
> <para>
> If desired, install the API documentation as the
> @@ -179,71 +180,73 @@
> <title>Command Explanations</title>
>
> <para>
> - <option>--enable-bccmd</option>: This switch enables building of the
> BCCMD
> + <parameter>--enable-bccmd</parameter>: This switch enables building of
> the BCCMD
> interface utility.
> </para>
>
> <para>
> - <option>--enable-dfutool</option>: This switch enables building of the
> DFU
> + <parameter>--enable-dfutool</parameter>: This switch enables building
> of the DFU
> firmware upgrade utility.
> </para>
>
> <para>
> - <option>--enable-dund</option>: This switch enables building of the
> DUN daemon.
> + <parameter>--enable-dund</parameter>: This switch enables building of
> the DUN daemon.
> </para>
>
> <para>
> - <option>--enable-hid2hci</option>: This switch enables building of the
> HID
> + <parameter>--enable-hid2hci</parameter>: This switch enables building
> of the HID
> mode switching utility.
> </para>
>
> <para>
> - <option>--enable-hidd</option>: This switch enables building of the
> HID daemon.
> + <parameter>--enable-hidd</parameter>: This switch enables building of
> the HID daemon.
> </para>
>
> <para>
> - <option>--enable-pand</option>: This switch enables building of the
> PAN daemon.
> + <parameter>--enable-pand</parameter>: This switch enables building of
> the PAN daemon.
> </para>
>
> <para>
> - <option>--enable-tools</option>: This switch enables building of the
> Bluetooth
> + <parameter>--enable-tools</parameter>: This switch enables building of
> the Bluetooth
> utilities.
> </para>
>
> <para>
> - <option>--enable-wiimote</option>: This switch enables building of the
> Wii
> + <parameter>--enable-wiimote</parameter>: This switch enables building
> of the Wii
> Remote plugin.
> </para>
>
> <para>
> - <option>--disable-test</option>: This switch disables installation of
> the
> + <parameter>--disable-test</parameter>: This switch disables
> installation of the
> test programs.
> </para>
>
> <para>
> - <option>--without-systemdunitdir</option>: This switch disables
> installation
> + <parameter>--without-systemdunitdir</parameter>: This switch disables
> installation
> of the systemd units.
> </para>
>
Now that you pointed it out, I've checked it again, and it is correct.
Also, same applies for --enable-alsa. You could just remove all of the
code instead of commenting it out. Keeps files cleaner.
> <para>
> - <option>--enable-alsa</option>: This switch enables ALSA support. Use
> if you
> - have <xref linkend="alsa-lib"/> installed.
> + <option>--enable-cups</option>: This switch enables CUPS backend
> support. Note
> + that CUPS does not need to be installed for this support.
> </para>
>
> + <!-- These options are automatically found and used if the packages are
> + available. There is no need to use these switches.
> <para>
> - <option>--enable-cups</option>: This switch enables CUPS backend
> support. Use
> - if you have <xref linkend="cups"/> installed.
> + <option>-enable-alsa</option>: This switch enables ALSA support. Use
> if you
> + have <xref linkend="alsa-lib"/> installed.
> </para>
>
> <para>
> - <option>--enable-gstreamer</option>: This switch enables GStreamer
> support. Use
> + <option>-enable-gstreamer</option>: This switch enables GStreamer
> support. Use
> if you have <xref linkend="gst-plugins-base"/> installed.
> </para>
>
> <para>
> - <option>--enable-usb</option>: This switch enables USB support. Use if
> you have
> + <option>-enable-usb</option>: This switch enables USB support. Use if
> you have
> <xref linkend="libusb-compat"/> installed.
> - </para>
> + </para> -->
>
> </sect2>
>
>
> Modified: trunk/BOOK/gnome/core/libsecret.xml
> ==============================================================================
> --- trunk/BOOK/gnome/core/libsecret.xml Thu Jan 24 08:36:58 2013
> (r10972)
> +++ trunk/BOOK/gnome/core/libsecret.xml Thu Jan 24 16:48:47 2013
> (r10973)
> @@ -91,7 +91,8 @@
>
> <bridgehead renderas="sect4">Optional (Required for the
> testsuite)</bridgehead>
> <para role="optional">
> - <xref linkend="gjs"/> and
> + <xref linkend="gjs"/>,
> + <xref linkend="dbus-python"/> and
> <xref linkend="pygobject3"/>
> </para>
>
Not important, but I guess it would be nice if you sorted it
alphabeticaly as I did for every other package in GNOME.
> @@ -112,7 +113,8 @@
> make</userinput></screen>
>
> <para>
> - To test the results, issue: <command>make check</command>.
> + To test the results, issue: <command>make -k check</command>. The
> + JavaScript tests fail for unknown reasons.
> </para>
>
This is strange. All tests passed on my side. You need to run them from
a GUI though.
> <para>
>
> Modified: trunk/BOOK/multimedia/libdriv/gst-plugins-bad.xml
> ==============================================================================
> --- trunk/BOOK/multimedia/libdriv/gst-plugins-bad.xml Thu Jan 24 08:36:58
> 2013 (r10972)
> +++ trunk/BOOK/multimedia/libdriv/gst-plugins-bad.xml Thu Jan 24 16:48:47
> 2013 (r10973)
> @@ -181,12 +181,12 @@
>
> <para>
> <parameter>--with-gtk=3.0</parameter>: Build against
> <application>GTK+ 3</application>
> - instead of the default <application>GTK+ 2</application>: recommended
> for a
> + instead of the default <application>GTK+ 2</application>; recommended
> for a
> <application>GNOME</application> Desktop.
> </para>
>
> <para>
> - <option>--disable-examples</option>: if the dependencies for the camera
> + <parameter>--disable-examples</parameter>: If the dependencies for the
> camera
> driver have been satisfied, the associated example code fails to
> build.
> </para>
>
>
> Modified: trunk/BOOK/multimedia/videoutils/mplayer.xml
> ==============================================================================
> --- trunk/BOOK/multimedia/videoutils/mplayer.xml Thu Jan 24 08:36:58
> 2013 (r10972)
> +++ trunk/BOOK/multimedia/videoutils/mplayer.xml Thu Jan 24 16:48:47
> 2013 (r10973)
> @@ -190,8 +190,8 @@
> <xref linkend="gtk2"/> (required for the GUI),
> <ulink url="http://www.ladspa.org/";>LADSPA</ulink>,
> <ulink url="http://sourceforge.net/projects/bs2b/";>libbs2b</ulink>,
> - <ulink url="http://www.lirc.org/";>LIRC</ulink> and
> - <ulink url="http://www.dolda2000.com/~fredrik/lirccd/";>LIRC Client
> Daemon</ulink>,
> + <ulink url="http://www.lirc.org/";>LIRC</ulink> (and
> + <ulink url="http://www.dolda2000.com/~fredrik/lirccd/";>LIRC Client
> Daemon</ulink>),
> <xref linkend="unrar"/>, and
> <xref linkend="libxslt"/>,
> <xref linkend="DocBook"/> and
> @@ -311,8 +311,8 @@
> installed <xref linkend="ffmpeg"/>. MPlayer includes a copy of the
> FFmpeg
> source and without this option it will statically compile the included
> version of FFmpeg into itself. Dynamically linking to a system
> installed
> - FFmpeg makes MPlayer and Mencoder about 9MB smaller. If MPlayer is
> crashy
> - or unstable linked to a system installed FFmpeg, try recompiling
> MPlayer
> + FFmpeg makes MPlayer and Mencoder about 9MB smaller. If MPlayer
> exhibits
> + unstable behavior linked to a system installed FFmpeg, try recompiling
> MPlayer
> without this option (it is always more thoroughly tested with the
> included
> FFmpeg).
> </para>
>
> Modified: trunk/BOOK/networking/netlibs/geoclue.xml
> ==============================================================================
> --- trunk/BOOK/networking/netlibs/geoclue.xml Thu Jan 24 08:36:58 2013
> (r10972)
> +++ trunk/BOOK/networking/netlibs/geoclue.xml Thu Jan 24 16:48:47 2013
> (r10973)
> @@ -161,7 +161,7 @@
> </para>
>
> <para>
> - <option>--libexecdir=/usr/lib/geoclue</option>: This option installs
> + <parameter>--libexecdir=/usr/lib/geoclue</parameter>: This option
> installs
> <application>GeoClue's</application> private executables into
> <filename class="libraryfile">/usr/lib/geoclue</filename>.
> </para>
>
I've always wanted to ask for this one.
Can't we just stop explaining --whateverdir=/path/to/whatever switches?
I mean, it should be obvious that it is setting installation paths for,
lets say, docs, executable, whatever.
> Modified: trunk/BOOK/postlfs/filesystems/fuse.xml
> ==============================================================================
> --- trunk/BOOK/postlfs/filesystems/fuse.xml Thu Jan 24 08:36:58 2013
> (r10972)
> +++ trunk/BOOK/postlfs/filesystems/fuse.xml Thu Jan 24 16:48:47 2013
> (r10973)
> @@ -97,6 +97,19 @@
> <sect2 role="installation">
> <title>Installation of Fuse</title>
>
> + <note>
> + <para>
> + After the <command>configure</command> script has finished you will
> + see a warning shown below. You can safely disregard this warning.
> + </para>
> +
> +<screen><computeroutput>configure: WARNING:
> +******************************************************************
> +* Please install util-linux version 2.18 or later which supports *
> +* --fake and --no-canonicalize options in mount and umount *
> +******************************************************************</computeroutput></screen>
> + </note>
> +
> <para>
> Install <application>Fuse</application> by running the following
> commands:
> @@ -106,6 +119,11 @@
> make</userinput></screen>
>
> <para>
> + If you have <xref linkend="doxygen"/> installed and wish to build the
> API
> + documentation, issue <command>doxygen doc/Doxyfile</command>.
> + </para>
> +
> + <para>
> This package does not come with a test suite.
> </para>
>
> @@ -114,9 +132,24 @@
> </para>
>
> <screen role="root"><userinput>make install &&
> -mv -v /usr/lib/libfuse.so.* /lib &&
> +
> +mv -v /usr/lib/libfuse.so.* /lib &&
> ln -sfv ../../lib/libfuse.so.&fuse-version; /usr/lib/libfuse.so &&
> -rm -rf /tmp/init.d</userinput></screen>
> +rm -rf /tmp/init.d &&
> +
> +install -v -m755 -d /usr/share/doc/fuse-&fuse-version; &&
> +install -v -m644 doc/{how-fuse-works,kernel.txt} \
> + /usr/share/doc/fuse-&fuse-version;</userinput></screen>
> +
> + <para>
> + If you built the API documentation, install it as the
> + <systemitem class="username">root</systemitem> user by issuing the
> + following commands:
> + </para>
> +
> +<screen role="root"><userinput>install -v -m755 -d
> /usr/share/doc/fuse-&fuse-version;/api &&
> +install -v -m644 doc/html/* \
> +
> /usr/share/doc/fuse-&fuse-version;/api</userinput></screen>
> </sect2>
>
> <sect2 role="commands">
> @@ -126,17 +159,18 @@
> href="../../xincludes/static-libraries.xml"/>
>
> <para>
> - <parameter>INIT_D_PATH=/tmp/init.d</parameter>: This parameter makes it
> - install its bootscript into <filename>/tmp/init.d</filename>.
> + <parameter>INIT_D_PATH=/tmp/init.d</parameter>: This parameter installs
> + the bootscript into <filename class="directory">/tmp/init.d</filename>
> as
> + a bootscript is not required.
> </para>
>
> <para>
> - <command>mv -v /usr/lib/libfuse.so.* /lib</command>: This moves
> - <application>FUSE</application> library to root filesystem
> - so that it is available at early boot in case
> - <filename class="directory">/usr</filename> is on seperate
> - partition and <xref linkend="ntfs-3g"/> is built with system
> - <application>FUSE</application>.
> + <command>mv -v /usr/lib/libfuse.so.* /lib</command>: This moves the
> + <application>FUSE</application> library to the root filesystem
> + so that it is available early in the boot process in case
> + <filename class="directory">/usr</filename> is mounted on a separate
> + partition and <xref linkend="ntfs-3g"/> is built with a
> system-installed
> + version of <application>FUSE</application>.
> </para>
>
> <para>
>
> Modified: trunk/BOOK/xincludes/static-libraries.xml
> ==============================================================================
> --- trunk/BOOK/xincludes/static-libraries.xml Thu Jan 24 08:36:58 2013
> (r10972)
> +++ trunk/BOOK/xincludes/static-libraries.xml Thu Jan 24 16:48:47 2013
> (r10973)
> @@ -6,6 +6,6 @@
> $Date$ -->
>
> <para>
> - <option>--disable-static</option>: This switch prevents
> + <parameter>--disable-static</parameter>: This switch prevents
> installation of static versions of the libraries.
> </para>
>
--
http://linuxfromscratch.org/mailman/listinfo/blfs-book
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page
