-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
get CocoaPods back on the rails #860
Comments
Per discussion with @ccgus we're going to go with 2.7.9 and beyond on CocoaPods. He made the tag and I'm going to push. Currently I'm getting linter errors despite the Pod working in Mac/iOS sample projects, so I'll keep at that. linter output
|
My hunch here is that there are problems in the project that linting exposes. At first I thought maybe they were only Full verbose linter log (~12k lines): https://gist.github.com/incanus/9c957f803931534d6207499de1635f45 Pulling out the relevant lines, the problems seem to be related to compiling the e.g.
and
both fail. |
I've at least now figured out that only the |
Confirmed that Looking a bit closer at these subspecs, both use the And the e.g.
So the next step would be to try building against a local checkout of the upstream sqlite3 pod with these settings adjusted. If that works, maybe we can convince upstream to bump up the deployment target. |
It looks like we are blocked on pushing a release to CocoaPods until the upstream clemensg/sqlite3pod#22 is solved, potentially with the pull request clemensg/sqlite3pod#23. This will fix the deployment targets issue for the I looked into testing this theory by cloning the upstream
So I can't actually successfully lint the FMDB Podspec if it includes the |
I should add that with this as with any use of FMDB via CocoaPods, you could always clone this repo locally and point to it with:
Of course, the @ccgus Should we consider removing those subspecs for now so that we can publish until the upstream issue is solved? |
Yea, let's comment out |
Ok, working off of @ccgus' change in 84d81a5 gets things to lint... on iOS and Mac. WatchOS Easiest way to verify is:
|
The problem would be that I didn't have a WatchOS simulator installed. Discovered via CocoaPods/CocoaPods#11558. I'm pleased to report: 🎺🎺🎺
@ccgus I'll leave it to you as to whether to close this or not, given the standalone subspec issues. |
Woo! |
Thanks so much @incanus ! |
Is SQLCipher coming back online anytime soon? I was just trying to update my pods and I got the following error:
I use FMDB with SQLCipher from CocoaPods. Thanks. |
@brendand none of the maintainers use SQLCipher, so we rely on folks like you to submit a patch for it. |
Is there more to it than just uncommenting the SQLCipher section of the Podspec? I've never worked on any Cocoapods things before. I just use them when available and I've been using FMDB / SQLCipher for years. @incanus said above that it was only the |
Good point. I don’t recall OTOH, but will try to get some time to build again and see if that’s the case, if you don’t first.
|
The SQLCipher subpec does indeed seem to |
Excellent. Hopefully the SQLCipher item can be uncommented and I can update my pods again. Thanks @incanus |
I'll uncomment it. I took it out to make getting the CocoaPods stuff working and the plan was to bring it back if folks complained :) |
OK, it's back up. @incanus can you do the magic cocoapods stuff when you get a chance? (Or … do I need to release a new version?) |
I think we’ll need a new tag to do things right, so that we don’t "change what 2.7.9 is" mid-stream.
|
Sorry, just getting a chance to look at this. @ccgus we need the version updated to 2.7.10 in the podspec itself — this should typically come before the Git tag. I didn't notice until I went to push — things linted fine but then I got a duplicate version error. |
I've added the tag and pushed it up. I can do another release (2.7.11!) if needed. |
Per #432 (comment) we have updated the podspec version as well as enabled tvOS support, then moved the tag since it only affected CocoaPods stuff and nothing code-wise. Version 2.7.10 is pushed! |
I'm taking a look at pushing the latest stable release, 2.7.8, to CocoaPods per @ccgus, who has made me a maintainer there. After success with that, I will push new versions going forward, too.
Two things are needed for this to happen:
[ ] A tag of2.7.8
needs to be created.A tag forv2.7.8
already exists, but past versions since2.7.1
have used a tag name without thev
and the release's Podspec would benefit from having this tag naming pattern without requiring other changes. Optionally, the oldv2.7.8
(andv2.7
) tag could be deleted for tidiness and to avoid confusion in the future, but it's not required, and possibly could break downstream users.This is needed since the Podspec refers to theversion
to know which:tag
to check out.I have tested this tag creation in a fork and it resolves the issue if that repo is used in the Podspec instead of this one.I will then
pod trunk push FMDB.podspec
from2.7.82.7.9 per the guide.Obsoletes #853 & #857.
The text was updated successfully, but these errors were encountered: