-
Notifications
You must be signed in to change notification settings - Fork 7
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
Build failure on iOS #141
Comments
Hey @mipastgt, thanks for the report! You're right that this should work without any extra steps on iOS. There is something about that particular environment that is breaking the build. Is this an OSS project that I can try running myself? |
This is not an OSS project but I can try to create a little reproducer. |
Thanks! I'd be happy to dig in if you do. |
I tried to set up a simple example but I could not reproduce the error with that. So I send you an example with a slightly more complex project setup which I derived from my original project and there the error can be reproduced. I hope this helps. The other code in there should not be relevant for this problem because the test only uses your lib. I assume the problem is related to the general project setup which is actually pretty standard. The only difference is that I have configured a jvmCommon source set, but this should also not be relevant because it is not used for the test in question. |
Great, thank you for this! I can't have a look at this right away but I'll block time next week. |
Unfortunately, I haven't been able to run the demo. Gradle is unable to sync due to:
However, I've briefly researched the error (disclaimer: I have zero experience with compose) and found this. Could you please give that option a try, and report back? I'm thinking that, out of the box, compose is incompatible with kotlinx-resources. The latter copies resource files which the former seems to be picking up. If I understand what that option above does, that should stop happening. And if it stops happening, I will look into establishing that dependency per the error message. I'd like to confirm if that's the issue first, though. Ideally, we would have a compose module in |
It appears compose already disables resources automatically for some other libraries that also handle resources. |
This is issue starts driving me nuts :-( I downloaded and tried the example code which I sent you but I could not reproduce the error anymore with that code. I then verified it in the original project where this example was derived from and the problem still exists. I then added this
to the So this indeed seems to be a conflict between Compose resource handling and your resource handling. |
I have to somehow revert my above statement. The whole issue does not seem to be really deterministic. Even without the above flag I can get the iOS tests to work if I do a Gradle clean and then run the iOS tests first. If I then run the same test a second time it fails with the above reported error message. I have attached the whole project again with the missing dependency removed so that you should now be able to build it. The current state is that if I execute the following steps:
|
Thanks for digging deeper into this @mipastgt, I appreciate it!
This probably means there are some left-overs (either by
This is my suspicion, considering how those other projects work similarly to this one.
But with the flag, after a gradle clean, it never fails, correct?
Appreciated, I'll have a crack at this. |
I am using your plugin/library in commonTest to read in some resource file text. This works nicely for desktop and Android but fails at build time for iOS. The message is:
I guess this should work without any extra steps on my side for iOS.
The text was updated successfully, but these errors were encountered: