[Rpm-maint] [rpm-software-management/rpm] Dynamic Spec generation (#1485)

Panu Matilainen notifications at github.com
Thu Feb 4 10:40:13 UTC 2021


@pmatilai commented on this pull request.



> @@ -1118,3 +1146,25 @@ rpmSpec rpmSpecParse(const char *specFile, rpmSpecFlags flags,
 {
     return parseSpec(specFile, flags, buildRoot, 0);
 }
+
+rpmRC parseGeneratedSpecs(rpmSpec spec)
+{
+    
+    ARGV_t argv = NULL;
+    int argc = 0;
+    int i;
+
+    char * specPattern = rpmGenPath("%{u2p:%{_builddir}}", spec->buildSubdir, "__rpmbuild_*.specpart");

"__rpmbuild_*.specpart" still looks quite the eyesore to me. These are not rpm internal things, but files we expect others to produce. I'd think "*.specpart" would be enough to differentiate from other stuff.

Also there's still the option of creating a special directory (".rpmbuild" or such in the builddir) which could be used for such things, including but not limited to debuginfo filelists, to avoid cluttering the main build directory (that gets requested separately every now and then). Such a thing doesn't have to be part of this PR, but should need to be done before a public release.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1485#pullrequestreview-583255087
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20210204/c30547b8/attachment.html>


More information about the Rpm-maint mailing list