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 > [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
