-
Notifications
You must be signed in to change notification settings - Fork 823
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
Platform settings not applied #247
Comments
@thecb4 |
the watchOS item is commented out in the above example. When I uncomment and fix the watchOS ", the base SDK still shows as macOS for the iOS target. |
Hi @thecb4 Also just a note, your
|
I installed via Homebrew. I am getting the following error when I run No "base" settings found
No "release config" settings found
No "$platform" settings found for each $platform. I figured it was because I wasn't using xcconfig files |
Strange, the latest homebrew install is working for me (and many others I assume)
|
/usr/local/bin/xcodegen may have found the culprit. I installed using Mint, not homebrew |
Oh, I just did some testing on this and it's highlighted a issue in the Mint installation. Xcodegen isn't reading the platform settings from the right place. |
awesome. I'll be on the lookout for the fix. |
Fixed in #248 and will be in the next release. For a temporary fix copy |
work around works perfectly. Thanks! |
Is this fix in XcodeGen, Mint, or both? I still run into it with the following:
I can use the work around above
but I'm trying to get this working in a CI environment and the install path for xcodegen may not work for me. I updated my setup script with a bit for a workaround for this issue. Still not excited about using the build path for the xcodegen package though:
|
FYI, I encountered this same issue today when installing mint + xcodegen. What version will have the fix? |
Just fixed this in XcodeGen 1.10.3 |
New to XcodeGen and trying to create a universal framework as a test. However, the below project.yml does not apply the correct base SDK or deployment targets by platform. Not sure what I'm missing.
The text was updated successfully, but these errors were encountered: