xcat2 / xcat-inventory

An inventory tool for xcat cluster
8 stars 16 forks source link

fix xcat-inventory --version print #VERSION_SUBSTITUTE# #134

Closed bybai closed 5 years ago

bybai commented 5 years ago

fix xcat-inventory --version print #VERSION_SUBSTITUTE# for https://github.com/xcat2/xcat-inventory/issues/133

UT:

# ./build-ubuntu -c xcat-inventory
###################################
# Building xcat-inventory package #
###################################
###################################
# Building xcat-inventory package #
###################################
VERINFO 0.1.5 (git commit 7ae3b70d6f4039f83870766d94d9ee574b58ce76)
dpkg-buildpackage: warning: using a gain-root-command while being root
dpkg-buildpackage: source package xcat-inventory
dpkg-buildpackage: source version 0.1.5-c21
dpkg-buildpackage: source distribution UNRELEASED
... ...
dpkg-deb: building package 'xcat-inventory' in '../xcat-inventory_0.1.5-c21_all.deb'.
pwd
/root/baiyuan/xcat-inventory/xcat-inventory
 dpkg-genchanges  >../xcat-inventory_0.1.5-c21_ppc64el.changes
dpkg-genchanges: warning: the current version (0.1.5-c21) is earlier than the previous one (2.9.13-1)
dpkg-genchanges: including full source code in upload
 dpkg-source --after-build xcat-inventory
dpkg-buildpackage: full upload; Debian-native package (full source is included)
/root/baiyuan/xcat-inventory

# dpkg -i xcat-inventory_0.1.5-c21_all.deb
(Reading database ... 108222 files and directories currently installed.)
Preparing to unpack xcat-inventory_0.1.5-c21_all.deb ...
Unpacking xcat-inventory (0.1.5-c21) over (0.1.5-c21) ...
Setting up xcat-inventory (0.1.5-c21) ...

# dpkg -l |grep xcat-inventory
ii  xcat-inventory                       0.1.5-c21                                  all          xcat inventory

# xcat-inventory --version
0.1.5 (git commit 7ae3b70d6f4039f83870766d94d9ee574b58ce76)
zet809 commented 5 years ago

Hi, @bybai , it is not what I want, I want the ubuntu package name have the same format with RPMs

[root@briggs01 ~]# rpm -aq |grep -i xcat-inventory
xcat-inventory-0.1.5-c13.noarch
zet809 commented 5 years ago

Good to me, I agree to merge, thx @bybai