Invalid representation for a dependency with spaces in its name

[bcylin]

platform :ios, '7.1'

pod 'NBUImagePicker'

with CocoaPods 0.34.1, it encounters:

[!] Invalid string representation for a specification: NBUKit/Library/RestKit Support (2.2.2). The string representation should include the name and optionally the version of the Pod.

https://github.com/CocoaPods/Specs/blob/master/Specs/NBUKit/2.2.2/NBUKit.podspec.json
https://github.com/CocoaPods/Core/blob/0.34.1/lib/cocoapods-core/specification.rb#L123

Will this be an issue due to the regular expression change in 0.34.0? or it's a problem with the NBUKit podspec?

[segiddins]

That pod spec has a space in the name of a subspec, which is believe is invalid.

[dlackty]

cc/ @rivera-ernesto

[dlackty]

@segiddins After a quick search for specs repo, I found there're also some other pods with space in name.

https://github.com/CocoaPods/Specs/blob/master/Specs/mopub-ios-sdk/2.3.0/mopub-ios-sdk.podspec.json#L81
https://github.com/CocoaPods/Specs/blob/master/Specs/PRAugmentedReality/1.0.4/PRAugmentedReality.podspec.json#L48
https://github.com/CocoaPods/Specs/blob/master/Specs/ShareKit/3.0.0/ShareKit.podspec.json#L348

And also NBUKit.

[rivera-ernesto]

Removing that space. But I wonder what has changed that broke this only now.

[rivera-ernesto]

Indeed, broken by 0.34.1.
Something that should be probably fixed as not everyone will be quick to update not only their pods but their dependencies.
Besides supporting spaces is always a plus.

---

Even worse pod install/update fail despite renaming the subspec.
Only solution is to delete or manually edit Podfile.lock as well.

[segiddins]

@fabiopelosin for now, do we just want to relax the regex?

[fabiopelosin]

👍

[colasbd]

+1

[benjamincombes]

👍

[kylef]

This has been fixed by 51bf848 and later on, we shouldn't allow spaces in sub-specs (tracked by #177).

[bcylin]

Thanks! 👍

[gunta]

@rivera-ernesto ↑