Dan Nicholson wrote:

> Comments on the attached diff.

It's a bit hard to look at the diff and understand what the final page
would look like, but a couple of comments:

+    <bridgehead renderas="sect4">Required</bridgehead>
+    <para role="required"><xref linkend="expat"/> or <xref
linkend="pkgconfig"/>
+    and <xref linkend="libxml2"/></para>

Why is it expat *or* pkgconfig?  These packages don't seem like they
would do the same thing.

Do the Python bindings not get built by default any more?  You removed
the --disable-python switch so I would guess that they don't.

In making these types of reviews, wouldn't it be better to just build
the book on quantum or anduin and point to that so we can see everything
in context?

  -- Bruce
-- 
http://linuxfromscratch.org/mailman/listinfo/blfs-dev
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page

Reply via email to