-
Notifications
You must be signed in to change notification settings - Fork 85
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
Updates for "dump diff" script. #2347
Conversation
larsbrinkhoff
commented
Dec 10, 2024
- Some cleanup/refactoring.
- Output an xfile to delete files that were removed between the two tapes.
- Make it posixier.
c5c165b
to
9ef47c9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not having much luck on macOS with this version of dump-diff.sh. I'm getting:
/Users/eswenson/ITS/ws/its/build/dump-diff.sh: line 98: !: command not found
/Users/eswenson/ITS/ws/its/build/dump-diff.sh: line 98: !: command not found
/Users/eswenson/ITS/ws/its/build/dump-diff.sh: line 98: !: command not found
/Users/eswenson/ITS/ws/its/build/dump-diff.sh: line 98: !: command not found
/Users/eswenson/ITS/ws/its/build/dump-diff.sh: line 98: !: command not found
head: illegal line count -- -1
?Open failure: No such file or directory
?Open failure: No such file or directory
This is after having shown me a bunch of NEW FILEs.
That |
Yes, I found that, removed the \ and got this:
What does a line count of -1 mean? Should that be 1? Did someone try to convert -1 to -n 1? |
I changed the head lines to:
And now the script runs to completion. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove the \ before the ! in line 98. Please fix the "head" command to not specify a negative line offset. No need for the +2 in the tail request -- just 2 is sufficient.
9ef47c9
to
e83806f
Compare
Thanks for checking on mac. I fixed |