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

Added support as Swift Package #247

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

nobreak
Copy link

@nobreak nobreak commented Sep 21, 2020

I had issues with Carthage and the new xcode12. It looks like that Carthage currently no more is usable with Xcode12 and iOS simulator. When you build the frameworks with 'carthage update' or 'carthage build' there comes an error. A deeper look says "have the same architectures (arm64) and can't be in the same fat output file" So the iOS Simulator build is missing in the framework. This already is discussed as issue here Carthage/Carthage#3019 and here ashleymills/Reachability.swift#370

My workarround was just to move from carthage to Swift Packages. I found out that is really easy to provide an existing iOS framework as Swift Package. Just create a fork of it, put there a configured Package.swift file into the root folder, double click it in finder to open it in Xcode as a SWIFT PACKAGE PROJECT !!! - now just add/change the path to the source files in the Package.swift file. When you could build the project, all is fine and push it to your repo - thats it. For this works.

Maybe you want to add Swift Package support ?

Philipp Homann and others added 4 commits September 22, 2020 00:04
…iPads or MacCatalyst requires an anchor point for presentation
Define source view and source rect for the activity popover, case  .pad idioms
@easydev991
Copy link

Can anybody merge this, please?

@RuiAAPeres
Copy link
Collaborator

@OlegEremenko991 does #246 solves your problem?

@easydev991
Copy link

@OlegEremenko991 does #246 solves your problem?

Actually this PR is preferred, Package.swift is fixed in this branch (source location is invalid in #246)

@RuiAAPeres
Copy link
Collaborator

Can you re-open a new PR and only have changes related to the Package? I can see that there are changes to GalleryViewController.swift, which we can address somewhere else.

@RuiAAPeres
Copy link
Collaborator

cc @tc-sgupta

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

Successfully merging this pull request may close these issues.

3 participants