[naemon-dev] livestatus commit

[Daniel Wittenberg]

It looks like the path also changed, was that on purpose too?

From:
%attr(0644,root,root) %{_libdir}/%{name}/livestatus.o

To:
%attr(0644,root,root) %{_libdir}/%{name}/%{name}-livestatus/livestatus.so

I've got the changed staged to fix up the spec file, probably want to add a
search/replace in the spec to update the config from livestatus.o to
livestatus.so too, but will get this out there first to fix our builds.

Dan


On Mon, Apr 21, 2014 at 2:56 PM, Anton Löfgren <alofgren at op5.com> wrote:

> Naemon doesn't care either way as long as the correct path is configured,
> which it will be even for existing users as long as the post step I mention
> earlier is in place.
>
> I'm not suggesting we start enforcing any kind of convention just for the
> sake of it.
>
> /Anton
>  On 21 Apr 2014 21:41, "Daniel Wittenberg" <dwittenberg2008 at gmail.com>
> wrote:
>
>> I'm not sure I have a strong opinion either way, just that if we decide
>> we want to change it, or even consider it in the future, now is the best
>> time to do it.  Can we have it use either .o or .so and just have
>> livestatus be .so for now?  That would allow backwards compatibility but
>> move in the direction we think we should ?
>>
>> I updated the Fedora build to use .so for now so at least the builds are
>> going again.
>>
>> Dan
>>
>>
>> On Mon, Apr 21, 2014 at 2:18 PM, Sven Nierlein <Sven.Nierlein at consol.de>wrote:
>>
>>> Hey,
>>>
>>> Basically every ndo module uses .o extension. At least ndo,
>>> mod-gearman, dnx and livestatus. Thats all i know.
>>> This change screws every existing naemon installation. Not that
>>> there are so many yet, but we should be very careful when changing
>>> fundamental things. So if the only reason for this change is, that
>>> this is more correct in terms of describing the file content, i'd vote
>>> for keeping things like they are unless we have a good reason
>>> to change that.
>>>
>>>  Sven
>>>
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://www.monitoring-lists.org/archive/naemon-dev/attachments/20140421/9161407b/attachment.html>