1269787 - Move appini_header.py invocation to moz.build

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Michael Shal [:mshal]]

We can use GENERATED_FILES.script to run it, but there are some slight complications. First, this script depends on a generated file (dist/bin/application.ini), which currently means that it will run in the misc tier. So, I've changed the misc vs. export tier logic for generated files to instead go based on file extension.

Second, even though the application.ini rule generated by moz.build specifies the 'misc' tier, make actually builds it during export since it is a dependency of application.ini.h. In order for that to continue to work when moving the application.ini.h script to moz.build, we have to make sure the paths match.

Comment 1

[Michael Shal [:mshal]]

Attachment: MozReview Request: Bug 1269787 - Choose tier for GENERATED_FILES based on extension; r?glandium

Some generated files will depend on other generated files, but still
need to be in the export tier because they are C++ headers.

Review commit: https://reviewboard.mozilla.org/r/50187/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/50187/

Comment 2

[Michael Shal [:mshal]]

Attachment: MozReview Request: Bug 1269787 - Use _pretty_path instead of full path for inputs; r?glandium

We use _pretty_path when specifying the targets of generated files, so
we need to use _pretty_path for the inputs as well. Otherwise make won't
know that they refer to the same file, and result in "No rule to make
target" errors.

Review commit: https://reviewboard.mozilla.org/r/50189/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/50189/

Comment 3

[Michael Shal [:mshal]]

Attachment: MozReview Request: Bug 1269787 - Move appini_header.py rule to moz.build; r?glandium

Review commit: https://reviewboard.mozilla.org/r/50191/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/50191/

Comment 4

[Mike Hommey [:glandium]]

Comment on attachment 8748245 [details]
MozReview Request: Bug 1269787 - Choose tier for GENERATED_FILES based on extension; r?glandium

https://reviewboard.mozilla.org/r/50187/#review47455

meh

Comment 5

[Mike Hommey [:glandium]]

Comment on attachment 8748246 [details]
MozReview Request: Bug 1269787 - Use _pretty_path instead of full path for inputs; r?glandium

https://reviewboard.mozilla.org/r/50189/#review47457

::: layout/style/test/gen-css-properties.py:12
(Diff revision 1)
> -    data = subprocess.check_output([exe])
> +    run_exe = exe if os.path.isabs(exe) else './%s' % exe
> +    data = subprocess.check_output([run_exe])

You should probably add a comment why this is needed.

Comment 6

[Mike Hommey [:glandium]]

Comment on attachment 8748247 [details]
MozReview Request: Bug 1269787 - Move appini_header.py rule to moz.build; r?glandium

https://reviewboard.mozilla.org/r/50191/#review47459

Comment 7

[Michael Shal [:mshal]]

(In reply to Mike Hommey [:glandium] from comment #5)
> Comment on attachment 8748246 [details]
> MozReview Request: Bug 1269787 - Use _pretty_path instead of full path for
> inputs; r?glandium
> 
> https://reviewboard.mozilla.org/r/50189/#review47457
> 
> ::: layout/style/test/gen-css-properties.py:12
> (Diff revision 1)
> > -    data = subprocess.check_output([exe])
> > +    run_exe = exe if os.path.isabs(exe) else './%s' % exe
> > +    data = subprocess.check_output([run_exe])
> 
> You should probably add a comment why this is needed.

Done.

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/5022b4e0ae89
https://hg.mozilla.org/integration/mozilla-inbound/rev/436ce820c291
https://hg.mozilla.org/integration/mozilla-inbound/rev/5c0ca5b6fd41

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/5022b4e0ae89
https://hg.mozilla.org/mozilla-central/rev/436ce820c291
https://hg.mozilla.org/mozilla-central/rev/5c0ca5b6fd41