xul preprocessor dies with certain define values




Build Config
15 years ago
6 months ago


(Reporter: Scott MacGregor, Unassigned)


Windows XP

Firefox Tracking Flags

(Not tracked)



(1 attachment)



15 years ago
When I try to run with the xul processor in Firebird or Thunderbird, the
processing script fails to run because of a parsing error with the list of
environment defines.

I did a little trial and error hacking in make-jar.pl and I think I've found the
 particular define that is causing problems:


I think these slashes are causing the script that parses the define list to
terminate early, treating the next define as the processor file argument:
$preprocessor $defines $preproc_file > $destPath")

For reference, here is the entire define list that gets generated when I build
thunderbird or firebird:

0x400 -DSTDC_HEADERS=1 -DNEW_H=\<new\> -DWIN32_LEAN_AND_MEAN=1 -DNO_X11=1 -D_X86
R=".mozilla" -DCPP_THROW_NEW=throw\(\) -DMOZ_XUL=1 -DMOZ_DLL_SUFFIX=".dll" -DJS_
I would guess this isn't a bug with the preprocessor itself but with the script
that calls it (it sounds like someone is interpolating into a shell without
escaping first, a big no-no).
Assignee: ian → mozbugs-build
preprocessor -> bryner
Assignee: mozbugs-build → bryner

Comment 3

15 years ago
Created attachment 126495 [details] [diff] [review]
ugly work around for thunderbird users that run into this problem

bryner, any chance you can take a look at this problem with the xul
pre-processor? I get at least one new person a week that runs into this issue,
telling me they can't build thunderbird.

Comment 4

14 years ago
Is this bug still an issue now that NEW_H was moved into xpcom-config.h ?


Comment 5

14 years ago
unfortunately I no longer have access to the two windows machines that had this
problem for me. Maybe someone on the cc list who ran into this could try again?
still seeing the problem today while working on Standalone Composer
Didn't we fix this?

Comment 8

14 years ago
I don't think so. We fixed it for -E, but not I think in other cases. I don't
think this is a problem with the preprocessor itself, but probably with
make-jars.pl (calling make-jars.pl unescapes the argument, and it's not being
Ah, ok. So not a preprocessor bug. Good good.
Product: Browser → Seamonkey
Assignee: bryner → nobody
QA Contact: granrosebugs → build-config
Component: Build Config → Build Config
Product: SeaMonkey → Core

Comment 10

6 months ago
Mass bug cleanup for Core:Build Config.

If you feel this bug has been closed in error, please re-open with new details.
Last Resolved: 6 months ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.