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

Jump to bottom

x/mobile/cmd/gomobile: init should write the errors to Stderr in non-verbose mode #11295

Closed
rakyll opened this issue Jun 19, 2015 · 4 comments
Closed

x/mobile/cmd/gomobile: init should write the errors to Stderr in non-verbose mode #11295

rakyll opened this issue Jun 19, 2015 · 4 comments
Labels
FrozenDueToAge
Milestone
Go1.5Maybe

Comments

@rakyll
Copy link
Contributor

rakyll commented Jun 19, 2015 

$ gomobile init
gomobile: exit status 2

gomobile doesn't pipe the stderrs of what it's shelling out to its stderr. When it exists with a non-zero error code, it must be able to dump the errors. The silent exit is not user friendly and is not expected.
@rakyll rakyll self-assigned this Jun 19, 2015
@rakyll rakyll added this to the Go1.5 milestone Jun 19, 2015
@nubunto
Copy link

nubunto commented Jun 20, 2015

Getting this on checkout 62f1, any quick n' dirty workaround?

@rakyll
Copy link
Contributor Author

rakyll commented Jun 20, 2015

Use gomobile init -v to see the error messages.

@gopherbot
Copy link

CL https://golang.org/cl/11266 mentions this issue.

@rakyll rakyll modified the milestones: Go1.5Maybe, Go1.5 Jun 30, 2015
@gopherbot
Copy link

CL https://golang.org/cl/11814 mentions this issue.

@golang golang locked and limited conversation to collaborators Jul 1, 2016
imWildCat pushed a commit to imWildCat/go-mobile that referenced this issue Apr 10, 2021
@crawshaw
cmd/gomobile: report errors from go install std
949a0d9 
While here, make the output of -v easier to read.

Fixes golang/go#11295, golang/go#11296.

Change-Id: I51974a3443bb60a5c0bb77ac784e48987eee8391
Reviewed-on: https://go-review.googlesource.com/11814
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
imWildCat pushed a commit to imWildCat/go-mobile that referenced this issue Apr 11, 2021
@crawshaw
cmd/gomobile: report errors from go install std
2ed7178 
While here, make the output of -v easier to read.

Fixes golang/go#11295, golang/go#11296.

Change-Id: I51974a3443bb60a5c0bb77ac784e48987eee8391
Reviewed-on: https://go-review.googlesource.com/11814
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
@rsc rsc unassigned rakyll Jun 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge
Projects
None yet
Milestone
Go1.5Maybe
Development

No branches or pull requests
3 participants
@rakyll @nubunto @gopherbot