Skip to content
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

v2.1 #21

Merged
merged 13 commits into from Jun 9, 2017
Merged

v2.1 #21

merged 13 commits into from Jun 9, 2017

Conversation

kc9wwh
Copy link
Owner

@kc9wwh kc9wwh commented Jun 9, 2017

Checks if installer is already present
Removes Script & Launch Daemon if System Checks are not passed
Fixed Issue #14
Fixed Issue #15

Joshua Roskos and others added 13 commits March 10, 2017 13:19
Pre-Release! Not Fully Tested!

Consolidated installer paths to variable.
Added use of Extension Attribute to check if machine already has copy
of installer.
Added logic to support EA.
First Boot script will try to remove both file locations.
Not the ideal way to solve this problem, but it works.  The ideal solution would simply not create the scripts unless the process validates.
This allows use of existing Apple cached installer, and checks the
version. If the version is < $version, run $download_trigger.
Clarified comments for variables. Converted tabs to spaces. Removed
agreetolicense.
Fixed example to include /Applications, used full paths to echo, rm,
and jamf commands. Quoted $OSInstaller.
Use existing OS installer
Updated composer screenshot
Added mention of custom-trigger in readme.md
Removes cleanupOSInstall if OS install is canceled.
Converted tabs to spaces
Adjusted removal of postInstall script and launch daemon
@kc9wwh kc9wwh self-assigned this Jun 9, 2017
@kc9wwh
Copy link
Owner Author

kc9wwh commented Jun 9, 2017

v2.1
Checks if installer is already present - @patgmac
Removes Script & Launch Daemon if System Checks are not passed - @magnusviri
Fixed Issue #14 - @magnusviri
Fixed Issue #15 - @patgmac

@kc9wwh kc9wwh merged commit b7d58b1 into master Jun 9, 2017
@kc9wwh kc9wwh deleted the v2.0 branch May 2, 2018 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants