Padraig,

Would it not be better to try and get the integer of the last element in the list you have split, as there could be any number of '-' in the name. Otherwise a name like foo-bar-1 would become, foo-bar-10 instead of foo-bar-2.

i = int(list[len(list) - 1])

     311 +                                try:
     312 +                                        i = int(list[1])
313 + proposed_name = self.__construct_be_name(
     314 +                                            list[0], i)
     315 +                                except ValueError:
316 + proposed_name = self.__construct_be_name(
     317 +                                            active_name, 0)


JR


Padraig O'Briain wrote:
The webrev, http://cr.opensolaris.org/~padraig/ips-6986-v2/, fixes
6986 Update Confirm Boot  Env Dialog

The change puts an initial suggestion for the new BE name in the text field.

Padraig
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to