-
Notifications
You must be signed in to change notification settings - Fork 8.1k
remove outdated linux dependencies #14688
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
remove outdated linux dependencies #14688
Conversation
…uid libuniwnd already removed
|
Re-running failed test on Windows, seems unrelated to change. |
|
Failing test is: "WinRM based remoting session abrupt disconnect"...I think this may be related to Disconnect issue Rob was mentioning last week. Will reach out to him |
| "libc6", | ||
| "libgcc1", | ||
| "libgssapi-krb5-2", | ||
| "liblttng-ust0", |
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.
Have you verified with the packages generated after this change can be built and installed?
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.
Balu verified and asked for this.
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.
verified this and ran on Coordinated pipeline and tested artifact deb package.
|
🎉 Handy links: |
remove outdated linux dependencies from packaging.psm1 file, specifically liblttng-ust0, libcurl libuuid libuniwnd already removed
remove linux dependency liblttng-ust0. Others mentioned in this issue #14597 were already removed.
PR Context
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.