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

Ninja -t browse opens localhost in browser, which doesn't work when using ssh #470

Merged
merged 1 commit into from
May 23, 2013

Conversation

rgeary1
Copy link
Contributor

@rgeary1 rgeary1 commented Dec 13, 2012

Browse tool should use the real hostname not http://localhost, to allow -t browse when using ssh

Fix attached.

@buildhive
Copy link

Evan Martin » ninja #249 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@buildhive
Copy link

Evan Martin » ninja #276 SUCCESS
This pull request looks good
(what's this?)

@buildhive
Copy link

Evan Martin » ninja #473 SUCCESS
This pull request looks good
(what's this?)

… using ssh

Change-Id: Ic4be6527151a7ff68afab62e61951071ad5694b7
@rgeary1
Copy link
Contributor Author

rgeary1 commented May 22, 2013

Updated to ninja 1.3

@buildhive
Copy link

Evan Martin » ninja #474 SUCCESS
This pull request looks good
(what's this?)

@evmar
Copy link
Collaborator

evmar commented May 23, 2013

Sorry I dropped this!

evmar added a commit that referenced this pull request May 23, 2013
Ninja -t browse opens localhost in browser, which doesn't work when using ssh
@evmar evmar merged commit 145d4f2 into ninja-build:master May 23, 2013
@rgeary1 rgeary1 deleted the remotebrowser branch May 23, 2013 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants