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

Fix doc build with gstreamer #882

Merged
merged 1 commit into from Aug 26, 2018
Merged

Conversation

@Manishearth
Copy link
Member

Manishearth commented Aug 24, 2018

Doc builder needs the gstreamer deps too.

fixes servo/servo#21416

Untested since I have to wait for buildbot to stop testing things to reload it.

r? @jdm


This change is Reviewable

@jdm
jdm approved these changes Aug 24, 2018
@Manishearth Manishearth force-pushed the Manishearth:gstreamer-doc branch from 2980bc5 to 7c55ae6 Aug 26, 2018
@Manishearth
Copy link
Member Author

Manishearth commented Aug 26, 2018

@bors-servo r=jdm

@bors-servo
Copy link
Contributor

bors-servo commented Aug 26, 2018

📌 Commit 7c55ae6 has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Aug 26, 2018

Testing commit 7c55ae6 with merge 5c37cc6...

bors-servo added a commit that referenced this pull request Aug 26, 2018
Fix doc build with gstreamer

Doc builder needs the gstreamer deps too.

fixes servo/servo#21416

Untested since I have to wait for buildbot to stop testing things to reload it.

r? @jdm

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/882)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Aug 26, 2018

☀️ Test successful - status-travis
Approved by: jdm
Pushing 5c37cc6 to master...

@bors-servo bors-servo merged commit 7c55ae6 into servo:master Aug 26, 2018
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@jdm jdm removed the S-needs-deploy label Sep 25, 2018
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.

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