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

Add a --files option #1422

Closed
johnkerl opened this issue Nov 7, 2023 · 5 comments · Fixed by #1426
Closed

Add a --files option #1422

johnkerl opened this issue Nov 7, 2023 · 5 comments · Fixed by #1426
Assignees

Comments

@johnkerl
Copy link
Owner

johnkerl commented Nov 7, 2023

The user would supply one file name per line like

cat filenames.txt

file0001.csv
file0002.csv
...
file8956.csv

Then they could run mlr --files filenames.txt ...

Originally posted by @derekmahar in #1421

@derekmahar
Copy link
Contributor

Thank you!

@aborruso
Copy link
Contributor

Thank you

@janxkoci
Copy link

Just curious, does it do the same thing as the following code?

mlr --mfrom $(cat filenames.txt) --

@johnkerl
Copy link
Owner Author

@janxkoci yes. With the exception that if filenames.txt is very long, putting them all on the command line via interpolation can result in an error from the shell saying the argument list is too long. Whereas mlr --files will step through filenames.txt successfully without hitting that limit.

@janxkoci
Copy link

I see, cool!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants