Skip to content
This repository has been archived by the owner on Sep 4, 2021. It is now read-only.

Fix release script re-exec #706

Merged
merged 2 commits into from
Jan 7, 2015
Merged

Fix release script re-exec #706

merged 2 commits into from
Jan 7, 2015

Conversation

lmars
Copy link
Contributor

@lmars lmars commented Jan 7, 2015

See commit messages for explanation.

This is so that `git rev-parse HEAD` detects the commit of the new
source, not the source which originally executed the script.

Signed-off-by: Lewis Marshall <lewis@lmars.net>
The trap which removes the newly cloned source does not execute if we
exec

Signed-off-by: Lewis Marshall <lewis@lmars.net>
@titanous
Copy link
Contributor

titanous commented Jan 7, 2015

LGTM

lmars added a commit that referenced this pull request Jan 7, 2015
@lmars lmars merged commit 01b35c7 into master Jan 7, 2015
@lmars lmars deleted the script-release-fix branch January 7, 2015 15:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants