Re: [OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951732268

Oh and @omos filed this https://gitlab.com/cki-project/kernel-
ark/-/merge_requests/1781 because he got hit with my buildreqs change from
last week.  That MR kinda undoes the check and defeats the spirit of it.
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH 3/3] [redhat] Add dist-buildreq-check for binary rpm builds

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951723323

Sure, but it doesn't describe why some rpmbuild commands get it, while others
do not.  Hence the -binary part.  If you are building binaries you needed the
extra suffix.  This makes it clear for future targets.  While the rule is the
same for your name, it doesn't appear to be as clear to me.
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951720936

@hertonrk-rh @prarit @jmflinuxtx

To answer a variety of questions.  Yes srpm processes the configs.  It kinda
has to otherwise what is rpmbuild going to use to build the kernel? :-)

Why is this an issue now? It popped up with my MR last week actually fixing
how this works (otherwise the dep was only rpmbuild). Some folks do not want
to install the full list of buildreqs just to generate a srpm and upload it.
Especially for users on Fedora trying to build a RHEL-X srpm and don't want to
have to install obsoleted or missing RHEL-X buildreqs (system-sb-certs, llvm-
tools, etc).

As for 'dnf builddep kernel' that could work too in some cases.  One quirk
with that approach is it needs to match the kernel in development, so a RHEL
kernel on a Fedora distro may not work.  But we can advertise this too if we
want.

I am also looking at the docs here: https://cki-project.gitlab.io/kernel-
ark/#cloning-the-repository should either be auto-generated or told to use
'dnf builddep kernel'.  I am working with COPR and they didn't have an easy
way to install the packages necessary to build the srpm other than guessing.

I am trying to improve the user experience
git clone 
make dist-srpmbuildreqs-check | sudo dnf install  (??)
make dist-srpm
koji build

or something like that.  IOW there is a gap here that I am trying to address
with a consistent story.  Something that isn't complicated (yeah, this MR
leans toward complicated).

Thoughts?
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


✅ PASS: Test report for kernel 5.17.9-200.fc35 (fedora-35)

2022-05-18 Thread CKI Project
Hello,

We ran automated tests on the following kernel build:

Kernel package: kernel-5.17.9-200.fc35
 Task URL: https://koji.fedoraproject.org/koji/taskinfo?taskID=87205287

The results of these automated tests are provided below.

Overall result: PASSED
 Tests: OK



All kernel binaries, config files, and logs are available for download here:

  
https://arr-cki-prod-datawarehouse-public.s3.amazonaws.com/index.html?prefix=datawarehouse-public/2022/05/18/542420152

Please reply to this email if you have any questions about the tests that we
ran or if you have any suggestions on how to make future tests more effective.
For the full detail on our testing procedures, please scroll to the bottom of
this message.

,-.   ,-.
   ( C ) ( K )  Continuous
