Darren, while you're in here:

IPS fixed 18835 a couple of weeks ago, so flush-content-cache-on-success now defaults to true, so we should remove that setting from zone_default.xml.src.

The remaining comment in ai_manifest.xml.src regarding languages is really awkward. How about "Language localizations are selected using package facets. The below stanza selects the languages that are standard on Solaris installation."

Dave

On 09/08/11 03:23, Darren Kenny wrote:
Ehm, yep, shouldn't have done this at such a late hour ;)

The updated review is at:

        https://cr.opensolaris.org/action/browse/caiman/dkenny/7087641-1/webrev/

Thanks,

Darren.

On 08/09/2011 00:18, Ethan Quach wrote:
Hey Darren,

Per the eval, doesn't zone_default.xml need to be modified as well?

-ethan


On 09/07/11 15:05, Darren Kenny wrote:
Hi,

Could I please get a review and approval of the fix for:

        7087641 Need locale facet defaults (and samples) in AI

The webrev is at:

        https://cr.opensolaris.org/action/browse/caiman/dkenny/7087641/webrev/

It essentially just adds the facets specification to the distributed default.xml
and ai_manifest.xml files.

I have not used XInclude since this would require that the file also be
downloaded to the AI client itself, which is not a change likely to be made at
this point.

Thanks,

Darren.
_______________________________________________
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

Reply via email to