[OS-BUILD PATCH] Fix dist-srpm-gcov

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Don Zickus <dzickus@xxxxxxxxxx>

Fix dist-srpm-gcov

make dist-srpm-gcov failed to do the expected thing.  It appears changes
from commit 2d525ab424f695 forced a second run of dist-sources.  This
second run regenerated the kernel.spec without the gcov pieces because
the env variables were not defined.

Fix this by setting the BUILDOPTS variable to global for the target.
Then simplify the target by leveraging the dist-srpm target.

Now dist-srpm-gcov just sets BUILDID and BUILDOPTS and then calls
dist-srpm to do the expected thing.

Tested by running 'make dist-srpm-gcov' and checking the resulting
redhat/rpm/SOURCES/kernel.spec file for the '_with_gcov 1' definition.

Signed-off-by: Don Zickus <dzickus@xxxxxxxxxx>

diff --git a/redhat/Makefile b/redhat/Makefile
index blahblah..blahblah 100644
--- a/redhat/Makefile
+++ b/redhat/Makefile
@@ -306,9 +306,9 @@ dist-all-rpms: dist-sources do-rpmbuild
 dist-srpm: RPMBUILDOPTS=--nodeps -bs
 dist-srpm: dist-sources do-rpmbuild
 
-dist-srpm-gcov:
-	make dist-sources BUILDID=".gcov" BUILDOPTS="+gcov"
-	RPMBUILDOPTS="--nodeps -bs" make 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

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1430
_______________________________________________
kernel mailing list -- kernel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to kernel-leave@xxxxxxxxxxxxxxxxxxxxxxx
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@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux