[Users] vzstats 0.2.1 debianization

[CoolCold]

On Sun, May 12, 2013 at 4:53 AM, Kir Kolyshkin <kir at openvz.org> wrote:
> On 05/11/2013 05:36 PM, CoolCold wrote:
>>
>> On Sun, May 12, 2013 at 3:47 AM, Kir Kolyshkin <kir at openvz.org> wrote:
>>>
>>> On 05/11/2013 04:25 PM, CoolCold wrote:
>>>>
>>>> I've updated debianization to match 0.3.2 version, source is available
>>>> on GitHub - https://github.com/CoolCold/vzstats .
>>>> Changes are:
>>>> 1) including manpage
>>>> 2) including bash completion
>>>> 3) added deps for lsb_release
>>>>
>>>> misses:
>>>> 1) manpage does contain wrong paths like /usr/libexec/vzstats/ which
>>>> should be changed to proper Debian ones.
>>>>
>>>> prebuild package for can be downloaded from
>>>> http://repo.coolcold.org/pool/main/v/vzstats/
>>>> or you may add
>>>> deb http://repo.coolcold.org/ squeeze-coolcold main contrib non-free
>>>> in sources.list to add repo ( http://repo.coolcold.org/README.TXT ).
>>>
>>>
>>>
>>> Thanks a lot, I hope some Debian/Ubuntu users will install this.
>>>
>>> For bash_completion, why don't you use something like
>>>      make DESTDIR='$(INSTALL_DIR)' install-bashcomp
>>> from debian/rules?
>>
>> To be honest, I'm not makefiles expert ;( and didn't even thought
>> about doing this way, so used much more simpler 'shell scripts'
>> methods ;)
>
>
> It's just I already have make install* targets, they are developed with
> packaging in mind
> (i.e. DESTDIR is supported). By doing your own install you basically
> duplicate the efforts
> and code, and will have to add stuff if I add stuff.
>
> I just realized I need "install-all" target.
Thanks for hints, I'll try to make rules file to be better, also just
realized need to add "Suggests" (may be even "Recommends") for
bash-completion too.

>
>
>>
>>> Same for other install targets.
>>> For bin location, you can use REPDIR=/usr/share/vzstats/
>>> (you still need to fix vzstats man page and script)
>>>
>>> In fact let me try to have REPDIR configurable...
>>>
>>>
>>>>
>>>> On Wed, May 1, 2013 at 4:21 AM, CoolCold <coolthecold at gmail.com> wrote:
>>>>>
>>>>> Hello!
>>>>> I've read your LJ blog post about vzstats (0.2.1) tool, but looks like
>>>>> it somehow misses Debian packages and debianization (I've found rpm
>>>>> spec only), so I've created initial debianization for it.
>>>>>
>>>>> Debianization is squashed into one commit -
>>>>>
>>>>>
>>>>> https://github.com/CoolCold/vzstats/commit/f2425b5ce97215c3ba4030e3c314a501920658a0
>>>>> , feel free to try it.
>>>>>   From my POV it should work in general, there are no any errors found
>>>>> by lintian, warnings only, but it is not perfect at all - using sed
>>>>> instead of dpatch/quilt and so on.
>>>>> Package building can be done via: git clone ... && cd vzstats &&
>>>>> dpkg-buildpackage -us -uc -sa -si -rfakeroot
>>>>>
>>>>>
>>>>> Also, I've found several issues:
>>>>> 1) osrelease script tryes to find /etc/*-release which is not used in
>>>>> Debian. As solution I think it is possible to use lsb_release as
>>>>> distro-independent way to detect release.
>>>>> 2) vzlist script relyes on some newer vzctl version, for example, my
>>>>> system has backported vzctl 3.0.30.2 , and it doesn't contain output
>>>>> fields like 'layout' and 'vswap' , so it just exits.
>>>>>
>>>>> While saying "Debian" here I meant "Squeeze" release.
>>>>>
>>>>> If anyone interested in, I can put debian package somewhere.
>>>>>
>>>>> --
>>>>> Best regards,
>>>>> [COOLCOLD-RIPN]
>>>>
>>>>
>>>>
>>> _______________________________________________
>>> Users mailing list
>>> Users at openvz.org
>>> https://lists.openvz.org/mailman/listinfo/users
>>
>>
>>
>> --
>> Best regards,
>> [COOLCOLD-RIPN]
>> _______________________________________________
>> Users mailing list
>> Users at openvz.org
>> https://lists.openvz.org/mailman/listinfo/users
>
>
> _______________________________________________
> Users mailing list
> Users at openvz.org
> https://lists.openvz.org/mailman/listinfo/users



--
Best regards,
[COOLCOLD-RIPN]