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

Version 5.0 #189

Merged
merged 8 commits into from
Oct 26, 2020
Merged

Version 5.0 #189

merged 8 commits into from
Oct 26, 2020

Conversation

alexisakers
Copy link
Owner

@alexisakers alexisakers commented Sep 25, 2020

In this version, we add support for the Swift Package Manager. To make these changes possible, we decided to make the following breaking changes:

  • Require iOS 11
  • Remove the possibility to create items in Objective-C

We are planning to release this version on October 25, 2020. During the next month, please reply to this PR with feedback about any bugs or request.

To use the release candidate version, please use the following tag in your package manager: 5.0.0-rc.2

kenzinn and others added 7 commits September 25, 2020 10:10
* Fix the background view origin when presenting

When presenting over a ViewController (the “from” VC), if that ViewController is not at the screen’s origin, the background view will not appear at the correct position above it.

This fixes the problem by setting the background view’s frame to the “from” VC’s view’s frame **converted into the coordinate system of the container view.**

* add PR to CHANGELOG
* Update project to support Swift Package Manager

* Update CHANGELOG

* Only create one library

* Reorganize Swift project to better support SPM

* Update README

* Clean up files

* Update BLTNItem.swift
As outlined by version 5 manifest.
@rodmaz
Copy link

rodmaz commented Sep 26, 2020

How come is this even tagged as release candidate?
The Swift Package does not even compile! BTW, my project has target iOS 13.0.

Screen Shot 2020-09-26 at 15 46 23

@alexisakers
Copy link
Owner Author

How come is this even tagged as release candidate?
The Swift Package does not even compile! BTW, my project has target iOS 13.0.

Screen Shot 2020-09-26 at 15 46 23

Thanks for your constructive feedback! As you can see, @ollitapa already fixed it in develop, I’m just gonna wait a couple more days before tagging a second version, in case more things come up.

@eminisrafil
Copy link

Thanks for maintaining this repo!

@aaronbrethorst
Copy link
Contributor

Hey Alex - Thanks for getting SPM working! I'd love to be able to move this dependency off Carthage and onto SPM. Any chance you could tag RC 2 today?

@alexisakers
Copy link
Owner Author

@aaronbrethorst Done!

@aaronbrethorst
Copy link
Contributor

Thank you!

@alexisakers alexisakers merged commit 37e13c4 into master Oct 26, 2020
alexisakers added a commit that referenced this pull request Oct 26, 2020
* Fix the background view origin when presenting (#183)

* Fix the background view origin when presenting

When presenting over a ViewController (the “from” VC), if that ViewController is not at the screen’s origin, the background view will not appear at the correct position above it.

This fixes the problem by setting the background view’s frame to the “from” VC’s view’s frame **converted into the coordinate system of the container view.**

* add PR to CHANGELOG

* Add support for Swift Package Manager (#158) (#178)

* Update project to support Swift Package Manager

* Update CHANGELOG

* Only create one library

* Reorganize Swift project to better support SPM

* Update README

* Clean up files

* Update BLTNItem.swift

* Rename master to main

* Update Podscpec

* Require iOS 11

* Update CHANGELOG

* Update podspec

* SPM also to require iOS 11 (#190)

As outlined by version 5 manifest.

Co-authored-by: Ken Zinn <ken.zinn@nonzerosolutions.com>
Co-authored-by: Olli Tapaninen <ollitapa@gmail.com>

Co-authored-by: Ken Zinn <ken.zinn@nonzerosolutions.com>
Co-authored-by: Olli Tapaninen <ollitapa@gmail.com>
@rodmaz
Copy link

rodmaz commented Nov 3, 2020

Has anyone tested this? It does not seem like it works properly (at least not on Xcode 12.1). Package does not compile.
Tested from main branch and version 5.0.0.

error: Multiple commands produce '/MyCode/MyProject-ezhoowqbocgzhycfoojnfdfrulyf/Build/Intermediates.noindex/BLTNBoard.build/Debug-iphoneos/BLTNBoard.build/Objects-normal/arm64/BLTNBoard.swiftmodule':
1) Target 'BLTNBoard' (project 'BLTNBoard') has compile command for Swift source files
2) Target 'BLTNBoard' (project 'BLTNBoard') has compile command for Swift source files

@ollitapa
Copy link
Contributor

ollitapa commented Nov 3, 2020

I've been using the package and it works fine for me. I think the error you are seeing is because you have included the SPM module in two or more Xcode style ".framework" modules. This is a "known" limitation in Xcode and to work around it, you need to make an umbrella ".framework" for your common SPM modules and import that in both of your own modules.

@rodmaz
Copy link

rodmaz commented Nov 3, 2020

@ollitapa It is in fact a framework but all other SPM packages in the framework just work perfectly.

Screen Shot 2020-11-03 at 09 42 10

@ollitapa
Copy link
Contributor

ollitapa commented Nov 5, 2020

Hmm, maybe you have some Carthage/CocoaPods leftovers? Some copy-frameworks step? At least my umbrella framework builds fine.
Screenshot 2020-11-05 at 11 54 12

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.

None yet

6 participants