Prefix all shell commands with backslashes to ignore aliases #65
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.
I recently began using exa to replace
ls
. To make the switch easier, I aliasedls
:As a result, the command this package uses to get the latest log file (
ls -t
) throws an error instead of listing the sorted files:The package uses the
ls
,head
,tail
, andgrep
shell commands, and they all could potentially be aliased/overwritten on the user's system. This could break the package, and there is no way of knowing because the output is suppressed:This PR simply adds backslashes to the beginning of each shell command so that aliases are ignored and the native shell commands are used instead.
I can manually specify the escaped command with
--file
to get it to work, but it's janky and you'll only know to do it this way if you know why it's throwing the error in the first place:php artisan tail --file="\`\\ls -t | \\head -1\`"
Thanks!