bayesimpact / bayesimpact/bayes-developer-setup
mktemp in git-review throws a usage error on OS X Yosemite
- Dominant language
- Python
- Stars
- 2
- Forks
- 5
- PR merge metrics
- No merged PRs in 30d
Description
On OS X Yosemite 10.10.5, the following line of git-review throws an error:
`readonly MESSAGE_FILE=$(mktemp)`
`usage: mktemp [-d] [-q] [-t prefix] [-u] template ...
mktemp [-d] [-q] [-u] -t prefix`
Apparently Yosemite doesn't allow for running mktemp with no arguments. This problem and a workaround is discussed [on this issue](https://github.com/elasticdog/transcrypt/issues/34).
Not a major issue as it looks like most other OS versions handle this.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start at the git-review entry point containing `readonly MESSAGE_FILE=$(mktemp)` and reproduce the command on OS X Yosemite. Compare the reported workaround from the linked transcrypt issue, then verify that git-review no longer emits the mktemp usage error on that platform.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- git, macos, shell
- Domain
- cli, operating-systems
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100