Hi Glenn,
On Wed, 14 Apr 2010, Glenn Lagasse wrote:
Hey Alok,
* Alok Aggarwal ([email protected]) wrote:
Please take a look at this two line change for -
15619 Text installer cannot see disks on a X4150 machine due to missing SUNWaac
driver
http://cr.opensolaris.org/~aalok/15619/
This looks good, though I wonder if you shouldn't use the renamed
package name (driver/storage/aac). I realize the other package names
aren't translated yet, but why introduce more wrong names is my
thinking.
I considered that while making the change but later
on decided to not do it that way for consistency.
Although now I see that the text-installer package has
the new name. I've changed it to driver/storage/aac.
Webrev updated.
Thanks for the review!
Alok
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
