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

Command fails if Xcode not installed into default path #6

Open
RicardoKoch opened this issue May 16, 2019 · 2 comments · May be fixed by #9
Open

Command fails if Xcode not installed into default path #6

RicardoKoch opened this issue May 16, 2019 · 2 comments · May be fixed by #9

Comments

@RicardoKoch
Copy link

ALTOOL= File.expand_path('/Applications/Xcode.app/Contents/Applications/Application\ Loader.app/Contents/Frameworks/ITunesSoftwareService.framework/Support/altool')

Symptom:
Command will fail if Xcode is not installed into default path.

Description:
Xcode path should ideally not be hardcoded to /Applications/Xcode.app.
Use the following command to read the active build tools path:
xcode-select -p

@mgrebenets
Copy link

+1. Some CI configurations have multiple Xcode apps installed and actually don't have /Applications/Xcode.app.
I'd say the plugin should have an optional parameter for passing in Xcode path.
So the caller can specify exact Xcode path, and if not specified then it should be autodetected using xcode-select -p.

@limikael
Copy link

It would be really nice to see that PR merged! In the meantime, here is hack as workaround. I run it in my install script before running fastlane:

ALTOOL='/Applications/Xcode.app/Contents/Applications/Application Loader.app/Contents/Frameworks/ITunesSoftwareService.framework/Support/altool'
if [ ! -f "$ALTOOL" ]; then
	ALDIR=`dirname "$ALTOOL"`
	mkdir -p "$ALDIR"

	printf '#!/usr/bin/env bash\nxcrun altool $@\n' > "$ALTOOL"
	chmod 755 "$ALTOOL"
fi

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 a pull request may close this issue.

3 participants