makepkginfo installing old(?) version of ManagedInstalls.plist about munki HOT 4 CLOSED

From [email protected] on March 03, 2010 10:04:30

Two problems here:

1. makepkginfo probably shouldn't cause anything to be written to /Library/Preferences. What's happening is
   that it's not seeing the file, falling back to default values for everything, and then writing out a file with those
   default values. The original intent was to make it easier for new munki admins, so they wouldn't have to create
   a preference file from scratch, but could instead edit an existing one.

2. The example /Library/Preferences/ManagedInstalls.plist should be updated and the documentation should
   be as well; the values that appear in the /Library/Preferences/ManagedInstalls.plist in the submitted scenario
   are the current munki defaults.

from munki.

From [email protected] on March 03, 2010 10:54:28

Re: #1) I agree that makepkginfo shouldn't write anything to /Library/Preferences.
Using the defaults would make sense if it can't find ManagedInstalls.plist.

Re: #2) I only noticed that something was off by going through the Bootstrapping
section of the GettingStartedWithMunki page in the wiki. There, it says to "Change the
ManifestURL" which wasn't in the version written out by makepakginfo. Is that setting required anymore?

from munki.

From [email protected] on March 03, 2010 10:58:33

I've updated the example ManagedInstalls.plist and the associated READ ME. Let me know if that clears things up
a bit.

Still considering what to do about writing out the default defaults - it's certainly not needed for munki to
function, and definitely shouldn't happen when running makepkginfo, but might be helpful for new munki
admins once they get to the point of running managedsoftwareupdate so they can SEE the defaults if they haven't
put a site-specific ManagedInstalls.plist in place.

Status: Started

from munki.

From [email protected] on March 03, 2010 12:17:19

made some changes to munkicommon.py. Previously, just importing munkicommon would call
munkicommon.prefs(), which had the side effect of writing out the prefs file if it did not previously exist. That's
now been changed; one must call a function within munkicommon that actually needs access to the prefs to call
munkicommon.prefs() now. Since makepkginfo does not call any functions within munkicommon that need
access to the prefs, makepkginfo no longer writes a new /Library/Preferences/ManagedInstalls.plist if one did
not previously exist.

Status: Fixed

from munki.