On Sun, May 12, 2013 at 4:53 AM, Kir Kolyshkin <[email protected]> wrote: > On 05/11/2013 05:36 PM, CoolCold wrote: >> >> On Sun, May 12, 2013 at 3:47 AM, Kir Kolyshkin <[email protected]> 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 <[email protected]> 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 >>> [email protected] >>> https://lists.openvz.org/mailman/listinfo/users >> >> >> >> -- >> Best regards, >> [COOLCOLD-RIPN] >> _______________________________________________ >> Users mailing list >> [email protected] >> https://lists.openvz.org/mailman/listinfo/users > > > _______________________________________________ > Users mailing list > [email protected] > https://lists.openvz.org/mailman/listinfo/users -- Best regards, [COOLCOLD-RIPN] _______________________________________________ Users mailing list [email protected] https://lists.openvz.org/mailman/listinfo/users
