New webrev at:
http://cr.opensolaris.org/~bpytlik/ips-3718-v2/

Shawn, I took several looks at those lines whose formatting you thought were strange, and I didn't see anything wrong with them. Would you take a look at this webrev and see if the concerns are still there, and if so, what's wrong?

Other changes include adding check_be_name as a static method of the API class and adding more text to the man page.

Brock

Brock Pytlik wrote:
http://cr.opensolaris.org/~bpytlik/ips-3718-v1/

Which hopes to fix

3718 plan_update_all needs to have an optional argument for the new BE name

http://defect.opensolaris.org/bz/show_bug.cgi?id=3718

2883 image-update should have option to choose new BE name
http://defect.opensolaris.org/bz/show_bug.cgi?id=2883

3748: beCopy should be called with no arguments, then the clone renamed
http://defect.opensolaris.org/bz/show_bug.cgi?id=3748

It adds a new optional parameter to the plan_update_all call of the API which allows specification of the name the cloned BE is given.

Thanks for taking a look,
Brock

_______________________________________________
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