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

Modify Package.swift to use source target instead of binary #2011

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

Conversation

jsflax
Copy link

@jsflax jsflax commented Dec 1, 2024

This modifies the Package.swift file to use source instead of binary, fixing a number of issues users have had with adding the package to their projects. It also prevents users from having to use a stale binary.

Relevant PRs for the gitmodules:

dinhvh/ctemplate#3
dinhvh/tidy-html5#1
dinhvh/libetpan#451

If this PR is accepted, there are a number of documentation changes that would also be required, since adding to SPM would be simplified.

@haithngn
Copy link
Member

haithngn commented Dec 2, 2024

Hi @jsflax,

Thank you for the PR—great work!
It’s time to move away from CocoaPods, so your help is greatly appreciated.
I’m currently reviewing this and the related PRs as well.

I’ll get back to you soon!

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.

2 participants