Skip to content
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

Fix CocoaPods installing failure #91

Merged
merged 2 commits into from
Apr 1, 2021
Merged

Fix CocoaPods installing failure #91

merged 2 commits into from
Apr 1, 2021

Conversation

muukii
Copy link
Collaborator

@muukii muukii commented Apr 1, 2021

To fix #90

@muukii muukii marked this pull request as ready for review April 1, 2021 16:02
@muukii muukii merged commit 0e502b9 into main Apr 1, 2021
@muukii muukii deleted the muukii/cocoapods branch April 1, 2021 16:03
@FedupTech
Copy link

FedupTech commented Apr 2, 2021

@muukii

Project pods
pod "Brightroom/Engine", "2.0.0-alpha.3"
pod "Brightroom/UI-Classic", "2.0.0-alpha.3"
pod "Brightroom/UI-Crop", "2.0.0-alpha.3"

Running on simulator os version from 12 to 14.4

By checking pr that code wasn't change to public var currentEdit: Edit on alpha.3

Error message :

Undefined symbols for architecture x86_64:
  "Brightroom.EditingStack.State.Loaded.currentEdit.setter : Brightroom.EditingStack.Edit", referenced from:
      key path setter for Brightroom.EditingStack.State.Loaded.currentEdit : Brightroom.EditingStack.Edit : Brightroom.EditingStack.State.Loaded in BlurryMaskingView.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

@muukii
Copy link
Collaborator Author

muukii commented Apr 2, 2021

@FedupTech
https://github.com/muukii/Brightroom/tree/2.0.0-alpha.4 includes this PR.
Would you try?

@FedupTech
Copy link

@FedupTech
https://github.com/muukii/Brightroom/tree/2.0.0-alpha.4 includes this PR.
Would you try?

Worked, thx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Couldn't run project after install pods
2 participants