773143 - Rename MOZ_ANGLE to MOZ_ANGLE_RENDERER [Port |Bug 772457 Remove compile-time option to disable ANGLE, replace it by an option to disable WebGL altogether (also disabling ANGLE)|]

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Comment 1

[Philip Chee]

https://hg.mozilla.org/mozilla-central/rev/3391c610940b#l1.12
https://hg.mozilla.org/mozilla-central/rev/3391c610940b#l2.12

Comment 2

[Philip Chee]

Do we need to change these comm-central files as well?
config/autoconf.mk.in
configure.in

Comment 3

[Serge Gautherie (:sgautherie)]

(In reply to Philip Chee from comment #2)
> Do we need to change these comm-central files as well?
> config/autoconf.mk.in
> configure.in

Yes.

Comment 5

[Mark Banner (:standard8)]

Please explain why you think we should track a bug when requesting tracking.

In this case, I can't see anything that needs to be tracked here - the defaults will still work for the standard Thunderbird configuration. For users using non-standard configs, they might need to specify two options rather than one, but that's not something worth tracking. If someone ports this, then I'd accept a patch to have it synchronised, but I'm not going to direct effort onto it specifically.

Comment 6

[Jens Hatlak (:InvisibleSmiley)]

Attachment: part 1: rename [Checkin: Comment 8]

I'm attaching two patches, one with the simple renaming (which will need no real review or discussion) and another one on top of the first that ports the extra bits from bug 772457 and some follow-up bugs concerning the same configure.in section. If it's decided that the latter one should have its own set of bugs or not be done at all, at least the first part won't be lost work.

Please note that I'm not a build system expert or anything close to that; I just felt this should be done rather sooner than later.

Comment 7

[Jens Hatlak (:InvisibleSmiley)]

Attachment: part 2: rest [Checkin: Comment 9]

[Feel free to add or change reviewers as necessary.]

Comment 8

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 678134 [details] [diff] [review]
part 1: rename [Checkin: Comment 8]

http://hg.mozilla.org/comm-central/rev/84173f8837f7

Comment 9

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 678135 [details] [diff] [review]
part 2: rest [Checkin: Comment 9]

http://hg.mozilla.org/comm-central/rev/afdbfb6acd3c
(with anti-bitrot for the first configure.in hunk)