-
Notifications
You must be signed in to change notification settings - Fork 220
[[ Bug 23234 ]] Fix iOS standalone build error when running from source #7580
base: develop-9.6
Are you sure you want to change the base?
[[ Bug 23234 ]] Fix iOS standalone build error when running from source #7580
Conversation
This patch fixes the error: Failed to open macho file at /../Xcodes/Xcode_12_5_0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang++ for reading: Too many levels of symbolic links It seems there is a chance g++ is not symlinked correctly, so use the full path to clang++ instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this only an issue when running from source?
In any case, please file a BZ issue so it has an issue number and update commit message / PR appropriately.
2e71675
to
9fdaa32
Compare
e3fe7ec
to
883a684
Compare
@runrevmark Done :) Does it need a release note, or is it considered "internal"? |
I think its esoteric enough that it doesn't need a release note - although how is g++ not symlinked correctly? Is it something about your setup? Something one of our internals setup scripts does? |
I do not think it is my setup, since @montegoulding experienced the same issue a couple of months ago with Xcode 12.4. It might be because of a change in recent Xcodes, since I see this error with Xcode 12.5, but not with Xcode 11.3 in a different Mac. My understanding is that somehow |
Okay so it sounds like it isn't necessarily a build from source issue then - isn't the modified code in code that runs in a built IDE too? |
The modified code is in the S/B, which does run in a built IDE too. It does not seem to happen with Xcode 12.1, otherwise people would have noticed. We have not released a build that supports Xcode 12.4 or Xcode 12.5 so far, so I am not entirely sure if this is a build from source issue or a general Xcode12.4+ issue. I think I will be able to tell if this is the case when #7579 is merged and we kick off a new build. So I am marking this PR as WIP until then |
Ah! Okay well if it is an xcode 12.4/12.5 issue then it doesn't need a release note as it won't have affected anyone :) |
@livecodepanos Is this definitely an issue we need to resolve for the new Xcode update? |
@runrevmark I have checked with a built version of 9.6.3 rc1 from staging, with Xcode 12.5, and this issue is not present. So it seems it happens only when building from source. |
This patch fixes the error:
It seems there is a chance
g++
is not symlinked correctly, so use the full path toclang++
instead.Closes https://quality.livecode.com/show_bug.cgi?id=23234