-
Notifications
You must be signed in to change notification settings - Fork 133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bitcode enabled framework not accepted by Appstore #537
Comments
@legion151
Archived and was able to rebuild from bitcode. Could you please try with simple case ? As there might other points of failure in your project. PS: It was required to fix a bit
added
|
I now tried with
The first weird thing is, that by normal adding our framework, XCode complains that it lacks bitcode (which it doesn't). However, recompiling it fails for me, also when i use your mentioned patch.
I will now try with ObjC. |
ObjC is exactly the same. I now try to build a completey new framework. |
So i played around a little. I have to admit that I am not really sure how to build a template robovm framework via mvn compiler plugin. Since the newest sample for this is from 2015. Is it possible, that the maven stack can introduce the problems? |
@legion151 create a new framework project using Idea -- and then build it with maven ? |
Hi and sry for the delay. Today i had some time to look this again. I now created new Framework via your description. Using gradle i must have set that wrong because XCode tells me the framework doesn't contain any bitcode. Using maven i got a framework which i could pack into an app and run it on device with bitcode enabled. i tried this with several versions of XCode with same results and wasn't able to find a way to fix ipatool.
which comes from ipatool line 373. May i ask you, which versions you used to get a recompiling done? I have:
|
@legion151 Hi,
|
Followed these steps again. No success for me. Anyway since not reproducable i will close this issue. You could do me a favour if u've got the time. And in the end - bitcode seems not to be that critical as long as it is not enforced by apple. Thx so far. |
@legion151
Xcode script fail to validate due to lot of empty parameters presents (in general the list of parameters looks as repeatable mess):
Also have compared your binary with my. and yours contains too little of bitcode file section, like 48 vs 1103. In general it would be great if you share a source code of dummy framework and command line parameters used to build it, just to check if I get result similar to yours. |
First things first: Thank you very much for looking at this. Your findings show, that my machine allready fails at gettin the framework correctly build. I assume the otool command you gave lacks a space between LLVM and bundle. I didn't really change a thing in the framework template which comes with robovm intelliJ plugin, except for enabling bitcode in config-files. The source can be found here I there will also push a logfile of intelliJ building the framework |
@legion151 hi,
It seems that shared (and one with defects) probably was built using maven tool. Would be great to have step you were following to build it. |
Hi there. I setup a new framework project for the git repo which i linked. |
@legion151 can you please share new binary ? thx |
thanks, same thing there, empty |
ok, reproduced on different Mac. will check what is a difference between these two |
What do you mean by "command line". |
@legion151 when you mentioned that you were using gradle for building -- I thought that you were building from command line using gradlew. But if you just created gradle kind of project and build with RoboVm-Create Framework menu -- then it doesn't matter. |
I couldn't find new information, yet. |
@legion151 in case of BC it can be solved. Will create a PR tomorrow. But probably we can target it for testing to v2.3.13 |
Awesome! \o/ Once again thanks for your great work. Thx |
Since this is closed but open (tested today with current main-branch, which still shows this), i would like to bump. Don't know when the next release is planned though. |
Hi @legion151 @dkimitsa any updates?
Do you know how it can be fixed? Using RoboVM version 2.3.13 with enabled bitcode. |
Hi @novacom34, unfortunately not. Only switching off bitcode works for me. @dkimitsa assumed this to be an issue with missing libraries on other machines than his. I don't really know the state of this problem. The hope was that v2.3.13 would fix it. |
@legion151 there is possible workaround for case we discussed, will check it later this week |
sounds great. |
@dkimitsa Any updates? |
Just tested with current master 887151b and still got the ipatool failed error as described above. |
I discovered that this issue is caused by the single object prelink you're doing.
I'm not sure how to resolve this though, is the single object prelink something you want to keep? |
@timower prelink causes issue as mentioned few comments above. Anyway bitcode is not required anymore since XCode 14. |
I'm using Xcode 13.4 |
pardon, issue is here, also there is a discussion somewhere at llvm without any solution to change behaviour. meanwhile do you still need bitcode stuff considering recent apple changes ? |
Please ensure you have given all the following requested information in your report.
Issue details
Bitcode support was added in
#443
which works so far.
At least for framework target. If framework is build with enabled Bitcode one can also build an app with with bitcode.
However the very app is then rejected by apples store during upload:
Also recompiling the app with "Rebuild from Bitcode" option set, fails, resulting in:
Reproduction steps/code
One might use a clone of
https://github.com/ecsec/open-ecard-ios
which should allow to reproduce the steps mentioned above.
Versions:
Build Targets:
The text was updated successfully, but these errors were encountered: