This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [trunk][patch] Install headers and plugin-version.o


>> + Â Â Â case $$path in \
>> + Â Â Â "$(srcdir)"*/config/* | "$(srcdir)"*.def ) \
>
> Sure you didn't mean this here?
> Â Â Â Â Â"$(srcdir)"/config/* | "$(srcdir)"/*.def ) \

Probably just a mistake. Will add the / and tent.

>> + Â Â Â Â base=`echo $$path | sed "s|$(srcdir)||"`;; \
>
> Ouch. Â$(srcdir) could contain a backslash, a dot, and other
> regex-active characters. ÂAutomake uses one of two methods to strip
> $(srcdir)/, namely either
> Âsrcdirstrip=`echo "$(srcdir)" | sed 's|.|.|g'`; \
> Âcase $$p in \
> Â Â$(srcdir)/*) f=`echo "$$p" | sed "s|^$$srcdirstrip/||"`;; \
> Â Â*) f=$$p;; \
> Âesac;
>
> or
> Âsrcdirstrip=`echo "$(srcdir)" | sed 's/[].[^$$\\*|]/\\\\&/g'`; \
> Âlist=`for p in $$list; do echo "$$p"; done | sed -e "s|$$srcdirstrip/||"`
>
> where the second works on lists and doesn't require case matching for
> whether a path contains $(srcdir)/ or not.

Will try that too. Thanks.

> Cheers,
> Ralf
>


Cheers,
-- 
Rafael Avila de Espindola

Google | Gordon House | Barrow Street | Dublin 4 | Ireland
Registered in Dublin, Ireland | Registration Number: 368047


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]