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

Make gstreamer packaging explicit #27302

Merged
merged 1 commit into from Jul 17, 2020
Merged

Make gstreamer packaging explicit #27302

merged 1 commit into from Jul 17, 2020

Conversation

@jdm
Copy link
Member

jdm commented Jul 16, 2020

This change aligns Windows and macOS in terms of loading an explicit set of included plugins when initializing gstreamer. It also creates a single source of truth - the set of expected plugins is generated in a build script by the same python code that is used for packaging, so it should be impossible for platforms relying on this system to get out of sync.

Fixes #27293

@highfive
Copy link

highfive commented Jul 16, 2020

Heads up! This PR modifies the following files:

  • @wafflespeanut: python/servo/gstreamer.py, python/servo/build_commands.py, python/servo/package_commands.py
@jdm
Copy link
Member Author

jdm commented Jul 16, 2020

@highfive highfive assigned Manishearth and unassigned nox Jul 16, 2020
@jdm
Copy link
Member Author

jdm commented Jul 16, 2020

I verified that a local build could still use webaudio, getUserMedia, and media elements after these changes.

@Manishearth
Copy link
Member

Manishearth commented Jul 16, 2020

@bors-servo
Copy link
Contributor

bors-servo commented Jul 16, 2020

📌 Commit af98ec4 has been approved by Manishearth

@bors-servo
Copy link
Contributor

bors-servo commented Jul 16, 2020

Testing commit af98ec4 with merge 8b1f4bf...

bors-servo added a commit that referenced this pull request Jul 16, 2020
Make gstreamer packaging explicit

This change aligns Windows and macOS in terms of loading an explicit set of included plugins when initializing gstreamer. It also creates a single source of truth - the set of expected plugins is generated in a build script by the same python code that is used for packaging, so it should be impossible for platforms relying on this system to get out of sync.

Fixes #27293
@bors-servo
Copy link
Contributor

bors-servo commented Jul 16, 2020

💔 Test failed - status-taskcluster

@jdm jdm force-pushed the jdm:gst-package-mac branch from af98ec4 to eb18f18 Jul 16, 2020
@jdm jdm force-pushed the jdm:gst-package-mac branch from eb18f18 to 1b8a5e0 Jul 16, 2020
@jdm
Copy link
Member Author

jdm commented Jul 16, 2020

@bors-servo try=wpt-mac

@bors-servo
Copy link
Contributor

bors-servo commented Jul 16, 2020

Trying commit 1b8a5e0 with merge 1cb8ede...

bors-servo added a commit that referenced this pull request Jul 16, 2020
Make gstreamer packaging explicit

This change aligns Windows and macOS in terms of loading an explicit set of included plugins when initializing gstreamer. It also creates a single source of truth - the set of expected plugins is generated in a build script by the same python code that is used for packaging, so it should be impossible for platforms relying on this system to get out of sync.

Fixes #27293
@bors-servo
Copy link
Contributor

bors-servo commented Jul 17, 2020

☀️ Test successful - status-taskcluster
State: approved= try=True

@jdm
Copy link
Member Author

jdm commented Jul 17, 2020

@bors-servo r=Manishearth

@bors-servo
Copy link
Contributor

bors-servo commented Jul 17, 2020

📌 Commit 1b8a5e0 has been approved by Manishearth

bors-servo added a commit that referenced this pull request Jul 17, 2020
Make gstreamer packaging explicit

This change aligns Windows and macOS in terms of loading an explicit set of included plugins when initializing gstreamer. It also creates a single source of truth - the set of expected plugins is generated in a build script by the same python code that is used for packaging, so it should be impossible for platforms relying on this system to get out of sync.

Fixes #27293
@bors-servo
Copy link
Contributor

bors-servo commented Jul 17, 2020

Testing commit 1b8a5e0 with merge aa6612b...

…the nightly package.
@jdm jdm force-pushed the jdm:gst-package-mac branch from be95d80 to 72040be Jul 17, 2020
@jdm
Copy link
Member Author

jdm commented Jul 17, 2020

@bors-servo r=Manishearth

@bors-servo
Copy link
Contributor

bors-servo commented Jul 17, 2020

📌 Commit 72040be has been approved by Manishearth

bors-servo added a commit that referenced this pull request Jul 17, 2020
Make gstreamer packaging explicit

This change aligns Windows and macOS in terms of loading an explicit set of included plugins when initializing gstreamer. It also creates a single source of truth - the set of expected plugins is generated in a build script by the same python code that is used for packaging, so it should be impossible for platforms relying on this system to get out of sync.

Fixes #27293
@bors-servo
Copy link
Contributor

bors-servo commented Jul 17, 2020

Testing commit 72040be with merge 3c8bca0...

@bors-servo
Copy link
Contributor

bors-servo commented Jul 17, 2020

💔 Test failed - status-taskcluster

@jdm
Copy link
Member Author

jdm commented Jul 17, 2020

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Jul 17, 2020

Testing commit 72040be with merge f6f470e...

bors-servo added a commit that referenced this pull request Jul 17, 2020
Make gstreamer packaging explicit

This change aligns Windows and macOS in terms of loading an explicit set of included plugins when initializing gstreamer. It also creates a single source of truth - the set of expected plugins is generated in a build script by the same python code that is used for packaging, so it should be impossible for platforms relying on this system to get out of sync.

Fixes #27293
@bors-servo
Copy link
Contributor

bors-servo commented Jul 17, 2020

💔 Test failed - status-taskcluster

@jdm
Copy link
Member Author

jdm commented Jul 17, 2020

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Jul 17, 2020

Testing commit 72040be with merge 86e7f9a...

@bors-servo
Copy link
Contributor

bors-servo commented Jul 17, 2020

☀️ Test successful - status-taskcluster
Approved by: Manishearth
Pushing 86e7f9a to master...

@bors-servo bors-servo merged commit 86e7f9a into servo:master Jul 17, 2020
2 checks passed
2 checks passed
Community-TC (pull_request) TaskGroup: success
Details
homu Test successful
Details
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.

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