Skip to content

Use ShellExecute rather than start.exe to open docs on windows#1117

Merged
bors merged 3 commits intorust-lang:masterfrom
mattico:use-shellexecute
May 18, 2017
Merged

Use ShellExecute rather than start.exe to open docs on windows#1117
bors merged 3 commits intorust-lang:masterfrom
mattico:use-shellexecute

Conversation

@mattico
Copy link
Copy Markdown
Contributor

@mattico mattico commented May 12, 2017

Closes #499

The implementation is not too complicated, so it's a bit better than spawning a cmd.exe just to open docs. It also should fix opening docs for the few people still using a year-old insider preview build.

mattico added 3 commits May 12, 2017 18:48
Closes rust-lang#499

The implementation is not too complicated, so it's a bit better than spawning a cmd.exe just to open docs. It also should fix opening docs for the few people still using a year-old insider preview build.
@Diggsey
Copy link
Copy Markdown
Contributor

Diggsey commented May 17, 2017

Thanks, this looks great!

@bors r+

@bors
Copy link
Copy Markdown
Contributor

bors commented May 17, 2017

📌 Commit 9733eaf has been approved by Diggsey

@bors
Copy link
Copy Markdown
Contributor

bors commented May 18, 2017

⌛ Testing commit 9733eaf with merge b8b620d...

bors added a commit that referenced this pull request May 18, 2017
Use ShellExecute rather than start.exe to open docs on windows

Closes #499

The implementation is not too complicated, so it's a bit better than spawning a cmd.exe just to open docs. It also should fix opening docs for the few people still using a year-old insider preview build.
@bors
Copy link
Copy Markdown
Contributor

bors commented May 18, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: Diggsey
Pushing b8b620d to master...

@bors bors merged commit 9733eaf into rust-lang:master May 18, 2017
@mattico mattico deleted the use-shellexecute branch May 18, 2017 16:11
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 this pull request may close these issues.

Use ShellExecute to open docs on windows

3 participants