Sarah:
I did not see it in the following:
http://cr.opensolaris.org/~sohn/webrev.ai-schema.v2/usr/src/cmd/auto-install/default.xml.html
There was no comments about entire @ certain build number at all. It
used to be in the old default.xml.html
On 08/17/10 10:19 AM, Sarah Jelinek wrote:
HI Mary,
Thank you for the review. Comments inline...
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 -->
I will add this back in.
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]#"/>
This is in the default.xml and ai_manifest.xml. Maybe I am
misunderstanding what you are referring to?
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 -->
thanks,
sarah
****
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
_______________________________________________
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
