Re: [OS-BUILD PATCHv2] redhat/configs/README: Update the README

2022-06-01 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1841#note_968039019 New version pushed after feedback in MR. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to

[OS-BUILD PATCHv2] redhat/configs/README: Update the README

2022-06-01 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava redhat/configs/README: Update the README The information in this file was a bit out-of-date and needed a refresh. Update the README. Signed-off-by: Prarit Bhargava diff --git a/redhat/configs/README b/redhat/configs/README index blahblah..blahblah 100644 ---

Re: [OS-BUILD PATCH] redhat/configs/README: Update the README

2022-06-01 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1841#note_968037233 >Another indentation issue, but more importantly I think you need to specify which config file. Ditto :) ___ kernel mailing list --

Re: [OS-BUILD PATCH] redhat/configs/README: Update the README

2022-06-01 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1841#note_968036746 >For some reason you indented here. This was intentional to indicate that it fell under the pending-* section. I'll just move it over a few spaces. I'm neither here nor there on

Re: [OS-BUILD PATCH] redhat/configs/README: Update the README

2022-06-01 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1841#note_968034619 >I think a an example, such as: "For instance, ark/rhel only support x86_64; however, fedora supports x86 (32-bit)" - makes sense. Heh :) Not calling out a specific difference was

Re: [OS-BUILD PATCH] redhat/configs/README: Update the README

2022-06-01 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1841#note_967953140 More likely, we will pick up riscv before RHEL does, though I do not know a timeframe on that. ___ kernel mailing list --

Re: [OS-BUILD PATCH] redhat/configs/README: Update the README

2022-06-01 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1841#note_967949964 An incorrect example, but an example. Fedora has not built i686 kernels for a very long time. We do still build armv7 kernels until Fedora 36 is EOL, but those are going away as

Re: [OS-BUILD PATCH] redhat/configs/README: Update the README

2022-06-01 Thread David Arcari (via Email Bridge)
From: David Arcari on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1841#note_967949232 Another indentation issue, but more importantly I think you need to specify which config file. ___ kernel mailing list --

Re: [OS-BUILD PATCH] redhat/configs/README: Update the README

2022-06-01 Thread David Arcari (via Email Bridge)
From: David Arcari on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1841#note_967947033 For some reason you indented here. ___ kernel mailing list -- kernel@lists.fedoraproject.org To unsubscribe send an email to

Re: [OS-BUILD PATCH] redhat/configs/README: Update the README

2022-06-01 Thread David Arcari (via Email Bridge)
From: David Arcari on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1841#note_967938220 I think a an example, such as: "For instance, ark/rhel only support x86_64; however, fedora supports x86 (32-bit)" - makes sense. ___ kernel

Re: [OS-BUILD PATCH 0/0] [redhat] New configs in init/Kconfig

2022-06-01 Thread via Email Bridge
From: Ondrej Mosnáček on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1828#note_967851990 OK, but wouldn't it save @jmflinuxtx some work if the Fedora configs are consolidated in the same MR rather than him having to add another commit that consolidates them into

[OS-BUILD PATCH 2/4] redhat/self-test: self-test data update

2022-06-01 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava redhat/self-test: self-test data update Fixup the self-test data wrt the last commit. Signed-off-by: Prarit Bhargava diff --git a/redhat/self-test/data/centos-2585cf9dfaad.el7 b/redhat/self-test/data/centos-2585cf9dfaad.el7 index blahblah..blahblah 100644 ---

[OS-BUILD PATCH 4/4] self-test: Fixup dist release test

2022-06-01 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava self-test: Fixup dist release test The dist release test expects *one* BUILD value. With the latest changes it is possible that two are output. Fix the dist release test to check only one BUILD value. Signed-off-by: Prarit Bhargava diff --git

[OS-BUILD PATCH 0/4] redhat: Fixup self-tests

2022-06-01 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava on gitlab.com Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1800 The merge window exposed a bug in the redhat/self-test/data/create-data.sh script where the local variables were being reported instead of the test variables. Fix the create-data.sh

[OS-BUILD PATCH 3/4] self-test: Fixup Makefile contents test

2022-06-01 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava self-test: Fixup Makefile contents test The redhat/Makefile contains a "?=" that is caught up in the grep of the Makefile contents test [1]. Ignore the match in the test. [1] redhat/Makefile:716 @cat Makefile.variables | grep -v "^#" | sed '/^$$/d' | tr -d " " | awk -F

[OS-BUILD PATCH 1/4] redhat/self-test: Fix up create-data.sh to not report local variables

2022-06-01 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava redhat/self-test: Fix up create-data.sh to not report local variables Fix the create-data.sh script by moving the reporting of the Makefile.variables into the dist-dump-variables target. This has the minor annoyance that dist-dump-variables will report Makefile.variables