[webkit-reviews] [PATCH] JavaScriptCore with bison 2.0
Toby Peterson
toby at opendarwin.org
Tue Jun 7 11:04:00 PDT 2005
On 07 Jun 2005, at 12.01, Darin Adler wrote:
> On Jun 7, 2005, at 1:52 AM, Toby Peterson wrote:
>
>
>> Second, bison 2.0 generates a different filename (grammar.hpp
>> instead of grammar.cpp.h). My solution's a little nasty, but it
>> works. It's worth noting that the output filename is configurable
>> on bison 2.0, e.g. --defines=grammar.cpp.h, but this would break
>> bison 1.28.
>>
>
> Looks good, I'm about to land the first half of your patch.
>
> For the filename half, though, I'd like something that does not
> include 2> /dev/null -- I don't want to hide other error messages
> bison might emit!
bison's errors will still be displayed; this only redirects cat's
stderr. If you're interesting in cat's output, it'd certainly be
possible to do something more complex:
[ -e grammar.cpp.h ] && cat grammar.cpp.h || [ -e grammar.hpp ] &&
cat grammar.hpp > grammar.h
Gorgeous, eh?
- Toby
More information about the webkit-reviews
mailing list