Hi Drew,
please see my comments below. They are all
related to 7094123, other bug fixes look
good to me.
Thank you,
Jan
general comment
---------------
I am not quite sure that suggested fix will correctly work
in all cases without changing related code handling
unconfiguration.
In particular, milestone-unconfig (responsible for removing
old SC profiles) seems to remove only profiles residing directly
in /etc/svc/profile/site/ directory, but doesn't remove
subdirectories, as those may contain user site profiles - those
should not be touched by sysconfig:
...
#
# Preserve profiles in the /etc/svc/profile/site directory
# which customize configuration process. All other profiles in that
# directory are removed. However profiles located in sub-directories
# of /etc/svc/profile/site are not altered. The command
# "find $SMF_PROFILE_SITE_DIR/* -prune" effectively equates to max
# depth 1, so only occurs in the $SMF_PROFILE_SITE_DIR.
#
find $SMF_PROFILE_SITE_DIR/* -prune -a ! -type d \
-a ! -name $CONFIG_XML ! -name $UNCONFIG_XML | xargs rm -f
...
I assume that problem will disappear with SC groupings project,
when it introduces different layout of SC profiles.
system-config/__init__.py
-------------------------
801-804 - this code cleans up temporary profile directory.
I am wondering if this needs some adjustment, as it currently
deletes only regular files. Perhaps we could just remove the
whole dir by os.rmtree(custom_profile_dir), then followed
by os.mkdir(custom_profile_dir).
838 - Looking at doc, os.path.split(options.profile)[1])))
would evaluate to empty string if options.profile contained
trailing slash. Do we need to somehow account for that case ?
On 05/07/12 18:31, Drew Fisher wrote:
Good morning!
Could I please get a few eyes on a webrev for the following CRs:
7055372 <http://monaco.us.oracle.com/detail.jsf?cr=7055372> Installation
with very long host name failed at checkpoint apply-sysconfig
7074867 <http://monaco.us.oracle.com/detail.jsf?cr=7074867> truncated string
"System Configuration Interactive (SCI) tool will be" extracted to message source
7094123 <http://monaco.us.oracle.com/detail.jsf?cr=7094123> 'sysconfig configure
-c<dir>' flattens out given<dir> tree
7166308 <http://monaco.us.oracle.com/detail.jsf?cr=7166308> sysconfig
LDAP profile screen should default to profile name field not the 2nd field on screen
https://cr.opensolaris.org/action/browse/caiman/drewfish/sysconfig_fixes/webrev/
Testing done:
7055372 - verified the example hostname in the CR correctly toggles
back to 'astring'
7094123 - verified a similar example from the CR behaves correctly all
the way through sysconfig configure
7166308 - verfied the profile_name field is highlighted first on the
LDAP screen
Thanks!
-Drew
_______________________________________________
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
