Closed
Bug 875518
Opened 11 years ago
Closed 11 years ago
Fix ANGLE compilation on NetBSD and DragonFly
Categories
(Core :: Graphics: CanvasWebGL, defect)
Tracking
()
RESOLVED
FIXED
mozilla33
People
(Reporter: ftigeot, Unassigned)
References
Details
Attachments
(2 files)
377 bytes,
patch
|
bjacob
:
review+
|
Details | Diff | Splinter Review |
808 bytes,
patch
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; DragonFly x86_64; rv:20.0) Gecko/20100101 SeaMonkey/2.17 (Beta/Release) Build ID: 20130513175155 Steps to reproduce: I tried to build various mozilla products on DragonFly. Actual results: Compilation failed with many of these errors: ./src/compiler/osinclude.h:22:2: error: #error Unsupported platform. Expected results: Builds should have succeeded.
Reporter | ||
Comment 1•11 years ago
|
||
Reporter | ||
Comment 2•11 years ago
|
||
NetBSD suffers from the same issue, the attached patch fix ANGLE compilation for both NetBSD and DragonFly.
Reporter | ||
Updated•11 years ago
|
Attachment #753502 -
Flags: review?(bjacob)
Updated•11 years ago
|
Attachment #753502 -
Flags: review?(bjacob) → review+
Comment 4•11 years ago
|
||
Wait, this patch is against a .orig file, you want to fix that ;-) Also, this needs a combination of 1) upstreaming as mentioned in comment 3, and 2) having a local gfx/angle/*.patch recorded and mentioned in gfx/angle/README.mozilla.
Reporter | ||
Comment 5•11 years ago
|
||
Reporter | ||
Comment 6•11 years ago
|
||
I initially added a wrong patch file, sorry about that :-/ For the rest, I thought Mozilla was the upstream since this code is present in the mozilla-central repository and many of the various tarballs your organization distributes. If Mozilla is not the real upstream and doesn't maintain this code, there is no point in trying to integrate patches here, especially if they will stay as patches. I'll go bother the ANGLE developers, whoever they are ;-)
Comment 7•11 years ago
|
||
Mozilla is not the upstream; the upstream is https://code.google.com/p/angleproject/ . The reason why this is in Mozilla-central is that we need to use very specific versions of ANGLE with a few specific patches. Likewise, WebKit (or Chromium, not sure) has its own copy of ANGLE. So in the long term, all what matters is that this be upstream in ANGLE, but if you want this fixed immediately in Firefox, without waiting for the next time that we sync with upstream, then you may still want this checked into mozilla-central.
Reporter | ||
Comment 8•11 years ago
|
||
Firefox is already patched locally in the dports and pkgsrc packaging systems; adding local patches to the Mozilla repositories would only increase your maintenance burden. I have opened a new issue on https://code.google.com/p/angleproject/ with the same fix. Marking as resolved invalid since I reported this particular problem to the wrong organization. Sorry for the time wasted.
Status: UNCONFIRMED → RESOLVED
Closed: 11 years ago
Resolution: --- → INVALID
Comment 9•11 years ago
|
||
(In reply to François Tigeot from comment #8) > Firefox is already patched locally in the dports and pkgsrc packaging > systems; adding local patches to the Mozilla repositories would only > increase your maintenance burden. It would not, because these local patches are already applied in the tree. But I'm not complaining at all if you're OK with just upstreaming this. Thanks!
Comment 10•10 years ago
|
||
Recent ANGLE update brought the fix from upstream.
Comment 11•10 years ago
|
||
Awesome. :)
You need to log in
before you can comment on or make changes to this bug.
Description
•