[syslog-ng] Re: minor patch (maybe)

[Dhawal Doshy]

Balazs Scheidler wrote:
> On Wed, 2007-02-07 at 15:01 +0530, Dhawal Doshy wrote:
>> Hello List,
>>
>> Am currently testing 2.0.2.. while building the RPM i noticed a broken 
>> dependency i.e. that of libevtlog-devel (which i though should be 
>> eventlog-devel). Here is a small patch, i would appreciate if the 
>> developers think this is not appropriate.
>>
>> [me at myhost rpms]# diff -u syslog-ng-2.0.2/syslog-ng.spec.bb 
>> syslog-ng.spec.bb
>> --- syslog-ng-2.0.2/syslog-ng.spec.bb   2007-01-29 14:22:23.000000000 +0530
>> +++ syslog-ng.spec.bb   2007-02-03 18:23:30.000000000 +0530
>> @@ -12,7 +12,7 @@
>>   Packager: Tamas Pal <folti at balabit.com>
>>   Vendor: Balabit IT Ltd.
>>   BuildRoot: %{_tmppath}/%{name}-root
>> -BuildRequires: bison, flex, gcc-c++, pkgconfig, glib2-devel, 
>> libevtlog-devel
>> +BuildRequires: bison, flex, gcc-c++, pkgconfig, glib2-devel, eventlog-devel
>>   Provides: syslog
>>   #BuildConflicts:
>>   #Exclusivearch: i386
> 
> hmm.. eventlog generates libevtlog-dev which provides libevtlog-devel.
> Why do you think the BuildRequires line is incorrect?
> 
> And btw, the spec.bb file is used by our internal build system, thus it
> might contain some BalaBitisms :) like the package naming which follows
> Debian's naming scheme because the build system works primarily with
> Debian packages.

google failed to turn up any references for libevtlog-devel :-( neither 
did any of the repositories mention an existence for it. However i did 
have a eventlog-devel rpm installed on the server which provides a 
libevtlog.a and hence the thought. In any case is this understanding 
correct for a centos box?

Also on the spec.bb, i unsuccessfully created a custom spec file based 
on Tony Earnshaw's post and wasn't quite satisfied till i discovered 
this one in the source itself. Works quite well i must say ;-)

Thanks for all your work on syslog-ng, it is a wonderful product. We are 
working on replacing syslog on all our servers pretty soon.

- dhawal