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

updated Bridging header to be compatible with SQLCipher 4.0.0 pod #864

Closed

Conversation

asidden
Copy link

@asidden asidden commented Dec 6, 2018

until this change, I was not able to install SQLite.swift with Xcode 10.1 via

pod 'SQLite.swift/SQLCipher'

SQLCipher dependency pod version resolved - 4.0.0

the build failed with error:

error: include of non-modular header inside framework module 'SQLite.SQLite_Bridging': '/Pods/SQLCipher/sqlite3.h'
#import "sqlite3.h"

however, I'm not sure this change does not affect Carthage or SPM installation.

@jonathanrfisher1
Copy link

This resolved my Xcode 10 issue with the same error. Looks like it's breaking CI though: module 'SQLCipher' not found

Do you know a fix for this @asidden ?

For now I plan on using your change with a git patch.

@asidden
Copy link
Author

asidden commented Dec 8, 2018

travis is building using Xcode 9, but unfortunately I don't have one to check with it.
may be a conditional macro should be involved.

@jberkel jberkel mentioned this pull request Dec 19, 2018
@jberkel
Copy link
Collaborator

jberkel commented Dec 19, 2018

This should work now in master (cf4183f)

@jberkel jberkel closed this Dec 19, 2018
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