Add hash for tmp dir location. #4
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This sets up the tmp directory to extract the archive. The location will be /tmp/tmpx-TOKEN where TOKEN is a hash of the dat archive. This lets us reuse the archive after each run. Fixes solidsnack#3.
|
Building and testing this today. |
|
Instead of moving options processing down into diff --git a/model-scripts/tmpx.sh b/model-scripts/tmpx.sh
index 3898e33..2976b97 100755
--- a/model-scripts/tmpx.sh
+++ b/model-scripts/tmpx.sh
@@ -52,11 +52,12 @@ opts() {
"$cmd" "$@"
}
go () {
- unpack_env > ./env
- unpack_run > ./run ; chmod ug+x ./run
- mkdir dat
+ [ -f env ] || unpack_env > ./env
+ [ -f run ] || unpack_run > ./run
+ chmod ug+x ./run
+ [ -d dat ] || mkdir dat
cd dat
- unpack_dat
+ [ -d dat ] || unpack_dat
if $run
then
( . ../env && ../run "$@" ) |
|
Ok so tell me your preference on option names. I had a little bit of trouble changing the args to work. I just hacked in the option as "-rm." which might not really make sense because it really doesn't have anything to do with "removing". Modifying the n-tuples for a whole new arg type were causing me problems so this just seemed easier. |
Might not be the best name but it was the easiest to link it into the -rm* family of options. The third Bool in the triple stands for using the same files if they already exit. This should not affect behavior of anything else besides what’s changed in tmpx.sh.
|
This has been released as |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This sets up the tmp directory to extract the archive. The location
will be /tmp/tmpx-TOKEN where TOKEN is a hash of the dat archive. This
lets us reuse the archive after each run.
Fixes #3.
Depending on support for this, I can make it optional so we can keep the old generated TMPDIR method.