Skip to content

Conversation

@skagedal
Copy link
Contributor

Closes issue #925 – not an SQLite.swift issue, but an SwiftPM one, but worthy of a note here imo.

@danramteke
Copy link

danramteke commented May 13, 2019

I would prefer that we make the change that Ankit recommends here https://bugs.swift.org/browse/SR-8629

Edit: The change recommended in the link is: "the generated project starts working as expected if you rename SQLite-Bridging.h to SQLiteObjc.h."

@skagedal
Copy link
Contributor Author

@danramteke Excellent, that is of course a much better solution if possible.

@ypopovych
Copy link
Collaborator

@danramteke can you provide PR for this fix?

@skagedal
Copy link
Contributor Author

@ypopovych Here is a PR – just an updated version of what @danramteke provided here

@skagedal
Copy link
Contributor Author

Confirmed that it works by just doing what I did here but instead pointing SwiftPM to my branch.

@ypopovych
Copy link
Collaborator

merged PR

@ypopovych ypopovych closed this May 31, 2019
@skagedal skagedal deleted the spm-readme-modules branch May 31, 2019 18:08
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.

4 participants