bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

Remove MOZ_WIDGET_GTK2

RESOLVED FIXED in mozilla27

Status

Firefox Build System
General
RESOLVED FIXED
5 years ago
5 months ago

People

(Reporter: stransky, Assigned: stransky)

Tracking

Trunk
mozilla27
All
Linux

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 4 obsolete attachments)

Let's remove MOZ_WIDGET_GTK2 and use MOZ_WIDGET_GTK everywhere. It helps to write the plugin code for gtk2/gtk3 toolkits.
Created attachment 805958 [details] [diff] [review]
patch
Attachment #805958 - Flags: review?(karlt)
Comment on attachment 805958 [details] [diff] [review]
patch

>+#if !(MOZ_WIDGET_GTK == 2)

(MOZ_WIDGET_GTK != 2) or (MOZ_WIDGET_GTK > 2) please.
Attachment #805958 - Flags: review?(karlt) → review+
Created attachment 806488 [details] [diff] [review]
patch for check in

Thanks!
Attachment #806488 - Flags: checkin?
Keywords: checkin-needed
Attachment #805958 - Attachment is obsolete: true
Comment on attachment 806488 [details] [diff] [review]
patch for check in

For single patch bugs, please just use the checkin-needed keyword :)
Attachment #806488 - Flags: checkin? → checkin+
This was backed out for crashtest-ipc and reftest-ipc failures, along with Linux mochitest-1 and reftest failures. Please rebase this patch and run it through Try again (linux32+64 opt+debug). If it's still green, it's probably a needs-clobber situation. In that case, please add a note to the CLOBBER file so that the build machines clobber first. Sorry for the hassle.

https://tbpl.mozilla.org/php/getParsedLog.php?id=28095796&tree=Mozilla-Inbound
https://tbpl.mozilla.org/php/getParsedLog.php?id=28099533&tree=Mozilla-Inbound
https://tbpl.mozilla.org/php/getParsedLog.php?id=28099217&tree=Mozilla-Inbound
Martin, can you remove the DownloadPlatform.cpp changes here, please because bug 884708 has the appropriate solution there.  That will have to land first.

I'm guessing the failures are for new instances of MOZ_WIDGET_GTK2 somewhere, which is why we need to remove this variable.  This kind of change shouldn't need a clobber.
Created attachment 807654 [details] [diff] [review]
patch, v2

Thanks. This patch does not touch configure.in nor DownloadPlatform.cpp so it sould be safe to apply it. 

We can do the configure.in change later when all parts are done.
Attachment #807654 - Flags: review+
Created attachment 807656 [details] [diff] [review]
patch, v3

Fixed a typo.
Attachment #807654 - Attachment is obsolete: true
Attachment #807656 - Flags: review+

Comment 12

5 years ago
Are you going to fix comm-central too?
(In reply to Philip Chee from comment #12)
> Are you going to fix comm-central too?

Ahh, good point. I'll add it to my todo-list.
Attachment #806488 - Attachment is obsolete: true
Created attachment 808503 [details] [diff] [review]
patch, v4

Looks like .js preprocessor doesn't like brackets. 
Fixed and pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=2e6cc43acec5
Attachment #807656 - Attachment is obsolete: true
Attachment #808503 - Flags: review+
Keywords: checkin-needed
Comment on attachment 808503 [details] [diff] [review]
patch, v4

Please land this patch before Bug 917270.
https://hg.mozilla.org/integration/fx-team/rev/7af33dde6db7
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/7af33dde6db7
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → mozilla27

Updated

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