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

Warning on external build tool configuration #419

Closed
frankatstyleshoots opened this issue Jun 13, 2018 · 8 comments
Closed

Warning on external build tool configuration #419

frankatstyleshoots opened this issue Jun 13, 2018 · 8 comments

Comments

@frankatstyleshoots
Copy link

Hello,

I have a project which uses an external build system target. The main target references this external build target in the Target Dependencies section of its Build Phases.

At compile time I get the following warning:

warning: [R.swift] project.pbxproj is internally inconsistent.
 - XCConfigurationList ([some id string here]) is not used
Perhaps a merge conflict?

The warning references the id of the XCConfigurationList associated with the PBXLegacyTarget section of the external build system target in the project.pbxproj file. Is this an R.swift issue, or is there any way to circumvent this warning? I don't think there is anything broken in the project.pbxproj file.

I'm using R.swift 4.0.0 with Xcode 9.4 (9F1027a).

Thanks!

@mac-cain13
Copy link
Owner

Could be that we detect this as an issue while it's not a problem in this case. It would mean that the XCConfigurationList is declared in the Xcodeproj file, but is never referenced from anywhere. I'm not sure if that is the default thing that happens with external build system targets.

It would help a lot if you can recreate the setup in a sample project so we can investigate it easier. Would that be something you could do?

@frankatstyleshoots
Copy link
Author

I'll see if I can put something together and will update here when I do...

@frankatstyleshoots
Copy link
Author

frankatstyleshoots commented Jun 14, 2018

Here ya go!

R.swiftTest.zip

@tomlokhorst
Copy link
Collaborator

tomlokhorst commented Jun 14, 2018

Thank you for this example project!

It seems XcodeEdit, our library for parsing xcodeproj files, doesn't support LegacyTargets. I'll add that to the library and we'll update the R.swift dependency in the next R.swift release.

@frankatstyleshoots
Copy link
Author

Awesome! Thank you!

@mac-cain13
Copy link
Owner

We've merged the fix, this will be in the next release!

@ghost
Copy link

ghost commented Jun 22, 2018

Do you have an ETA for the next release? :)

@mac-cain13
Copy link
Owner

Just released R.swift 5.0.0.alpha.1 that contains this fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants