[Rpm-maint] [rpm-software-management/rpm] Make %{buildsubdir} usable without being a side effect of %setup. (#860)

Panu Matilainen notifications at github.com
Wed Oct 9 08:32:52 UTC 2019


pmatilai commented on this pull request.



> @@ -345,7 +349,7 @@ static int doSetupMacro(rpmSpec spec, const char *line)
     }
 
     if (!createDir) {
-	rasprintf(&buf, "cd '%s'", spec->buildSubdir);
+	buf = rpmExpand("%{?buildsubdir:cd '%{buildsubdir}'}", NULL);

I mean, this is inside %setup macro, %buildsubdir is always defined there. The special case of build-in-place jumps over this whole section.

-- 
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/860#discussion_r332889486
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20191009/d6a47731/attachment.html>


More information about the Rpm-maint mailing list