1500484 - widget/gtk/nsClipboardWayland.cpp:32: fatal error: sys/epoll.h: No such file or directory

Status: RESOLVED FIXED

(Core :: Widget: Gtk, defect)

Description

[Jan Beich]

$ c++ -v
FreeBSD clang version 7.0.0 (tags/RELEASE_700/final 342383) (based on LLVM 7.0.0)
Target: x86_64-unknown-freebsd12.0
Thread model: posix
InstalledDir: /usr/bin

$ ./mach bootstrap
$ echo "ac_add_options --enable-default-toolkit=cairo-gtk3-wayland" >>.mozconfig
$ ./mach build
[...]
In file included from obj-x86_64-unknown-freebsd12.0/widget/gtk/Unified_cpp_widget_gtk1.cpp:74:
In file included from widget/gtk/nsClipboardWayland.cpp:32:
obj-x86_64-unknown-freebsd12.0/dist/system_wrappers/sys/epoll.h:3:15: fatal error:
      'sys/epoll.h' file not found
#include_next <sys/epoll.h>
              ^~~~~~~~~~~~~
1 error generated.

Comment 1

[Jan Beich]

Attachment: v1

https://treeherder.mozilla.org/#/jobs?repo=try&revision=540a91b9580354f23a76acd72f4ac3e7d6c575f7

Comment 2

[Jan Beich]

Attachment: v1 (esr60)

Rebased in case of backport. Only context lines changed.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=d884084eac9239b0e78266bd8b285f4d53f6cb7d

Comment 3

[Martin Stránský [:stransky] (ni? me) (PTO, back on Feb 17)]

Comment on attachment 9018657 [details] [diff] [review]
v1

Looks good, Thanks.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

I'd be fine uplifting this to Fx64/ESR60.4, but it's too late for 63 at this point. Feel free to nominate this for Beta/ESR60 approval after it lands on m-c.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/422dcc2a11ca
Unbreak Wayland build on FreeBSD after bug 1417890. r=stransky

Comment 6

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/422dcc2a11ca

Comment 7

[Jan Beich]

Comment on attachment 9018663 [details] [diff] [review]
v1 (esr60)

[ESR Uplift Approval Request]

If this is not a sec:{high,crit} bug, please state case for ESR consideration: Facilitate regression tracking against patched downstream builds. Also help building Tor Browser which is based on ESR.

User impact if declined: Broken build with Wayland support enabled on non-Linux.

Fix Landed on Version: 64

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): Can only break build on Linux. However, Wayland isn't enabled on automation yet.

String or UUID changes made by this patch: 

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Bug 1417890

User impact if declined: Broken build with Wayland support enabled on non-Linux.

Is this code covered by automated tests?: Yes

Has the fix been verified in Nightly?: Yes

Needs manual test from QE?: No

If yes, steps to reproduce: 

List of other uplifts needed: None

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): Can only break build on Linux. However, Wayland isn't enabled on automation yet.

String changes made/needed:

Comment 8

[Jan Beich]

Comment on attachment 9018663 [details] [diff] [review]
v1 (esr60)

Oops, wrong attachment and wrong flag.

Comment 9

[Jan Beich]

Comment on attachment 9018657 [details] [diff] [review]
v1

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Bug 1417890

User impact if declined: Broken build with Wayland support enabled on non-Linux.

Is this code covered by automated tests?: Yes

Has the fix been verified in Nightly?: Yes

Needs manual test from QE?: No

If yes, steps to reproduce: 

List of other uplifts needed: None

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): Can only break build on Linux. However, Wayland isn't enabled on automation yet.

String changes made/needed:

Comment 10

[Jan Beich]

(In reply to Sebastian Hengst [:aryx] (needinfo on intermittent or backout) from comment #6)
> https://hg.mozilla.org/mozilla-central/rev/422dcc2a11ca
> Target Milestone: --- → mozilla65

hg.mozilla.org shows milestone 64.0a1 but https://hg.mozilla.org/releases/mozilla-beta/file/tip/widget/gtk/nsClipboardWayland.cpp still has <sys/epoll.h> which confirms it landed in Firefox 65. Kinda confusing.

Comment 11

[Ryan VanderMeulen [:RyanVM]]

(In reply to Jan Beich from comment #10)
> hg.mozilla.org shows milestone 64.0a1 but
> https://hg.mozilla.org/releases/mozilla-beta/file/tip/widget/gtk/
> nsClipboardWayland.cpp still has <sys/epoll.h> which confirms it landed in
> Firefox 65. Kinda confusing.

That's because we bump the version on m-c first, but most patches land on integration branches first, so the version can be a bit odd depending on when they first saw the version bump commit.

Comment 12

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9018657 [details] [diff] [review]
v1

NPOTB bustage fix for downstream distros using Wayland. Approved for 64.0b4.

Comment 13

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9018663 [details] [diff] [review]
v1 (esr60)

Approved for the next ESR60-based release too (either 60.3.1 or 60.4.0, depending on whatever we end up building next).

Comment 14

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/1565c0905a5d

Comment 15

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-esr60/rev/6f1a76d107dc