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

Enable GStreamer on Windows. #24004

Merged
merged 1 commit into from Aug 27, 2019
Merged

Enable GStreamer on Windows. #24004

merged 1 commit into from Aug 27, 2019

Conversation

@jdm
Copy link
Member

jdm commented Aug 19, 2019

I have verified that desktop and emulator UWP builds work with this set of DLLs.


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes do not require tests because no tests for UWP

This change is Reviewable

@jdm
Copy link
Member Author

jdm commented Aug 19, 2019

r? @ferjm

@highfive highfive assigned ferjm and unassigned nox Aug 19, 2019
@jdm jdm force-pushed the win-gstreamer branch from caf35ec to 27f7b87 Aug 19, 2019
@jdm
Copy link
Member Author

jdm commented Aug 19, 2019

Manish says this prevents the UWP app from launching on the device, so don't merge it yet.

@bors-servo
Copy link
Contributor

bors-servo commented Aug 22, 2019

The latest upstream changes (presumably #23961) made this pull request unmergeable. Please resolve the merge conflicts.

@jdm
Copy link
Member Author

jdm commented Aug 26, 2019

I have verified that this does work on the device and on desktop, so I'm going to go ahead and merge this.

@jdm jdm force-pushed the win-gstreamer branch from 27f7b87 to c0ff30b Aug 26, 2019
@jdm
Copy link
Member Author

jdm commented Aug 26, 2019

r? @ferjm

@jdm jdm removed the S-needs-rebase label Aug 26, 2019
@ferjm
Copy link
Member

ferjm commented Aug 27, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Aug 27, 2019

📌 Commit c0ff30b has been approved by ferjm

@bors-servo
Copy link
Contributor

bors-servo commented Aug 27, 2019

Testing commit c0ff30b with merge a34128d...

bors-servo added a commit that referenced this pull request Aug 27, 2019
Enable GStreamer on Windows.

I have verified that desktop and emulator UWP builds work with this set of DLLs.

---
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes do not require tests because no tests for UWP

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

bors-servo commented Aug 27, 2019

💔 Test failed - status-taskcluster

@CYBAI
Copy link
Collaborator

CYBAI commented Aug 27, 2019

@bors-servo retry

  • #24066 (I believe this failure is unrelated 👀)
@bors-servo
Copy link
Contributor

bors-servo commented Aug 27, 2019

Testing commit c0ff30b with merge 9fe3c4f...

bors-servo added a commit that referenced this pull request Aug 27, 2019
Enable GStreamer on Windows.

I have verified that desktop and emulator UWP builds work with this set of DLLs.

---
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] These changes do not require tests because no tests for UWP

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

bors-servo commented Aug 27, 2019

☀️ Test successful - linux-rel-css, linux-rel-wpt, status-taskcluster
Approved by: ferjm
Pushing 9fe3c4f to master...

@bors-servo bors-servo merged commit c0ff30b into master Aug 27, 2019
3 checks passed
3 checks passed
Taskcluster (pull_request) TaskGroup: success
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
@SimonSapin SimonSapin deleted the win-gstreamer branch Sep 30, 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.