On 07/20/2011 12:48 AM, Jan Friesse wrote:
> Steven Dake wrote:
>> On 07/19/2011 08:01 AM, Jan Friesse wrote:
>>> Signed-off-by: Jan Friesse <jfrie...@redhat.com>
>>> ---
>>> corosync.spec.in | 2 +-
>>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/corosync.spec.in b/corosync.spec.in
>>> index 37e53ed..823ad3d 100644
>>> --- a/corosync.spec.in
>>> +++ b/corosync.spec.in
>>> @@ -138,7 +138,7 @@ fi
>>> %{_sysconfdir}/dbus-1/system.d/corosync-signals.conf
>>> %endif
>>> %if %{with snmp}
>>> -%(_datadir)/snmp/mibs/COROSYNC-MIB.txt
>>> +%{_datadir}/snmp/mibs/COROSYNC-MIB.txt
>>> %endif
>>
>> does this patch change anything?
>
> Ya, but it's very hard to spot (especially with small/bad fonts, it took
> me a while to notice it too). It changes round brackets ( ) to curly
> bracket { }. First means "execute in shell" (we really don't want to
> execute _datadir" command) and second "expand variable value" (this is
> what we want).
>
>>
>>> %{_initrddir}/corosync
>>> %{_initrddir}/corosync-notifyd
>>
>
Reviewed-by: Steven Dake <sd...@redhat.com>
_______________________________________________
Openais mailing list
Openais@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/openais
