bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: HP-UX sed limited to 100 commands problematic for modules/wchar


From: Eric Blake
Subject: Re: HP-UX sed limited to 100 commands problematic for modules/wchar
Date: Tue, 08 Feb 2011 13:52:30 -0700
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.13) Gecko/20101209 Fedora/3.1.7-0.35.b3pre.fc14 Lightning/1.0b3pre Mnenhy/0.8.3 Thunderbird/3.1.7

On 02/08/2011 12:45 PM, Albert Chin wrote:
>   wchar.h: wchar.in.h $(CXXDEFS_H) $(ARG_NONNULL_H) $(WARN_ON_USE_H)
>         $(AM_V_GEN)rm -f address@hidden $@ && \
>         { echo '/* DO NOT EDIT! GENERATED AUTOMATICALLY! */'; \
> 27:       sed -e 's|@''INCLUDE_NEXT''@|$(INCLUDE_NEXT)|g' \
>               -e 's|@''PRAGMA_SYSTEM_HEADER''@|@PRAGMA_SYSTEM_HEADER@|g' \
>               -e 's|@''PRAGMA_COLUMNS''@|@PRAGMA_COLUMNS@|g' \
>               ...
>               -e '/definitions of _GL_FUNCDECL_RPL/r $(CXXDEFS_H)' \
>               -e '/definition of _GL_ARG_NONNULL/r $(ARG_NONNULL_H)' \
>               -e '/definition of _GL_WARN_ON_USE/r $(WARN_ON_USE_H)' \
> 128:        < $(srcdir)/wchar.in.h; \
> 
> So, 101 lines which causes sed to fail.
> 
> I suppose we need to require AC_PROG_SED and use $SED instead of sed
> accordingly?

Thanks for the report.  However, the correct fix is to break this into
multiple sed scripts (we've already done it for other large headers,
such as unistd).

-- 
Eric Blake   address@hidden    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]