LGTM

Michal

jmr wrote:
Respin based on comments from Darren Kenny against the gate.

Thanks Darren.

- Quotes around "$ret" to try minimise adverse code executing
- Removal of what appears to me to be an excess download() call.
- Addition of 2>/dev/null to gconftool call since I've seen this cause problems
 before in some GNOME scripts.

http://cr.opensolaris.org/~jmr/pm_6970_startpage_updateable_gate_13Mar_1030am/

JR

jmr wrote:
Please use:
http://cr.opensolaris.org/~jmr/pm_6970_startpage_updateable_gate_12Mar_7pm

I had left a debug setting for default locale in the previous one.

JR


jmr wrote:
Hi,

This webrev provides a means to update the Packagemanager Start Page files dynamically.

As part of the update_refresh.sh script that is run 5 times daily (0,9,12,18,21) to update the catalog, a wget has been added to fetch a start page tar gzip file (startpagebase-<locale prefix>.tar.gz) from a URL specified in the PM gconf key (/apps/packagemanager/preferences/startpage_url). This is unpacked to a gui cache (/var/pkg/gui/cache/startpagebase/<locale>) and picked up on startup by the PM.

6970 Start page needs to be updateable

http://cr.opensolaris.org/~jmr/pm_6970_startpage_updateable_gate_12Mar_6pm/

JR
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss


_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to