Michael Schwendt wrote:
> Agreed. It's a trade-off. Guards aren't bad, but in this case their
> benefit is questionable. It probably doesn't work completely anyway, since
> if the build framework uses Autotools, there likely are no pregenerated
> Makefiles, and only a successful run on the configure script will generate
> them.

Well, the "exit" in the guard would fail the build even before that, whereas 
the "%configure || :" would just succeed if %configure succeeds, and then 
the subsequent "make" would run too. (That said, that's not necessarily a 
problem, it just makes the "|| :" redundant. Failing %configure would fail 
"make" too anyway, for the reason you describe.)

What the guard does not catch is if the package only ships configure.ac and 
expects you to run autoconf yourself, or if it uses CMake or some other 
configury tool. But then the "make" will likely fail anyway for the reason 
you describe.

        Kevin Kofler

-- 
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel
Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct

Reply via email to