Re: [OS-BUILD PATCHv3] redhat/kernel.spec.template: update compression variables to support zstd

2023-08-23 Thread Scott Weaver (via Email Bridge)
From: Scott Weaver on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2650#note_1525779750 Not worth it, just wondering. Thanks Brian. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to

✅ PASS (SKIPPED 1 of 154): Test report for 6.4.12-100.fc37 (kernel-fedora)

2023-08-23 Thread cki-project
Hi, we tested your kernel and here are the results: Overall result: PASSED Merge: OK Compile: OK Test: OK Tested-by: CKI Project Kernel information: Brew / Koji Task ID: 105195582 You can find all the details about the test run at

Re: [OS-BUILD PATCHv3] redhat/kernel.spec.template: update compression variables to support zstd

2023-08-23 Thread Brian Masney (via Email Bridge)
From: Brian Masney on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2650#note_1525688570 I left xz as a dependency because it is still used by these places: ``` redhat/Makefile:TARFILE:=linux-$(SPECTARFILE_RELEASE).tar.xz redhat/Makefile:@rm -f

Re: [OS-BUILD PATCHv3] redhat/kernel.spec.template: update compression variables to support zstd

2023-08-23 Thread Scott Weaver (via Email Bridge)
From: Scott Weaver on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2650#note_1525618035 Just a nit but it looks like in the current case `xz` will be defined twice in the list. Can `xz` can be dropped from the list or maybe we still need it somewhere else if %compression

Re: [OS-BUILD PATCHv3] redhat/kernel.spec.template: update compression variables to support zstd

2023-08-23 Thread Scott Weaver (via Email Bridge)
From: Scott Weaver on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2650#note_1525612236 Hi @bmasney. IIUC you're just adding the variable `compression_flags` for future use but this drops the `-9` that used to be defined. Should it be `%global compression_flags --compress

Re: [OS-BUILD PATCH 1/3] redhat: Use kspdx-tool output for the License: field

2023-08-23 Thread Thorsten Leemhuis (via Email Bridge)
From: Thorsten Leemhuis on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2648#note_1525439020 The "and Redistributable, no modification permitted" seems to be dropped (or easy to miss in this noisy License tag). Is this international? I wonder why it's there in the fist

[OS-BUILD PATCHv3] redhat/kernel.spec.template: update compression variables to support zstd

2023-08-23 Thread Brian Masney (via Email Bridge)
From: Brian Masney redhat/kernel.spec.template: update compression variables to support zstd Upstream Status: RHEL Only The kernel.spec.template file has a compression variable to define which binary is used to compress the kernel modules. The binary is currently invoked in the following

Re: [OS-BUILD PATCHv2] redhat/kernel.spec.template: update compression variables to support zstd

2023-08-23 Thread Brian Masney (via Email Bridge)
From: Brian Masney on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2650#note_1524848745 I just pushed up a fix for the stdout issue. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to

[OS-BUILD PATCHv2] redhat/kernel.spec.template: update compression variables to support zstd

2023-08-23 Thread Brian Masney (via Email Bridge)
From: Brian Masney redhat/kernel.spec.template: update compression variables to support zstd Upstream Status: RHEL Only The kernel.spec.template file has a compression variable to define which binary is used to compress the kernel modules. The binary is currently invoked in the following

Re: [OS-BUILD PATCH] redhat/kernel.spec.template: update compression variables to support zstd

2023-08-23 Thread Patrick Talbert (via Email Bridge)
From: Patrick Talbert on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2650#note_1524843445 Compression level for xz is just a dash followed by a number so I suppose the intent here was to have a compression level of 9, or "-9".

Re: [OS-BUILD PATCH] redhat/kernel.spec.template: update compression variables to support zstd

2023-08-23 Thread Patrick Talbert (via Email Bridge)
From: Patrick Talbert on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2650#note_1524841502 ~~~ 00:29:35 Compressing kernel modules ... 00:29:35 + find /tmp/rpmbuild/BUILDROOT/kernel-6.5.0-0.rc7.89bf6209cad6.52.tes t.eln.x86_64/lib/modules/ -type f -name '*.ko' 00:29:35 +

Re: [OS-BUILD PATCH] redhat/kernel.spec.template: update compression variables to support zstd

2023-08-23 Thread Patrick Talbert (via Email Bridge)
From: Patrick Talbert on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2650#note_1524839951 Passing `-c` to `xz` means "Write the compressed or decompressed data to standard output instead of a file." and that is what appears to be happening.

Re: [OS-BUILD PATCH] redhat/kernel.spec.template: update compression variables to support zstd

2023-08-23 Thread Michael Hofmann (via Email Bridge)
From: Michael Hofmann on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2650#note_1524804299 The pipeline in this MR add a huge amount of binary data to the build.log, e.g. https://s3.amazonaws.com/arr-cki-prod-trusted-artifacts/trusted-

[OS-BUILD PATCH 1/3] redhat: Use kspdx-tool output for the License: field

2023-08-23 Thread Vitaly Kuznetsov (via Email Bridge)
From: Vitaly Kuznetsov redhat: Use kspdx-tool output for the License: field Use kspdx-tool (https://gitlab.com/vkuznets/kspdx-tool) to find all licenses in the source code and its output to the License: field of the specfile. Package the tool and its current output. Signed-off-by: Vitaly

[OS-BUILD PATCH 2/3] redhat: Introduce "Verify SPDX-License-Identifier tags" selftest

2023-08-23 Thread Vitaly Kuznetsov (via Email Bridge)
From: Vitaly Kuznetsov redhat: Introduce "Verify SPDX-License-Identifier tags" selftest Add a test to check that all SPDX-License-Identifier: specified licenses in the code match the License: tag in the specfile. Signed-off-by: Vitaly Kuznetsov diff --git

[OS-BUILD PATCH 0/3] redhat: Keep track of SPDX-License-Identifier tags in the code

2023-08-23 Thread Vitaly Kuznetsov (via Email Bridge)
From: Vitaly Kuznetsov on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2648 Be more picky about kernel's source code licenses: - Keep track of all currently used license in redhat/LICENSES and use that for the License: fields in the specfile. - Add a

Re: [OS-BUILD PATCH] redhat/kernel.spec.template: update compression variables to support zstd

2023-08-23 Thread Brian Masney (via Email Bridge)
From: Brian Masney on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2650#note_1524503029 Regarding the error about `VIDEO_IMX8_ISI_M2M=m`, I don't see that set anywhere in my branch: ``` brian-x1:~/src/redhat/kernel-ark (zstd-compression)$ git grep VIDEO_IMX8_ISI_M2M

[OS-BUILD PATCH] redhat/kernel.spec.template: update compression variables to support zstd

2023-08-23 Thread Brian Masney (via Email Bridge)
From: Brian Masney redhat/kernel.spec.template: update compression variables to support zstd Upstream Status: RHEL Only The kernel.spec.template file has a compression variable to define which binary is used to compress the kernel modules. The binary is currently invoked in the following