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

Missing attributes #314

Open
ghost opened this issue Oct 9, 2015 · 2 comments
Open

Missing attributes #314

ghost opened this issue Oct 9, 2015 · 2 comments

Comments

@ghost
Copy link

ghost commented Oct 9, 2015

Versions used:
XCode version: 7.0.1
xcodeproj version: git master

I'm trying to use cocoapods to add some dependencies to our project, only I'm running
into some problems with this.
I'm using Qt to create the initial xcode project.

(1) this looks like a bug and easy to solve:

Some of the build phases miss a name attribute

PBXResourcesBuildPhase
PBXFrameworksBuildPhase
PBXSourcesBuildPhase

some other buildphases do have a name attribute (for example PBXCopyFilesBuildPhase)

Adding a new buildphase from xcode also sets the name attribute (this is the visible name in xcode
under the build phase tab). So It should be supported in order to use a xcode generated project.

(2) would be easy to solve, but I'm not sure the attribute should be there..
An other problem I'm runnin into is missing the refType attribute in PBXFileReference.
This one is also ignored/removed by xcode so maybe it's old?.. (I can't seem to find anything about this)

(3) Definitely not a bug, but maybe an improvement for later...
But this leads to the following question:
Why the choice to bail out when attributes exist that you don't know of? Would it not be more
future proof to ignore the fact that you don't know about them (or just warn the user) and leave the original
value intact?
(this way when something changes things don't break immediatly, but it's still noticed)

@segiddins
Copy link
Member

Could you provide an example project that demonstrates the issues you're seeing?

@mgerhardy
Copy link

I also stumbled upon the refType thing.

https://github.com/mgerhardy/caveexpress commit 2fd95be690ab5cac6c2b1be4edbaea34e85354ff

after cloning just run ./contrib/scripts/ios.sh

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

No branches or pull requests

2 participants