Brock Pytlik wrote:
> Michal Pryc wrote:
>> Hi,
>> Small fixes for few minor bugs, webrev:
>>
>> http://cr.opensolaris.org/~migi/30_10_2008_bugfixes_3203_3202_4018_1579_2121/
>>
>>
>> Fixes:
>> [4018]
>> Package Manager - Window width bug
>> http://defect.opensolaris.org/bz/show_bug.cgi?id=4018
>>
>> [3202]
>> Add dialog for setting name for Boot Environment
>> http://defect.opensolaris.org/bz/show_bug.cgi?id=3202
>>
> I would've expected this to be put off for the same reasons that giving
> the user the ability to choose the name for their new BE is not going
> into this release. Mainly that more work needs to be done to understand
> what is, and is not a vaild BE name, and how that interacts with what
> ZFS thinks is a valid file system name. The recent issue with
> authorities demonstrates some of the problems we can get into if we're
> not careful about these kinds of things. I wasn't able to find the spot
> in the code where this is added, so can you point me to the lines where
> this information is taken from the user and then validated to make sure
> it's a valid BE name? I would argue that if this functionality is in the
> UM now, it should be removed as well.
Brock,
It's my mistake to not write more about this fix. We have decided to NOT 
have the dialog for setting name for Boot Environment, but the xDesign 
wanted to have dialog informing users that new BE will be created. 
Mostly because this is something new to most of the people not familiar 
with OpenSolaris, so following wireframes:
http://xdesign.sfbay.sun.com/projects/solaris/subprojects/package_mngt/UI_specs/ui_spec_phase2/html-mockup/52_update_confirm.htm

This was implemented, but setting the BE name is actual not being 
implemented and it's for futher discussion as I've written in the Bug desc:
http://defect.opensolaris.org/bz/show_bug.cgi?id=3202#c2


best
Michal

>
> Brock
>> [3203]
>> Add dialog for update error - Not enought free space
>> http://defect.opensolaris.org/bz/show_bug.cgi?id=3203
>>
>> [1579]
>> Details text field should be anchored to bottom
>> http://defect.opensolaris.org/bz/show_bug.cgi?id=1579
>>
>> [2121]
>> Install/Upgrade Check dialog needs to be more informative
>> http://defect.opensolaris.org/bz/show_bug.cgi?id=2121
>>
>>
>>
>> best
>> Michal
>> _______________________________________________
>> 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