Skip to content
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

'bat cache' still takes precedence over existing files. #666

Closed
eth-p opened this issue Sep 20, 2019 · 2 comments
Closed

'bat cache' still takes precedence over existing files. #666

eth-p opened this issue Sep 20, 2019 · 2 comments
Labels
bug Something isn't working

Comments

@eth-p
Copy link
Collaborator

eth-p commented Sep 20, 2019

Example:

touch cache
touch test
bat cache test

Expected: Both files would be printed.
Actual:

error: Found argument 'test' which wasn't expected, or isn't valid in this context

USAGE:
    bat cache [OPTIONS]

For more information try --help

This still occurs in bat 0.12.1.

@eth-p eth-p added the bug Something isn't working label Sep 20, 2019
@sharkdp
Copy link
Owner

sharkdp commented Sep 21, 2019

I was about to write: "this is the last nail in the coffin for the bat cache subcommand".
But there is a simple fix for this, see #669.

At least the corner cases are getting more involved... but seriously, if someone has a good solution (see discussion in #584), let's discuss alternative clean solutions again. Otherwise, this is probably going to haunt us forever (the "666" issue number seems fitting, I guess 😄).

@eth-p eth-p closed this as completed in f5d3599 Sep 27, 2019
@sharkdp
Copy link
Owner

sharkdp commented Mar 22, 2020

This has been fixed in bat 0.13.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants