Skip to content

Comments

Update virtualenv package name for Debian systems#9116

Merged
bors-servo merged 1 commit intoservo:masterfrom
adrianheine:patch-1
Jan 2, 2016
Merged

Update virtualenv package name for Debian systems#9116
bors-servo merged 1 commit intoservo:masterfrom
adrianheine:patch-1

Conversation

@adrianheine
Copy link
Contributor

Since Debian Jessie (current stable), the executable is in a virtualenv package.

Review on Reviewable

Since Debian Jessie (current stable), the executable is in a `virtualenv` package.
@highfive
Copy link

highfive commented Jan 1, 2016

Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @Manishearth (or someone else) soon.

@highfive highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jan 1, 2016
@jdm
Copy link
Member

jdm commented Jan 1, 2016

@bors-servo: r+
Thanks for the PR!

@bors-servo
Copy link
Contributor

📌 Commit be45aff has been approved by jdm

@bors-servo
Copy link
Contributor

⌛ Testing commit be45aff with merge e457e07...

@highfive highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jan 1, 2016
bors-servo pushed a commit that referenced this pull request Jan 1, 2016
Update virtualenv package name for Debian systems

Since Debian Jessie (current stable), the executable is in a `virtualenv` package.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9116)
<!-- Reviewable:end -->
@highfive highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jan 1, 2016
@bors-servo
Copy link
Contributor

💔 Test failed - linux-rel

@highfive highfive added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Jan 2, 2016
@jdm
Copy link
Member

jdm commented Jan 2, 2016

@bors-servo
Copy link
Contributor

⌛ Testing commit be45aff with merge b8fd8d9...

@highfive highfive removed the S-tests-failed The changes caused existing tests to fail. label Jan 2, 2016
bors-servo pushed a commit that referenced this pull request Jan 2, 2016
Update virtualenv package name for Debian systems

Since Debian Jessie (current stable), the executable is in a `virtualenv` package.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9116)
<!-- Reviewable:end -->
@highfive highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jan 2, 2016
@bors-servo
Copy link
Contributor

💔 Test failed - linux-rel

@highfive highfive added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Jan 2, 2016
@jdm
Copy link
Member

jdm commented Jan 2, 2016

@bors-servo
Copy link
Contributor

⚡ Previous build results for android, gonk, linux-dev, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt are reusable. Rebuilding only linux-rel...

@bors-servo
Copy link
Contributor

💔 Test failed - linux-rel

@highfive highfive added the S-tests-failed The changes caused existing tests to fail. label Jan 2, 2016
@jdm
Copy link
Member

jdm commented Jan 2, 2016

@bors-servo
Copy link
Contributor

⚡ Previous build results for android, gonk, linux-dev, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt are reusable. Rebuilding only linux-rel...

@bors-servo
Copy link
Contributor

☀️ Test successful - android, gonk, linux-dev, linux-rel, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt

@bors-servo bors-servo merged commit be45aff into servo:master Jan 2, 2016
@adrianheine adrianheine deleted the patch-1 branch January 2, 2016 19:00
@SimonSapin SimonSapin removed the S-tests-failed The changes caused existing tests to fail. label Jan 27, 2016
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.

5 participants