-
Notifications
You must be signed in to change notification settings - Fork 3
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
[VIDEO-2688] - Install Windows MSIs from custom source #175
Conversation
…verridden URI with uninstall clean up
@lyramcmillan is this ready for review? looks like some unit tests are failing. |
Nope - trying to figure out why tests are failing. There's something bad in the test setup I think but it's proving intractable. Making into a draft. |
…on failure. Tryimg another version to see if it's something about 1.22.7
…ssues in the code
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM -- just update the readme
README.md
Outdated
### `devMsiUrl` | ||
|
||
An optional override url for downloading the Windows gstreamer development libraries and tools MSI installer. If not specified, the default is `https://gstreamer.freedesktop.org/data/pkg/windows/<version>/gstreamer-1.0-devel-<arch>-<version>.msi`. | ||
If the msiUrl is specified the devMsiUrl must also be specified. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
techinically a run-on sentence. just add a comma there, or was that supposed to be then?
README.md
Outdated
|
||
### `msiUrl` | ||
|
||
An optional override url for downloading the Windows gstreamer runtime MSI installer. If not specified, the default is `https://gstreamer.freedesktop.org/data/pkg/windows/<version>/gstreamer-1.0-<arch>-<version>.msi`. If this input is specified then the action post run will be run and gsrtreamer will be uninstalled. These are primarily meant to be used in self hosted runners and not run in the gstreamer hosted runners that are bootstrapped and executed as non-persistent containers. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gsrtreamer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will need a follow up on why the windows-2022 unit tests are failing. otherwise lets go!!!
Description
Updates supporting downloading Windows MSIs from an overridden URI with uninstall clean up. Added optional inputs and code to install MSIs from specified URLs using optional build run parameter for isolating the install location. Also has post build action cleanup uninstall of gstream,er
Ticket(s) Addressed:
Design Summary
Configuration changes:
Testing Notes and Procedure
Related Pull Requests