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

Enhancements to sh template #776

Merged
merged 1 commit into from Oct 25, 2014

Conversation

Projects
None yet
2 participants
@gerbercj
Contributor

gerbercj commented Oct 3, 2014

This PR includes two significant changes:

  • If this version of the code is already in current:
    • If not forced, do not install the code
    • If forced, rename the old directory and write out a new one
  • Ignore functions in environment when saving .install-metadata, due to an issue with FPM 1.25.29-31 and Bash 4.3.27
Chris Gerber
Enhancements to sh template
This PR includes two significant changes:

* If this version of the code is already in current:
  * If not forced, do not install the code
  * If forced, rename the old directory and write out a new one

* Ignore functions in environment when saving .install-metadata, due to an issue with FPM 1.25.29-31 and Bash 4.3.27

@jordansissel jordansissel added the O(1) label Oct 25, 2014

jordansissel added a commit that referenced this pull request Oct 25, 2014

@jordansissel jordansissel merged commit c453f6a into jordansissel:master Oct 25, 2014

@jordansissel

This comment has been minimized.

Owner

jordansissel commented Oct 25, 2014

Woooot!

prof-milki pushed a commit to prof-milki/xpm that referenced this pull request Dec 18, 2014

jls
Merge pull request jordansissel#776 from Tapjoy/feature/sh_template_e…
…nhancements

Enhancements to sh template

prof-milki pushed a commit to prof-milki/xpm that referenced this pull request Dec 27, 2014

jordansissel added a commit that referenced this pull request Apr 24, 2015

jordansissel added a commit that referenced this pull request Jun 20, 2016

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