[libvirt PATCH] spec: Explain the BuildRequires on firewalld-filesystem
Ján Tomko
jtomko at redhat.com
Thu Oct 8 11:15:17 UTC 2020
On a Thursday in 2020, Neal Gompa wrote:
>On Thu, Oct 8, 2020 at 7:06 AM Andrea Bolognani <abologna at redhat.com> wrote:
>>
>> It's not immediately obvious why it is needed.
>>
>> Suggested-by: Pavel Hrdina <phrdina at redhat.com>
>> Signed-off-by: Andrea Bolognani <abologna at redhat.com>
>> ---
>> libvirt.spec.in | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/libvirt.spec.in b/libvirt.spec.in
>> index 6d4eef86ad..c0d84c0e75 100644
>> --- a/libvirt.spec.in
>> +++ b/libvirt.spec.in
>> @@ -399,6 +399,7 @@ BuildRequires: rpcgen
>>
>> BuildRequires: libtirpc-devel
>>
>> +# Needed for the %firewalld_reload macro
>> %if %{with_firewalld_zone}
>> BuildRequires: firewalld-filesystem
>> %endif
>> --
>> 2.26.2
>>
>
>I'd suggest putting the comment inside the conditional, but otherwise it's fine.
>
It looks nicer on the outside. And it matches the style in the rest of
the file.
>Reviewed-by: Neal Gompa <ngompa13 at gmail.com>
>
Reviewed-by: Ján Tomko <jtomko at redhat.com>
Jano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20201008/0dd996ac/attachment-0001.sig>
More information about the libvir-list
mailing list