[Libvirt-cim] [PATCH] Make sure provider revision is a number
Viktor Mihajlovski
mihajlov at linux.vnet.ibm.com
Wed Oct 2 15:12:10 UTC 2013
On 10/02/2013 03:46 PM, John Ferlan wrote:
...
>
> To "fix" I changed the "(rev, changeset)" to "(rev, str(changeset))"
>
> I adjusted and pushed.
thanks
...
>
> I'm not a "build guru", but I think the libvirt-cim build probably needs
> to generate the values more frequently than just in autoconfiscate.sh
> seeing as that's 'perhaps' run less frequently than someone making
> changes to the git repository and rebuilding...
I agree it is a bit tricky in development environments, in the
long run however I think it's correct to calculate the 'official'
revision only during full (RPM) builds from GIT master.
The problem is that everyone is typically working from private
branches which tend to be 'longer' than master.
A section in a cimtest README explaining the ramifications
might help though...
--
Mit freundlichen Grüßen/Kind Regards
Viktor Mihajlovski
IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Martina Köderitz
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294
More information about the Libvirt-cim
mailing list