Matthew Burgess wrote: > On Fri, 3 Sep 2010 19:28:26 +0100, Ken Moffat <[email protected]> > wrote: > >> sed -i -e '/^_PKG_TEXT/s/dnl/[]dnl/' \ >> -e '/^To get/s/dnl/[]dnl/' pkg.m4 >> >> will do it, for those who dislike patching. The simple "change >> '])dnl'" version I tried also changed other lines. > > It turns out that this was actually a bug in Autoconf, not in pkg-config. > See http://lists.gnu.org/archive/html/bug-autoconf/2010-09/msg00040.html and > http://git.savannah.gnu.org/cgit/autoconf.git/commit/?id=f6030a3204a9b87e9b346e6eff4ce1117382fc86 > for the upstream details. > > Sadly, I caught this too late for 6.7, oh well! > > Here's the sed that'll achieve the same as the upstream fix (minus their > test suite addition, of course): > > sed -i -e 's/])m4_defun/[]&/' lib/m4sugar/m4sugar.m4 > > Ken, what package was it you were updating that tickled this bug? I'll > try to reproduce and confirm the fix before commiting this one to the book.
Should we add an errata item? It does strike me that this is something that would be a rare problem and we do work around it in pkg-config. -- Bruce -- http://linuxfromscratch.org/mailman/listinfo/lfs-dev FAQ: http://www.linuxfromscratch.org/faq/ Unsubscribe: See the above information page
