amiaopensource / ltopers

Bash scripts to manage LTO cartridges with LTFS
https://github.com/amiaopensource/ltopers
MIT License
40 stars 8 forks source link

README - update writelto, attempts to fix #169 #196

Open kieranjol opened 5 years ago

kieranjol commented 5 years ago

I'm almost certain that this is fairly wrong in a few ways, but maybe it'll get the ball rolling. Any edits/fixes are very welcome..

kieranjol commented 5 years ago

Lol I'm sorry, I'm going pretty blind here.looks like your PR would make this redundant as the docs would be correct of that was merged?

privatezero commented 5 years ago

I think that even after that PR is merged, the docs still should be updated along the lines you are doing! I see that PR more as a check against data not being entered initially. People should probably know that they need to add both those values for a complete executable command.

kieranjol commented 5 years ago

I agree, maybe the all caps, with an example command to make it extra clear?

privatezero commented 5 years ago

I think that makes sense!

retokromer commented 5 years ago

ping

retokromer commented 4 years ago

ping ping

retokromer commented 4 years ago

Has this PR not been superseded by https://github.com/amiaopensource/ltopers/commit/ed8ccabe9e5e09bf578e3cf2f043dc48b84fd4fb?