Re: [Libvirt-cim] [PATCH] [TEST] Fix CIM_REV to handle non integer strings
>
> -CIM_REV = int(os.getenv("CIM_REV"))
> +rev = os.getenv("CIM_REV").strip("+")
> +if rev.isalnum():
>
Are we planning to use alphabets in the REV string, if yes then the I
say +1 for the changes.
If the REV string does not contain alphabets then we can use isdigit()
instead.
Thanks and Regards,
Deepti.
Good point. I don't expect there to be any alphabetic characters. I'll
send a new patch using isdigit().
Thanks!
--
Kaitlin Rupert
IBM Linux Technology Center
kaitlin(a)linux.vnet.ibm.com