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

Second %pre should be %post #822

Merged
merged 1 commit into from Apr 10, 2015

Conversation

Projects
None yet
3 participants
@dlanderson
Contributor

dlanderson commented Dec 9, 2014

Fixes #803

@djhaskin987

This comment has been minimized.

Contributor

djhaskin987 commented Apr 10, 2015

+1. Thanks for fixing this, my bad! :)

@@ -139,7 +139,7 @@ then
fi
<% end -%>
<% if script?(:after_upgrade) or script?(:after_install) -%>
%pre
%post

This comment has been minimized.

@jordansissel

jordansissel Apr 10, 2015

Owner

doh! Good catch.

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

Merge pull request #822 from dlanderson/patch-1
Second %pre should be %post

@jordansissel jordansissel merged commit 847a8d6 into jordansissel:master Apr 10, 2015

@jordansissel

This comment has been minimized.

Owner

jordansissel commented Apr 10, 2015

Eyeballed; lgtm! I didn't test yet, but that's ok. :)

@djhaskin987

This comment has been minimized.

Contributor

djhaskin987 commented Apr 10, 2015

I tested this 👍

@jordansissel

This comment has been minimized.

Owner

jordansissel commented Apr 10, 2015

❤️

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

Merge pull request #822 from dlanderson/patch-1
Second %pre should be %post

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

Merge pull request #822 from dlanderson/patch-1
Second %pre should be %post
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment