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

Update mac native package for upcoming release #33696

Merged
merged 1 commit into from
Jun 2, 2016
Merged

Update mac native package for upcoming release #33696

merged 1 commit into from
Jun 2, 2016

Conversation

clburlison
Copy link
Contributor

What does this PR do?

Fixes the package build process for the upcoming native mac package.

What issues does this PR fix or reference?

This solves an issue where the package resources and package scripts directories were being added to the root of the install drive. Which is a big no-no.

I also snuck a spelling change into this PR for:

  • xCode to Xcode
  • and capitalize Command Line Tools

to match Apple's official documentation.

Ref: https://developer.apple.com/xcode/ and https://developer.apple.com/library/ios/technotes/tn2339/_index.html

Lastly, added a directory .gitignore so temp files from the OS X package creation are never uploaded to git.

Previous Behavior

previousbehavior

New Behavior

newbehavior

Tests written?

No. However the I test the changes by creating a new package.


Super excited for the focus on OS X.

Thanks!

This solves an issue where the package `resources` and package `scripts` were being added to the root of the install drive.

Change Spelling for:
* xCode to XCode
* and capitalize Command Line Tools

to match Apple's offical documentation.

Ref: https://developer.apple.com/xcode/
and https://developer.apple.com/library/ios/technotes/tn2339/_index.html
@cachedout
Copy link
Contributor

Signed off on by @twangboy

Thanks, @clburlison . This looks good.

@cachedout cachedout merged commit 1d6582b into saltstack:2016.3 Jun 2, 2016
@clburlison clburlison deleted the 2016.3-pkg-fix branch June 2, 2016 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants