-
Notifications
You must be signed in to change notification settings - Fork 189
Possible issue with NoCodeSign.xcconfig #302
Comments
It does make sense to save the error message that you have for the history. What error message was before the #300 was applied and what error message do you have now? |
Prior to #300, we had the binary on
|
I confirm my patch fixes it. I’ll do another PR if you want. |
Can you post step by step instructions with all details: CMake version, Polly toolchain, Xcode version, log with sign instruction line, the code itself. |
Hello, I’m not sure what to do with the details since it’s code from work. It’s open-source though so I guess it should be okay. I’m adding @KhalilBellakrid and @pollastri-pierre to the discussion for this purpose. The failible job can be found here. |
There is no need to show the whole project, of course only minimal example would be nice to have.
Okay, at least some info :) It's Xcode 10 and we are failing while checking that C compiler is not broken. |
Yes, and I think it’s due to the empty entitlements. I only a few about iOS, so I wrote a |
I've added iOS tests to CI: b14ab7b CMake 3.13.2, CMake 3.13.2,
I've noticed that you've used this line:
|
Interesting, thanks for your feedback! |
Closing inactive discussion. |
Hey,
I made a PR lately to fix a weird signing issue with the
ios-nocodesign-*
toolchains. While it worked on pretty much all of our CI jobs, one failed when CMake tried to check whether the compiler was okay. I think the problem is due to one extra line in the NoCodeSign.xcconfig file, but I’m unsure and I don’t know whether others reproduce.The line I deleted on a local fork is:
leaving the file as:
I’m currently testing my patch before opening a new PR. What do you think?
The text was updated successfully, but these errors were encountered: