Skip to content

Conversation

@mergesort
Copy link
Contributor

This fixes a bug where the Info.plist file from this repo was included in the compile sources phase, which causes a build failure with the new Xcode 9 build system.

This fixes a bug where the Info.plist file from this repo was included in the compile sources phase, which causes a build failure with the new Xcode 9 build system.
s.source = { :git => "https://github.com/seedco/StackViewController.git", :tag => s.version.to_s }
s.ios.deployment_target = '9.0'
s.source_files = 'StackViewController/**'
s.source_files = 'StackViewController/**/*.{h,swift}'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mergesort thank you for the PR!
Did you mean to have source files set to StackViewController/*.{h,swift} ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, my mistake. Lemme tweak that.

@mergesort
Copy link
Contributor Author

Updated with the right file path.

@vikmeup
Copy link
Contributor

vikmeup commented Aug 25, 2017

@mergesort 👍

@vikmeup vikmeup merged commit 5fba46b into seedco:master Aug 25, 2017
@mergesort
Copy link
Contributor Author

@vikmeup I just realized that I didn't bump the version number. Would it make sense to give it a version number of 0.3.1 in another patch? I don't have CocoaPods push access, so it would make sense to come from your side if you do choose to do that.

Thanks!

@vikmeup
Copy link
Contributor

vikmeup commented Aug 28, 2017

@mergesort I had problems running pod trunk push, I can take a look little later to push cocoa pods update, at the main time can you point to master? thanks!

@vikmeup
Copy link
Contributor

vikmeup commented Aug 28, 2017

@mergesort done! version 0.3.1 successfully published.

@mergesort
Copy link
Contributor Author

Thanks so much!

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