[Rpm-maint] [rpm-software-management/rpm] rpmfc: Return failures in rpmfcExec (#729)

[Igor Gnatenko]

ignatenkobrain commented on this pull request.



> @@ -444,17 +444,12 @@ int rpmfcExec(ARGV_const_t av, StringBuf sb_stdin, StringBuf * sb_stdoutp,
     }
     ec = getOutputFrom(xav, buf_stdin, buf_stdin_len, sb,
 		       failnonzero, buildRoot, dup);
-    if (ec) {
-	sb = freeStringBuf(sb);
-    }

to be honest, I just felt weird that it was freed when failnonzero.. I did not have any specific case why I wanted it to be different.

SO I suppose if we change behavior, we should change behavior of return code because now it is just 0 - ok, 1 - fail. while in some cases you want to get actual return code. 

So I propose to merge it without behavior change so we don't have to fix anything else on the way. And when we decide to change behavior, then we will look at it again.

WDYT?

-- 
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/729#discussion_r289812788
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20190603/b02918fe/attachment.html>