Misc pyconfigure cleanups

RESOLVED FIXED in Firefox 48

Status

defect
RESOLVED FIXED
3 years ago
Last year

People

(Reporter: glandium, Assigned: glandium)

Tracking

(Blocks 1 bug)

unspecified
mozilla48
Dependency tree / graph

Firefox Tracking Flags

(firefox48 fixed)

Details

Attachments

(2 attachments)

Assignee

Description

3 years ago
No description provided.
Assignee

Comment 2

3 years ago
The configure.in is a small wrapper around python configure, that is
still a m4 script so that people running autoconf manually can still
do so without breaking their stuff (and we have jobs that do that
on automation as well).

But considering how simple the m4 is, to avoid having the autoconf
checking code twice (once in client.mk and once in
build/moz.configure/old.configure), we can just process it with sed
instead of autoconf.

Review commit: https://reviewboard.mozilla.org/r/45905/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/45905/
Attachment #8740684 - Flags: review?(gps) → review+
Comment on attachment 8740684 [details]
MozReview Request: Bug 1264129 - Add verbosity for the shell and autoconf checks in moz.configure. r?gps

https://reviewboard.mozilla.org/r/45903/#review42471
Comment on attachment 8740685 [details]
MozReview Request: Bug 1264129 - Use sed to update configure from configure.in, instead of autoconf. r?gps

https://reviewboard.mozilla.org/r/45905/#review42475

Nice hack.

Always happy to r+ files that net remove lines from client.mk.
Attachment #8740685 - Flags: review?(gps) → review+

Updated

Last year
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.