Opened 2 years ago
Closed 2 years ago
#65306 closed defect (fixed)
watchman: multiple dependencies failing for 10.14 and earlier: invalid output constraint '=@ccc' in asm
Reported by: | mascguy (Christopher Nielsen) | Owned by: | mascguy (Christopher Nielsen) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.7.2 |
Keywords: | Cc: | danchr (Dan Villiom Podlaski Christiansen) | |
Port: | watchman fizz fbthrift fb303 wangle |
Description (last modified by mascguy (Christopher Nielsen))
Various dependencies of watchman
are all failing to build on 10.14 and earlier, due to asm
-related errors like the following:
error: invalid output constraint '=@ccc' in asm
Based on local testing, all can be fixed by blacklisting Clang versions prior to 11:
compiler.blacklist-append \ {clang < 1100}
The list of ports that need to be fixed are:
fizz
fbthrift
fb303
wangle
watchman
Change History (16)
comment:1 Changed 2 years ago by mascguy (Christopher Nielsen)
Summary: | fizz: @2022.06.06: builds failing for 10.14: → fizz: @2022.06.06: builds failing for 10.14: invalid output constraint '=@ccc' in asm |
---|
comment:2 follow-up: 3 Changed 2 years ago by danchr (Dan Villiom Podlaski Christiansen)
comment:3 Changed 2 years ago by mascguy (Christopher Nielsen)
Replying to danchr:
The correct fix for this to use a newer Clang from MacPorts, right? Is there a standard way of picking the version?
Since the port compiles fine with Clang 11 on 10.15, I'd suggest the following. Just tested in my 10.14 VM, and this resolves the issue:
compiler.blacklist-append \ {clang < 1100}
Also remember to add portgroup for compiler_blacklist_versions
.
On a related note, it looks like wangle
has the same issue. So we'll want to apply the same fix there too.
comment:4 Changed 2 years ago by mascguy (Christopher Nielsen)
Add another port to the list needing the same fix: fbthrift
.
FYI, I'm working through the dependency list, to get watchman
working on macOS releases earlier than 10.15. And the latter may also be on the list, I'll find out momentarily.
comment:5 Changed 2 years ago by mascguy (Christopher Nielsen)
Description: | modified (diff) |
---|---|
Port: | watchman fbthrift fb303 wangle added |
Summary: | fizz: @2022.06.06: builds failing for 10.14: invalid output constraint '=@ccc' in asm → watchman: multiple dependents failing for 10.14 and earlier: invalid output constraint '=@ccc' in asm |
comment:6 Changed 2 years ago by mascguy (Christopher Nielsen)
Summary: | watchman: multiple dependents failing for 10.14 and earlier: invalid output constraint '=@ccc' in asm → watchman: multiple dependencies failing for 10.14 and earlier: invalid output constraint '=@ccc' in asm |
---|
comment:7 Changed 2 years ago by mascguy (Christopher Nielsen)
Yep, so the bottom line is that watchman
and the four other deps mentioned, all need the same fix. (Essentially all of the Facebook-provided components.)
comment:8 Changed 2 years ago by Christopher Nielsen <mascguy@…>
comment:9 Changed 2 years ago by Christopher Nielsen <mascguy@…>
comment:10 Changed 2 years ago by Christopher Nielsen <mascguy@…>
comment:11 Changed 2 years ago by Christopher Nielsen <mascguy@…>
comment:12 Changed 2 years ago by Christopher Nielsen <mascguy@…>
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:13 Changed 2 years ago by mascguy (Christopher Nielsen)
Cc: | danchr added |
---|---|
Resolution: | fixed |
Status: | closed → reopened |
Ugh, apparently most of these aren't building with Clang 11 either. I'll bump the minimum to 12.
comment:14 Changed 2 years ago by mascguy (Christopher Nielsen)
Owner: | changed from danchr to mascguy |
---|---|
Status: | reopened → assigned |
comment:15 Changed 2 years ago by Christopher Nielsen <mascguy@…>
comment:16 Changed 2 years ago by mascguy (Christopher Nielsen)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
The correct fix for this to use a newer Clang from MacPorts, right? Is there a standard way of picking the version?