`-',-.`-'   Kernel
  ( I ) Integration
   `-'
__

Hardware testing

We booted each kernel and ran the following tests:

  aarch64:
Host 1:

   ⚡ Internal infrastructure issues prevented one or more tests (marked
   with ⚡⚡⚡) from running on this architecture.
   This is not the fault of the kernel that was tested.

    ⚡⚡⚡ SELinux Custom Module Setup
    ⚡⚡⚡ machineinfo
   ⚡⚡⚡ Boot test
   ⚡⚡⚡ IPMI driver test
   ⚡⚡⚡ IPMItool loop stress test
   ⚡⚡⚡ stress: stress-ng - interrupt
   ⚡⚡⚡ stress: stress-ng - cpu
   ⚡⚡⚡ stress: stress-ng - cpu-cache
   ⚡⚡⚡ stress: stress-ng - memory
    ⚡⚡⚡ Storage blktests - blk
    ⚡⚡⚡ Storage blktests - nvme-tcp
    ⚡⚡⚡ Storage block - filesystem fio test
    ⚡⚡⚡ Storage block - queue scheduler test
    ⚡⚡⚡ Storage block - storage fio numa
   ⚡⚡⚡ Reboot test

Host 2:

   ⚡ Internal infrastructure issues prevented one or more tests (marked
   with ⚡⚡⚡) from running on this architecture.
   This is not the fault of the kernel that was tested.

    ⚡⚡⚡ SELinux Custom Module Setup
    ⚡⚡⚡ machineinfo
   ⚡⚡⚡ Boot test
   ⚡⚡⚡ ACPI table test
   ⚡⚡⚡ ACPI enabled test
   ⚡⚡⚡ i2c: i2cdetect sanity
   ⚡⚡⚡ LTP - cve
   ⚡⚡⚡ LTP - sched
   ⚡⚡⚡ LTP - syscalls
   ⚡⚡⚡ LTP - can
   ⚡⚡⚡ LTP - commands
   ⚡⚡⚡ LTP - containers
   ⚡⚡⚡ LTP - dio
   ⚡⚡⚡ LTP - fs
   ⚡⚡⚡ LTP - fsx
   ⚡⚡⚡ LTP - math
   ⚡⚡⚡ LTP - hugetlb
   ⚡⚡⚡ LTP - mm
   ⚡⚡⚡ LTP - nptl
   ⚡⚡⚡ LTP - pty
   ⚡⚡⚡ LTP - ipc
   ⚡⚡⚡ LTP - tracing
   ⚡⚡⚡ LTP: openposix test suite
   ⚡⚡⚡ CIFS Connectathon
   ⚡⚡⚡ POSIX pjd-fstest suites
   ⚡⚡⚡ Loopdev Sanity
   ⚡⚡⚡ jvm - jcstress tests
   ⚡⚡⚡ Memory: fork_mem
   ⚡⚡⚡ Memory function: memfd_create
   ⚡⚡⚡ AMTU (Abstract Machine Test Utility)
   ⚡⚡⚡ Networking bridge: sanity
   ⚡⚡⚡ Ethernet drivers sanity
   ⚡⚡⚡ Networking MACsec: sanity
   ⚡⚡⚡ Networking socket: fuzz
   ⚡⚡⚡ Networking sctp-auth: sockopts test
   ⚡⚡⚡ Networking: igmp conformance test
   ⚡⚡⚡ Networking route: pmtu
   ⚡⚡⚡ Networking route_func - local
   ⚡⚡⚡ Networking route_func - forward
   ⚡⚡⚡ Networking TCP: keepalive test
   ⚡⚡⚡ Networking UDP: socket
   ⚡⚡⚡ Networking cki netfilter test
   ⚡⚡⚡ Networking tunnel: geneve basic test
   ⚡⚡⚡ Networking tunnel: gre basic
   ⚡⚡⚡ L2TP basic test
   ⚡⚡⚡ Networking tunnel: vxlan basic
   ⚡⚡⚡ Networking tunnel: permtest
   ⚡⚡⚡ Networking ipsec: basic netns - transport
   ⚡⚡⚡ Networking ipsec: basic netns - tunnel
   ⚡⚡⚡ Networking vnic: ipvlan/basic
   ⚡⚡⚡ Libkcapi AF_ALG test
   ⚡⚡⚡ pciutils: update pci ids test
   ⚡⚡⚡ ALSA PCM loopback test
   ⚡⚡⚡ ALSA Control (mixer) Userspace Element test
   ⚡⚡⚡ Storage: block zstd smoke test
   ⚡⚡⚡ storage: dm/common
   ⚡⚡⚡ lvm thinp sanity
   ⚡⚡⚡ lvm snapper test
   ⚡⚡⚡ storage: SCSI VPD
   ⚡⚡⚡ Storage: swraid mdadm raid_module test
   ⚡⚡⚡ Storage: swraid scsi_raid
   ⚡⚡⚡ trace: ftrace/tracer
    ⚡⚡⚡ Podman system test - as root
    ⚡⚡⚡ Podman system test - as user
    ⚡⚡⚡ NFS Connectathon
    ⚡⚡⚡ Firmware test suite
    ⚡⚡⚡ Memory function: kaslr
    ⚡⚡⚡ Networking VRF: sanity
    ⚡⚡⚡ audit: audit testsuite test
    ⚡⚡⚡ Usex - version 1.9-29
    ⚡⚡⚡ Storage: block zstd compression test
    ⚡⚡⚡ lvm thinp stqe test
    ⚡⚡⚡ lvm cache test
    ⚡⚡⚡ storage: software RAID testing
    ⚡⚡⚡ storage: targetcli basic
   ⚡⚡⚡ Reboot test

Host 3:

   ⚡ Internal infrastructure issues prevented one or more tests (marked
   with ⚡⚡⚡) from running on this architecture.
   This is not the fault of the kernel that was tested.

    ⚡⚡⚡ SELinux Custom Module Setup
    ⚡⚡⚡ machineinfo
   ⚡⚡⚡ Boot test
    ⚡⚡⚡ Storage blktests - nvmeof-mp
   ⚡⚡⚡ Reboot test

Host 4:

   ⚡ Internal infrastructure issues prevented 

Re: [OS-BUILD PATCH 1/3] [redhat] Explicitly set srpm requirements

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951705198

I can't prove it mathematically, but in order to build the SRPM, you need to
build the configs.  What does dist-configs-check do that regular dist-configs
does not?  And if it is missing something, then we add it in there.
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-05-18 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951641657

@hertonrk-rh @jmflinuxtx : I think we just all asked the same question of
@dzickusrh, but in slightly different ways :smile:
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH 1/3] [redhat] Explicitly set srpm requirements

2022-05-18 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951640333



OOC what happens if the SRPM builds correctly, but something like
'dist-configs-check' fails because there are missing BuildRequires?

IOW, is SPECSRPMBUILDREQS really complete?
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-05-18 Thread Justin M. Forbes (via Email Bridge)
From: Justin M. Forbes on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951633741

srpm generation does process configs.  While you may not need full deps for a
proper config check, there are some which are required. That was what I meant
about this being a new list we will have to maintain.  Which buildreq's are
critical? I am somewhat curious as to what the driver is for this though. Is
it the space required, or just having to manually install all of the deps? If
it is the later, 'sudo dnf builddep kernel' makes it pretty easy.
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH 3/3] [redhat] Add dist-buildreq-check for binary rpm builds

2022-05-18 Thread Prarit Bhargava (via Email Bridge)
From: Prarit Bhargava on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951632878



I understand why you've named this 'do-rpmbuild-binary' but I think that
will be confusing to a casual reader.  At first glance I thought it
implied that do-rpmbuild is for generating SRPMs only.  Perhaps a better
name here would be 'do-rpmbuild-buildreq-check'?  I realize that is an
annoyingly long name but it does describe what the target actually does.
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-05-18 Thread Herton R. Krzesinski (via Email Bridge)
From: Herton R. Krzesinski on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951631511

However, what about the case srpm builds correctly, but configs fail on build,
due an extra buildrequires that is now present? In that case, I believe
SPECSRPMBUILDREQS needs some packages added, like glibc-static.
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-05-18 Thread Herton R. Krzesinski (via Email Bridge)
From: Herton R. Krzesinski on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951629181

or change the dist-config-check dependencies to add dist-buildreq-check, I
supposed it can work.
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-05-18 Thread Herton R. Krzesinski (via Email Bridge)
From: Herton R. Krzesinski on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786#note_951627747

I believe you need to add a new dist-configs-prep-rpm and dist-configs-prep-
srpm: srpm generation does not process configs right? But other targets that
may process it will need full build requires because some package missing can
change the config generation.
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[OS-BUILD PATCH 2/3] [redhat] Makefile: drop duplicate dist-sources deps

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Ondrej Mosnacek 

[redhat] Makefile: drop duplicate dist-sources deps

do-rpmbuild itself already depends on dist-sources, thus no need to
specify when we depend on do-rpmbuild already.

Signed-off-by: Ondrej Mosnacek 

diff --git a/redhat/Makefile b/redhat/Makefile
index blahblah..blahblah 100644
--- a/redhat/Makefile
+++ b/redhat/Makefile
@@ -574,29 +574,29 @@ do-rpmbuild: dist-sources
$(RPMBUILD) --define "_sourcedir $(SOURCES)" --define "_builddir 
$(RPM)/BUILD" --define "_srcrpmdir $(RPM)/SRPMS" --define "_rpmdir $(RPM)/RPMS" 
--define "_specdir $(RPM)/SPECS" --define "dist $(DIST)" $(RPMBUILDOPTS) 
$(RPM)/SOURCES/$(PACKAGE_NAME).spec
 
 dist-all-rpms: RPMBUILDOPTS=--target $(MACH) -ba
-dist-all-rpms: dist-sources do-rpmbuild
+dist-all-rpms: do-rpmbuild
 
 dist-srpm: RPMBUILDOPTS=--nodeps -bs
-dist-srpm: dist-sources do-rpmbuild
+dist-srpm: do-rpmbuild
 
 dist-srpm-gcov: BUILDID=.gcov
 dist-srpm-gcov: BUILDOPTS+=+gcov
 dist-srpm-gcov: dist-srpm
 
 dist-rpms: RPMBUILDOPTS=--target $(MACH) -bb
-dist-rpms: dist-sources do-rpmbuild
+dist-rpms: do-rpmbuild
 
-dist-kernel-%: dist-sources
+dist-kernel-%:
RPMBUILDOPTS="--target $(MACH) --with $* -bb" make do-rpmbuild
 
 dist-prep: RPMBUILDOPTS=--nodeps --target noarch -bp
-dist-prep: dist-sources do-rpmbuild
+dist-prep: do-rpmbuild
 
 dist-perf: RPMBUILDOPTS=--without up --without smp --without zfcpdump 
--without debug --without doc --without headers --without  --without doc 
--without debuginfo --target $(MACH) -bb
-dist-perf: dist-sources do-rpmbuild
+dist-perf: do-rpmbuild
 
 dist-rpm-baseonly: RPMBUILDOPTS=--target $(MACH) --without debug --without 
debuginfo --without vdso_install --without bpftool --without perf --without 
tools -bb
-dist-rpm-baseonly: dist-sources do-rpmbuild
+dist-rpm-baseonly: do-rpmbuild
 
 # unless you know what you're doing, you don't want to use the next four ones
 dist-release-finish: setup-source

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[OS-BUILD PATCH 1/3] [redhat] Explicitly set srpm requirements

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus 

[redhat] Explicitly set srpm requirements

It isn't clear what packages are necessary to be installed to generate
the srpm for the kernel.  Use another spec macro to call out the exact
packages needed. This allows a makefile target to be used to verify the
right packages have been installed.

Signed-off-by: Don Zickus 

diff --git a/redhat/Makefile b/redhat/Makefile
index blahblah..blahblah 100644
--- a/redhat/Makefile
+++ b/redhat/Makefile
@@ -84,6 +84,7 @@ TESTPATCH:=$(REDHAT)/linux-kernel-test.patch
 SPECCHANGELOG:=$(PACKAGE_NAME).changelog-$(RHEL_MAJOR).$(RHEL_MINOR)
 CHANGELOG_PREV:=$(PACKAGE_NAME).changelog-$(RHEL_MAJOR).$(shell expr 
$(RHEL_MINOR) - 1)
 ARCH_LIST=aarch64 ppc64le s390x x86_64
+SPECSRPMBUILDREQS := git-core make gcc flex bison bzip2
 
 ifndef DISTRO
   ifneq ($(findstring .fc,$(DIST)),)
@@ -391,7 +392,7 @@ rh-configs: dist-rhel-configs
 dist-configs-check: dist-configs-prep
+cd $(REDHAT)/configs; ./process_configs.sh 
$(PROCESS_CONFIGS_CHECK_OPTS) "" ""
 
-dist-configs-prep: dist-clean-configs dist-buildreq-check
+dist-configs-prep: dist-clean-configs dist-srpmbuildreq-check
+cd $(REDHAT)/configs; ./build_configs.sh "partial" "snip"
+cd $(REDHAT)/configs; ./build_configs.sh "$(PACKAGE_NAME)" "$(FLAVOR)"
 
@@ -469,6 +470,17 @@ dist-git-version-check:
exit 1; \
fi
 
+dist-srpmbuildreq-check: setup-source
+   @PKGLIST="rpm-build $(SPECSRPMBUILDREQS)"; \
+   MISSING=""; \
+   for pkg in $$PKGLIST; do \
+   rpm -q --whatprovides $$pkg >/dev/null || MISSING="$$MISSING 
$$pkg"; \
+   done; \
+   if [ -n "$$MISSING" ]; then \
+   echo "Error: please 'dnf install $$MISSING'"; \
+   exit 1; \
+   fi
+
 dist-buildreq-check: setup-source
@PKGLIST="rpm-build $$(rpmspec -q --buildrequires 
$(SOURCES)/$(SPECFILE) | cut -d ' ' -f 1)"; \
MISSING=""; \
@@ -812,6 +824,7 @@ dist-full-help:
 
@echo  ''
@echo  'Misc targets:'
+   @echo  '  dist-srpmbuildreq-check - Checks for presence of packages 
required for the srpm'
@echo  '  dist-buildreq-check - Checks for presence of packages 
required for build'
@echo  '  dist-get-latest - Returns the latest "known good" kernel from 
brew. This'
@echo  'should not be confused with the latest 
top-of-tree'
diff --git a/redhat/genspec.sh b/redhat/genspec.sh
index blahblah..blahblah 100755
--- a/redhat/genspec.sh
+++ b/redhat/genspec.sh
@@ -70,6 +70,7 @@ test -f "$SOURCES/$SPECFILE" &&
s/%%SPECINCLUDE_RHEL_FILES%%/$SPECINCLUDE_RHEL_FILES/
s/%%SPECPATCHLIST_CHANGELOG%%/$SPECPATCHLIST_CHANGELOG/
s/%%SPECVERSION%%/$SPECVERSION/
+   s/%%SPECSRPMBUILDREQS%%/$SPECSRPMBUILDREQS/
s/%%SPECTARFILE_RELEASE%%/$SPECTARFILE_RELEASE/" "$SOURCES/$SPECFILE"
 test -n "$RHSELFTESTDATA" && test -f "$SOURCES/$SPECFILE" && sed -i -e "
/%%SPECCHANGELOG%%/r $SOURCES/$SPECCHANGELOG
diff --git a/redhat/kernel.spec.template b/redhat/kernel.spec.template
index blahblah..blahblah 100755
--- a/redhat/kernel.spec.template
+++ b/redhat/kernel.spec.template
@@ -570,9 +570,10 @@ Requires: kernel-modules-uname-r = %{KVERREL}
 #
 # List the packages used during the kernel build
 #
-BuildRequires: kmod, patch, bash, coreutils, tar, git-core, which
-BuildRequires: bzip2, xz, findutils, gzip, m4, perl-interpreter, perl-Carp, 
perl-devel, perl-generators, make, diffutils, gawk
-BuildRequires: gcc, binutils, redhat-rpm-config, hmaccalc, bison, flex, gcc-c++
+BuildRequires: %%SPECSRPMBUILDREQS%%
+BuildRequires: kmod, patch, bash, coreutils, tar, which
+BuildRequires: xz, findutils, gzip, m4, perl-interpreter, perl-Carp, 
perl-devel, perl-generators, diffutils, gawk
+BuildRequires: binutils, redhat-rpm-config, hmaccalc, gcc-c++
 BuildRequires: net-tools, hostname, bc, elfutils-devel
 BuildRequires: dwarves
 BuildRequires: python3-devel

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[OS-BUILD PATCH 3/3] [redhat] Add dist-buildreq-check for binary rpm builds

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus 

[redhat] Add dist-buildreq-check for binary rpm builds

By simplifying the package check to srpm only, we neglect to check
the packaging for local binary rpm building.  Add that Makefile
dependency for those cases.

Signed-off-by: Don Zickus 

diff --git a/redhat/Makefile b/redhat/Makefile
index blahblah..blahblah 100644
--- a/redhat/Makefile
+++ b/redhat/Makefile
@@ -573,8 +573,10 @@ dist-test-patch: generate-testpatch-tmp
 do-rpmbuild: dist-sources
$(RPMBUILD) --define "_sourcedir $(SOURCES)" --define "_builddir 
$(RPM)/BUILD" --define "_srcrpmdir $(RPM)/SRPMS" --define "_rpmdir $(RPM)/RPMS" 
--define "_specdir $(RPM)/SPECS" --define "dist $(DIST)" $(RPMBUILDOPTS) 
$(RPM)/SOURCES/$(PACKAGE_NAME).spec
 
+do-rpmbuild-binary: dist-buildreq-check do-rpmbuild
+
 dist-all-rpms: RPMBUILDOPTS=--target $(MACH) -ba
-dist-all-rpms: do-rpmbuild
+dist-all-rpms: do-rpmbuild-binary
 
 dist-srpm: RPMBUILDOPTS=--nodeps -bs
 dist-srpm: do-rpmbuild
@@ -584,19 +586,19 @@ dist-srpm-gcov: BUILDOPTS+=+gcov
 dist-srpm-gcov: dist-srpm
 
 dist-rpms: RPMBUILDOPTS=--target $(MACH) -bb
-dist-rpms: do-rpmbuild
+dist-rpms: do-rpmbuild-binary
 
 dist-kernel-%:
-   RPMBUILDOPTS="--target $(MACH) --with $* -bb" make do-rpmbuild
+   RPMBUILDOPTS="--target $(MACH) --with $* -bb" make do-rpmbuild-binary
 
 dist-prep: RPMBUILDOPTS=--nodeps --target noarch -bp
 dist-prep: do-rpmbuild
 
 dist-perf: RPMBUILDOPTS=--without up --without smp --without zfcpdump 
--without debug --without doc --without headers --without  --without doc 
--without debuginfo --target $(MACH) -bb
-dist-perf: do-rpmbuild
+dist-perf: do-rpmbuild-binary
 
 dist-rpm-baseonly: RPMBUILDOPTS=--target $(MACH) --without debug --without 
debuginfo --without vdso_install --without bpftool --without perf --without 
tools -bb
-dist-rpm-baseonly: do-rpmbuild
+dist-rpm-baseonly: do-rpmbuild-binary
 
 # unless you know what you're doing, you don't want to use the next four ones
 dist-release-finish: setup-source

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[OS-BUILD PATCH 0/3] Implement better SRPM BuildRequires check

2022-05-18 Thread Don Zickus (via Email Bridge)
From: Don Zickus on gitlab.com
Merge Request: https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1786

Developers want to build a srpm locally and submit it to a build server like
Koji.  The current process checks the BuildRequires in the kernel.spec file to
ensure all the right packages are there.  However, that list includes too many
packages.  Reduce the list to the bare minimum.

In addition cleanup the Makefile and utilize the original buildreqs-check
target better.

Signed-off-by: Don Zickus 

---
 redhat/Makefile |  33 -
 redhat/genspec.sh   |   1 +
 redhat/kernel.spec.template |   7 ---
 3 files changed, 29 insertions(+), 12 deletions(-)
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure