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

Launch UWP app from command line #24534

Merged
merged 3 commits into from Oct 25, 2019
Merged

Launch UWP app from command line #24534

merged 3 commits into from Oct 25, 2019

Conversation

@ferjm
Copy link
Member

ferjm commented Oct 24, 2019

  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors

This PR makes it possible to launch the UWP app from the command line by executing (on any path) the Servo command. It requires building and launching Servo once on VS, so the command alias is properly registered.

It also allows passing command line arguments. So we can do for example Servo --pref dom.webxr.enabled.

Debugging is also possible when launching the app from the command line. It requires setting the Launch Application option (right click on ServoApp (Universal Windows) on the Solution Explorer view -> Properties -> Configuration Properties -> Debugging) to No.

@ferjm
Copy link
Member Author

ferjm commented Oct 24, 2019

@highfive highfive assigned paulrouget and unassigned asajeffrey Oct 24, 2019
@paulrouget
Copy link
Contributor

paulrouget commented Oct 24, 2019

Aren't we gonna lose the dom.webxr.enabled pref when deploying from VS?

@ferjm
Copy link
Member Author

ferjm commented Oct 24, 2019

Yes. I just fixed it with the last commit.

@paulrouget
Copy link
Contributor

paulrouget commented Oct 25, 2019

Works well :) Thank you.

@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented Oct 25, 2019

📌 Commit aeac55a has been approved by paulrouget

@bors-servo
Copy link
Contributor

bors-servo commented Oct 25, 2019

Testing commit aeac55a with merge f70df1c...

bors-servo added a commit that referenced this pull request Oct 25, 2019
Launch UWP app from command line

- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors

This PR makes it possible to launch the UWP app from the command line by executing (on any path) the `Servo` command. It requires building and launching Servo once on VS, so the command alias is properly registered.

It also allows passing command line arguments. So we can do for example `Servo --pref dom.webxr.enabled`.

Debugging is also possible when launching the app from the command line. It requires setting the `Launch Application` option (right click on `ServoApp (Universal Windows`) on the Solution Explorer view -> Properties ->  Configuration Properties -> Debugging) to `No`.
@bors-servo
Copy link
Contributor

bors-servo commented Oct 25, 2019

💔 Test failed - status-taskcluster

@jdm
Copy link
Member

jdm commented Oct 25, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Oct 25, 2019

💣 Failed to start rebuilding: Unknown error

@bors-servo
Copy link
Contributor

bors-servo commented Oct 25, 2019

Testing commit aeac55a with merge 9117e97...

bors-servo added a commit that referenced this pull request Oct 25, 2019
Launch UWP app from command line

- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors

This PR makes it possible to launch the UWP app from the command line by executing (on any path) the `Servo` command. It requires building and launching Servo once on VS, so the command alias is properly registered.

It also allows passing command line arguments. So we can do for example `Servo --pref dom.webxr.enabled`.

Debugging is also possible when launching the app from the command line. It requires setting the `Launch Application` option (right click on `ServoApp (Universal Windows`) on the Solution Explorer view -> Properties ->  Configuration Properties -> Debugging) to `No`.
@bors-servo
Copy link
Contributor

bors-servo commented Oct 25, 2019

☀️ Test successful - linux-rel-css, linux-rel-wpt, status-taskcluster
Approved by: paulrouget
Pushing 9117e97 to master...

@bors-servo bors-servo merged commit aeac55a into servo:master Oct 25, 2019
2 checks passed
2 checks passed
Taskcluster (pull_request) TaskGroup: success
Details
homu Test successful
Details
@ferjm ferjm deleted the ferjm:uwp.cli.args branch Oct 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
You can’t perform that action at this time.