Sue:
For the default.xml, I noticed the following:
1. There used to be the following to allow DDU searchall but it was
comment out. If user want to use searchall, they can uncomment it:
<!--
28 Add missing driver packages to a booted install image so an
29 installation can complete. Add packages to target as well.
30<searchall> searches and installs from configured repo.
31 -->
32<!--
33 Uncomment before using
34<ai_add_drivers>
35<searchall/>
36</ai_add_drivers>
37 -->
Shouldn't we update it and still comment it out in the new default.xml:
<!--
Add missing driver packages to a booted install image so an
installation can complete. Add packages to target as well.
<searchall> searches and installs from configured repo.
-->
<!--
Uncomment before using
<add_drivers>
<search_all>
<source>
<publisher name="opensolaris.org">
<origin name="http://ipkg.sfbay/dev"/>
</publisher>
</source>
</search_all>
</add_drivers>
-->
2. There used to be comments to tell user how to install a particular build
and it was very useful for testing as follow:
43<!--
44 By default the latest build available, in the specified IPS
45 repository, is installed.
46 If another build is required, the build number has
47 to be appended to the 'entire' package in following
48 form:
49
50<pkg name="pkg:/[email protected]#"/>
Shouldn't we still include it and update it as follow:
43<!--
44 By default the latest build available, in the specified IPS
45 repository, is installed.
46 If another build is required, the build number has
47 to be appended to the 'entire' package in following
48 form:
49
50<name>pkg:/[email protected],build#</name>
51 -->
51 -->
On 08/17/10 06:59 AM, Sue Sohn wrote:
On 08/17/10 05:09, Sue Sohn wrote:
On 08/16/10 14:32, Dave Miner wrote:
On 08/16/10 03:55 PM, Sue Sohn wrote:
Thanks to everyone for your comments. Below are pointers to updated
and
incremental webrevs which address those comments. We'd appreciate a
quick
turnaround on this one so that we can finish the review by COB
Wednesday, 8/18.
Webrev:
-------
Full webrev: http://cr.opensolaris.org/~sohn/webrev.ai-schema.v2/
Incr. webrev:
http://cr.opensolaris.org/~sohn/webrev.ai-schema.v2.incr/
Note that the incremental webrev did not pick up the diffs for the
renamed
publish_manifest.py file and instead shows all changes from this
project.
Just some nits in the manpage changes, installadm.1m.txt:
417: s/xml/XML/
Will also change on 279
422,430,431,433,444: s/criteria/criterion/ since these are singular
references.
Fixed.
448: s/More than one/Multiple/
Yes, better.
666: "a criterion"
OK.
Thanks,
Sue
Dave
Just noticed that 688 also needs an update, will be changing that as
well:
SUNWinstalladm-tools -> install/installadm
Sue
_______________________________________________
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
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
--
Sponsor me in the Relay For Life of Mountain View
Sat 10am to Sun 10am, May 22-23, 2010 at Cuesta Park
All proceeds go to American Cancer Society
http://main.acsevents.org/goto/Mary.Ding
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
