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.

Sign up for GitHub

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

Jump to bottom

Add verbose option and more informative errors #20

Closed
4brunu opened this issue Oct 3, 2018 · 1 comment · Fixed by #22
Closed

Add verbose option and more informative errors #20

4brunu opened this issue Oct 3, 2018 · 1 comment · Fixed by #22
Labels
enhancement New feature or request

Comments

@4brunu
Copy link
Contributor

4brunu commented Oct 3, 2018

Sometimes Bridgecraft stop's with some short and generic errors.
When this happen, I need to open the Bridgecraft project, run it with some extra prints, and then run the commands manually in the terminal to get more informative errors.

Here is an example of an error I'm getting now.
When I run Bridgecraft from the command line, this is the output.

** BUILD FAILED **


The following build commands failed:
	Check dependencies
(1 failure)
cannot dry-run build for XcodeProject.xcodeproj
make: *** [generate-source] Error 2

I have no idea where is the problem.

But when I run the same command that Bridgecraft run's on the terminal /usr/bin/xcodebuild clean build -dry-run -project XcodeProject.xcodeproj -target XcodeProjectTarget -UseModernBuildSystem=0 -sdk iphonesimulator -destination platform="iOS Simulator,name=iPhone X,OS=latest", this is the output.

User defaults from command line:
    UseModernBuildSystem = 0

Build settings from command line:
    SDKROOT = iphonesimulator12.0

Prepare build
note: Using legacy build system

=== CLEAN TARGET XcodeProjectTarget OF PROJECT XcodeProjectTarget WITH THE DEFAULT CONFIGURATION (Debug) ===

Check dependencies
[BEROR]No architectures to compile for (ONLY_ACTIVE_ARCH=YES, active arch=i386, VALID_ARCHS=arm64 armv7s).

Create product structure
/bin/mkdir -p /Users/User/Developer/XcodeProject/build/Debug-iphonesimulator/XcodeProjectTarget.app

Clean.Remove clean build/XcodeProjectTarget.build/Debug-iphonesimulator/XcodeProjectTarget.build
    builtin-rm -rf /Users/User/Developer/XcodeProject/build/XcodeProjectTarget.build/Debug-iphonesimulator/XcodeProjectTarget.build

Clean.Remove clean build/Debug-iphonesimulator/XcodeProjectTarget.app
    builtin-rm -rf /Users/User/Developer/XcodeProject/build/Debug-iphonesimulator/XcodeProjectTarget.app

** CLEAN SUCCEEDED **

Prepare build
note: Using legacy build system

=== BUILD TARGET XcodeProjectTarget OF PROJECT XcodeProjectTarget WITH THE DEFAULT CONFIGURATION (Debug) ===

Check dependencies
No architectures to compile for (ONLY_ACTIVE_ARCH=YES, active arch=i386, VALID_ARCHS=arm64 armv7s).

** BUILD FAILED **


The following build commands failed:
	Check dependencies
(1 failure)

Now I know exactly what's the problem.

So I have two suggestions:
- add a --verbose flag that output's all the shell commands and it's outputs
- show more informative errors.

If you want I can try to help.

What do you think of this suggestion?
@lvsti
Copy link
Owner

lvsti commented Oct 3, 2018

totally agree

@lvsti lvsti added the enhancement New feature or request label Oct 3, 2018
@4brunu 4brunu mentioned this issue Oct 4, 2018
Merged
@lvsti lvsti closed this as completed in #22 Oct 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve error messages 4brunu/Bridgecraft
2 participants
@4brunu @lvsti