[Users] vzstats 0.2.1 debianization

[Kir Kolyshkin]

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.

>
>> 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