Closed
Bug 1387510
Opened 7 years ago
Closed 7 years ago
configure fails on openbsd since bug 1363655
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox55 unaffected, firefox56 fixed, firefox57 fixed)
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox55 | --- | unaffected |
firefox56 | --- | fixed |
firefox57 | --- | fixed |
People
(Reporter: gaston, Assigned: gaston)
References
Details
Attachments
(1 file)
(deleted),
patch
|
froydnj
:
review+
lizzard
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
Title says it all; but in beta and central, configure fails:
0:24.32 ERROR: Could not find the clang shared library in the path /usr/local/lib
0:24.32 returned by `llvm-config --libdir` (searched for files [u'libclang.so.1.0']).
0:24.32 Please check your system configuration.
openbsd is not linux, and libclang.so *will* have a varying .so version (right now it's 5.0), so whatever the python code added in https://hg.mozilla.org/integration/mozilla-inbound/rev/cc58f2b74c4f is doing, it should match libclang.so.*...
Assignee | ||
Updated•7 years ago
|
Blocks: 1363655
status-firefox55:
--- → unaffected
status-firefox56:
--- → affected
status-firefox57:
--- → affected
Assignee | ||
Comment 1•7 years ago
|
||
This sorta works (also needs to import glob..)
if host.kernel == 'Linux':
libclang_choices.append('libclang.so.1')
+ if host.os == 'OpenBSD':
+ libclang_choices = glob.glob(path + '/libclang.so.*.*')
+
# At least one of the choices must be found.
Comment 2•7 years ago
|
||
(In reply to Landry Breuil (:gaston) from comment #1)
> This sorta works (also needs to import glob..)
>
> if host.kernel == 'Linux':
> libclang_choices.append('libclang.so.1')
>
> + if host.os == 'OpenBSD':
> + libclang_choices = glob.glob(path + '/libclang.so.*.*')
> +
> # At least one of the choices must be found.
Assuming that this works to pass configure *and* build with stylo -- not sure if clang-sys is set up to handle OpenBSD's constantly varying .so versions -- r=me.
Assignee | ||
Comment 3•7 years ago
|
||
I'm not sure for clang-sys nor runtime, but at least with this configure passed and m-i built (with --enable-stylo in .mozconfig)
Assignee: nobody → landry
Attachment #8893949 -
Flags: review?(nfroyd)
Comment 4•7 years ago
|
||
Comment on attachment 8893949 [details] [diff] [review]
bug-1387510
Review of attachment 8893949 [details] [diff] [review]:
-----------------------------------------------------------------
If you say it builds, I'm happy. Thank you!
Attachment #8893949 -
Flags: review?(nfroyd) → review+
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Comment 5•7 years ago
|
||
I've hit this with self-built libclang on linux or macOS as well. It's just that distro-installed versions typically have a symlink to libclang.so, but if we search for libclang.so.1 we might as well search for .4 and .5 too.
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ab81d3d87d87
Use glob() when looking for libclang.so on OpenBSD. r=froydnj
Keywords: checkin-needed
Assignee | ||
Comment 7•7 years ago
|
||
(In reply to Ralph Giles (:rillian) | needinfo me from comment #5)
> I've hit this with self-built libclang on linux or macOS as well. It's just
> that distro-installed versions typically have a symlink to libclang.so, but
> if we search for libclang.so.1 we might as well search for .4 and .5 too.
that's a slightly different thing - hardcoding .4 or .5 *will* fail at some point on OpenBSD, as we keep bumping libclang major when it's updated and functions are removed/changed in it - hence globbing...
Comment 8•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Assignee | ||
Comment 9•7 years ago
|
||
Comment on attachment 8893949 [details] [diff] [review]
bug-1387510
Approval Request Comment
[Feature/Bug causing the regression]: 1363655
[User impact if declined]: FTBFS on OpenBSD
[Is this code covered by automated tests?]: yes, http://buildbot.rhaalovely.net/builders/
[Has the fix been verified in Nightly?]: Yes
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: NPOTB/Tier3
[String changes made/needed]: None
Attachment #8893949 -
Flags: approval-mozilla-beta?
Comment on attachment 8893949 [details] [diff] [review]
bug-1387510
Build fix, let's uplift for 56 beta 1.
Attachment #8893949 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 11•7 years ago
|
||
bugherder uplift |
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•