-
Notifications
You must be signed in to change notification settings - Fork 92
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
Updated checksum to match binary artifact #275
Conversation
Anytime you guys are ready ! :D this is blocking |
Please consider this PR as soon as possible. We also have this problem on CI/CD. One of the packages we use has a dependency on AppsFlyerLib. Clearing caches doesn't help. |
@af-obodovskyi Would you please merge this PR? Thank you! |
@amit-kremer93 @af-obodovskyi could you look into this with urgency? This is breaking many organisation's CIs. |
is there any good reason why this isnt merged in yet? this is blocking... |
Thank you for prepping this PR! I could benefit from it as well 🙏 |
Please merge this. I need this! |
@richroguetech , @hbanzon , @srdanrasic , @JohnnyTseng @scottlemke Please consider using one of the following repositories for the SPM. SPM support will be removed from the main repository because of the extremely long checkout time, which affects the SPM package integration time. For statically linked library For dynamically linked library For Strict (No IDFA colection) library |
@af-obodovskyi what release processes are you talking about? It's just links to binaries and zero source code :D With the same result you can publish separate version tags in separate branches |
@ftp27 not the release process, SPM package integration time. I will edit my comment |
@af-obodovskyi can someone explain this? I used your repos and i got this: . The /Frameworks/AppsFlyerLib.framework' specifies a MinimumOSVersion of '100.0' but contains a 32-bit architecture that is unsupported on iOS 12 and later. Apple Developer Relations |
@af-obodovskyi I get the following error using the Static library. Seems liek there's some changes that will be required on our end if we switch repos.
|
Can we get more of a ramp up time to adapt to the new repos but in the meantime merge any of the PRs that fix this issue? |
@hbanzon I assume you're using Tuist too? If so, you need to reference the static of dynamic library name to fix that issue. Something like: |
Thank you @scottlemke! |
No description provided.