r6993 - in trunk/BOOK: . introduction/welcome postlfs/shells

Randy McMurchy randy at linuxfromscratch.org
Wed Aug 1 06:17:35 PDT 2007


ag at linuxfromscratch.org wrote:
> Author: ag
> Date: 2007-07-31 13:21:02 -0600 (Tue, 31 Jul 2007)
> New Revision: 6993
> 
> Modified:
>    trunk/BOOK/general.ent
>    trunk/BOOK/introduction/welcome/changelog.xml
>    trunk/BOOK/postlfs/shells/dash.xml
> Log:
> Updated to Dash-0.5.4

Ag, there are several things you need to fix with this commit.
Don't worry about it, but  please fix them at your convenience.


>      <listitem>
> +      <para>July 31th, 2007</para>
> +      <itemizedlist>

It would be July 31st, not July 31th.



>    <!ENTITY dash-download-http "http://gondor.apana.org.au/~herbert/dash/files/dash-&dash-version;.tar.gz">
> -  <!ENTITY dash-download-ftp  "ftp://ftp.debian.org/debian/pool/main/d/dash/dash_&dash-version;.orig.tar.gz">
> -  <!ENTITY dash-md5sum        "1a3cd6669459be4344ec55ec9d4914f8">
> +  <!ENTITY dash-download-ftp  "">
> +  <!ENTITY dash-md5sum        "bc457e490a589d2f87f2333616b67931">
>    <!ENTITY dash-size          "200 KB">
>    <!ENTITY dash-buildsize     "1.8 MB">
>    <!ENTITY dash-time          "0.1 SBU">

As Manuel mentioned, use " " for an empty entity.


> -        <para>Download (FTP): <ulink url="&dash-download-ftp;"/></para>
> -      </listitem>
> -      <listitem>

Please add this back. *Every* package page should have both the
HTTP and FTP download URL listitem entries. One of them may be
empty, but it should be there in case we add an FTP URL later.


>  
> -    <bridgehead renderas="sect3">Additional Downloads</bridgehead>
> -    <itemizedlist spacing='compact'>
> -      <listitem>
> -        <para>Required Patch containing bug fixes from upstream: <ulink
> -        url="&patch-root;/dash-&dash-version;-fixes-1.patch"/></para>
> -      </listitem>
> -    </itemizedlist>
> +    <bridgehead renderas="sect3">Dash Dependencies</bridgehead>
>  
> +    <bridgehead renderas="sect4">Optional</bridgehead>
> +    <para role="optional"><ulink url="http://www.thrysoee.dk/editline/">libedit</ulink>
> +    (command line editor library)</para>
> +

Please remove the libedit dependency. We don't put those in the
book for any of the packages (there may still be one or two, but
we are removing them as we discover them). We don't put them
because it breaks readline functionality and libedit is really
only for BSD systems.




> -    <para><command>LC_ALL=C make</command>: This ensures that the
> -    <command>sort</command> command executed during the build process returns
> -    the expected results.</para>
> +    <para><option>--with-libedit</option>: To compile <application>Dash</application>
> +    with libedit support.</para>

Please remove this as well (BSD only).

-- 
Randy




More information about the blfs-book mailing list