Hi Jan --
Here are my comments:
pre_pkg_img_mod.py
-----------------------------
line 450: Why was "etc/system" removed from the save list?
set_lang
---------------
How is the default lang set? I assume it occurs in smf; just curious
more than anything.
system-config/__init__.py
---------------------------------
line 865: the man page shows
-c config_profile.xml |dir
instead of
-c profile | dir
we should be consistent with the man page.
Other than that, it looks good.
Thanks,
ginnie
On 07/26/11 10:19 AM, Jan Damborsky wrote:
Hi Ginnie,
could I please ask you to review fixes for following SC bugs ?
7061367 Install technologies should benefit from PSARC/2011/097 for
purposes of locale(5) configuration
7068682 sysconfig(1m) Usage message incomplete: [-c profile] should
be [-c profile | dir]
7064593 "sysconfig configure -c <dir>" should fail if <dir> is empty
7068646 sysconfig(1m) should refuse SC profiles without .xml suffix
6997850 Stop populating hosts(4) file once CR 6996436 is addressed
7071116 terminal type does not reflect configuration in SC profile
7068462 slim unit test failures after sysconfig putback
7069484 pep8 errors in sysconfig/__init__.py
7065611 Add locale support to sc profile
webrev:
https://cr.opensolaris.org/action/browse/caiman/dambi/sc-bugs/webrev-0726/
As usual, other comments are more than welcome :-)
I would appreciate if comments could be provided before 7/28 Th COB,
so that these changes can appear in 172 (I am ooto next week).
Thank you very much,
Jan
testing done:
* built LiveCD and Sparc TI ISOs with modified Distro Constructor
containing all changes
- 7061367 - during LiveCD boot, selected Spanish on text language
screen - verified that LiveCD
desktop came up in Spanish and that installed system came up with
es_ES.UTF-8 configured as default
locale.
- verified with Sparc text installer that selected locale
(en_US.UTF-8) was correctly
configured on installed system.
- 6997850 - verified that loopback address is associated with
correct hostname in hosts(4)
file on install media as well as on installed system
- 7071116 - verified that terminal type is correctly configured on
installed system
* process of building other install media is still in progress - will
be tested before push
* ran sysconfig CLI in different scenarios to verify fixes for
7068682, 7064593, 7068646
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
