-
Notifications
You must be signed in to change notification settings - Fork 521
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
Third-party package updates #2464
Third-party package updates #2464
Conversation
BuildRequires: meson | ||
BuildRequires: %{_cross_os}glibc-devel | ||
BuildRequires: %{_cross_os}libffi-devel | ||
BuildRequires: %{_cross_os}libmount-devel | ||
BuildRequires: %{_cross_os}libpcre-devel |
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.
This switch from PCRE1 to PCRE2 is fantastic!
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.
Since we moved to pcre2, do we need to change the License
field for this package? as we as the comment above:
# pcre is BSD-3-Clause
@@ -116,8 +110,6 @@ Requires: %{_cross_os}libuuid | |||
cp Documentation/licenses/COPYING.* . | |||
|
|||
%build | |||
# We have patches that touch the build system, so we need to regenerate |
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.
🎉
27593f8
to
a357547
Compare
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
a357547
to
1457b08
Compare
The last two forced pushes include:
|
I built and booted a |
1457b08
to
b2a708d
Compare
Forced push includes:
|
b2a708d
to
de69cfc
Compare
(Forced push fixes the referenced file in |
de69cfc
to
160967d
Compare
(Forced push fixes the upstream URL for |
Can you open issues for what you'd like to see happen with |
|
||
%install | ||
install -d %{buildroot}%{_cross_bindir} | ||
install -d %{buildroot}%{_cross_tmpfilesdir} | ||
install -d %{buildroot}%{_cross_templatedir} | ||
install -d %{buildroot}%{_cross_datadir}/nvidia-container-toolkit | ||
install -d %{buildroot}%{_cross_factorydir}/etc/nvidia-container-runtime | ||
install -p -m 0755 nvidia-container-toolkit %{buildroot}%{_cross_bindir}/nvidia-container-runtime-hook | ||
install -p -m 0755 nvidia-container-runtime-hook %{buildroot}%{_cross_bindir}/ |
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.
extremely minor nit: I try to avoid trailing slashes for directories like this since, if done habitually, it tends to lead to double slashes (//
) in path names in build output, which is harmless but annoying.
Not worth fixing here, it's mostly about asking the question of whether a slash is needed after a given macro when you're writing spec files.
Sure @bcressey! I need to spend some time reading in detail how each of the features in the packages work |
This also drops the 0001-sys_linux patch since that's been included in 4.3. Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Fixes CVE-2022-36109 Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
This regenerates the 0005 patch since it longer cleanly applies. Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Fixes CVE-2020-35512. The libdbus developers don't distribute gzipped files anymore. Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Fixes CVE-2022-40674 Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
160967d
to
17044dd
Compare
(Nothing functional changed in last forced push, I only added missing |
In newer versions of the library, the `gio-launch-desktop` desktop helper binary was added. There isn't a meson option to avoid building the helper binary, thus it is excluded in the RPM recipe. Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
libnvidia-container added support to configure cgroups through a go library, which is currently "experimental". In order to use the stable behavior, this package has to be compiled with `WITH_NVCGO=n`. This also updates the upstream URL to use a more specific file name instead of only the version of the package used. Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
This also updates the upstream URL to use a more specific file name instead of only the version of the package used. Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Fixes CVE-2022-31676 In ed84c0a175c0122555a6a7cc947590569d28becb, support to list docker and kubernetes containers was added to open-vm-tools. We should consider enabling this feature in the future. Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
This also drops 1000-libmount-kernel-compat.patch, since the patch was required for kernels < 5.8, and we no longer have the sources for the 5.4 kernel. Signed-off-by: Arnaldo Garcia Rincon <agarrcia@amazon.com>
17044dd
to
004b8e7
Compare
(Forced push removes note about license in libglib) |
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.
🚀
Issue number:
Closes #2411
Description of changes:
libnvidia-container added support for some experimental features gated behind the
WITH_NVCGO
flag. For the time being, we are not using those features since that will require a huge fundamental change to how we use the library.open-vm-tools added support for a plugin to list containers running in either docker or kubernetes. For this update, I skipped the support for the plugin, but we should turn this on as a fast-follow PR.
There are a few packages that added security fixes:
The following packages were skipped:
There isn't any security fix for any of them, but we need to spend some time figuring out why they aren't happy with the provided arguments.
The following package requires more testing:
And lastly, the following packages will be updated in a different PR:
Testing done:
cargo make test
for aws-k8s-1.22cargo make test
for aws-k8s-1.21chronyd
runs for at least 1hrTerms of contribution:
By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.