Skip to content

Conversation

@bvirlet
Copy link

@bvirlet bvirlet commented Feb 24, 2021

Two fixes:

  • PackageDescription 4 doesn't support the platforms parameter, so it's incompatible with the annotation // swift-tools-version:4.1. Since you have a fine [email protected] file, I have removed the platform parameters from Package.swift. I confirm that this fixes importing the package with SPM 5.3.
  • When importing the package, you can have duplicate symbol issues because variables are defined in the header file. I have declared them in the header file and moved the definition in the implementation file.

@mnorth719
Copy link

Thanks for implementing this @bvirlet! I am having a similar issue and confirmed that your fix is working for me.

@bvirlet
Copy link
Author

bvirlet commented Jan 4, 2022

Hi @iziz. Do you plan to merge this? I have updated my fork with your latest changes (thanks) but the first commit is still needed to prevent the duplicate symbol issues.
Thanks!

@bvirlet
Copy link
Author

bvirlet commented Jan 4, 2022

Reopened with the just the required commit #358

@bvirlet bvirlet closed this Jan 4, 2022
@bvirlet bvirlet deleted the fix-swiftpm branch January 4, 2022 10:28
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