New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix installer postflight #949

Merged
merged 1 commit into from Feb 7, 2018

Conversation

Projects
None yet
3 participants
@MagerValp
Copy link
Contributor

MagerValp commented Jan 30, 2018

Bad quoting of $3 in the installer postflight caused 0600 permissions on the MunkiReport.plist preference file.

Fix installer postflight
Bad quoting of $3 in the installer postflight caused 0600 permissions on the MunkiReport.plist preference file.
@poundbangbash

This comment has been minimized.

Copy link
Contributor

poundbangbash commented Jan 30, 2018

Just another data point.
Prior to running my /Library/Preferences/MunkiReport.plist was

-rw-------  1 root  wheel  2982 Jan 20 23:51 /Library/Preferences/MunkiReport.plist

After running the bash install script it's now

-rw-r--r--  1 root  wheel  2982 Jan 30 17:09 /Library/Preferences/MunkiReport.plist

I also ran it thru creating a pkg. The postinstall script in the pkg looks good and after an install it has the correct permissions on the MunkiReport.plist.

-Eric

@bochoven bochoven merged commit e9881f1 into munkireport:master Feb 7, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment