search

amiaopensource / ltopers

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

Misc. Documentation Updates #169

Closed libbyhopfauf closed 9 months ago

libbyhopfauf commented 6 years ago

Here are a couple of items that need to be updated in the documentation. Thanks!

1.) Make the reference to the tape name/number/id/serial consistent across all documentation.

2.) In the "Writing to LTO" section, the command should be updated (you are not prompted to do anything when just running "writelto"): writelto -t ${tapeserial} /Volumes/${sourcedirectory}

retokromer commented 6 years ago

+1

Once you have updated the «regular» documentation, if you wish, I volunteer for updating the man as well (and the -h, supposing they are not included in the «regular» ;-)

retokromer commented 6 years ago

@libbyhopfauf Has this been resolved in https://github.com/amiaopensource/ltopers/pull/173?

privatezero commented 6 years ago

I think @libbyhopfauf was also referencing the other associated info (such as the man page) so I think this is only partially resolved so far

retokromer commented 6 years ago

I haven’t noticed incongruencies in the man page when I updated it… yet I am not English speaker/writer.

privatezero commented 6 years ago

Ugh! My apologies - I had forgotten about https://github.com/amiaopensource/ltopers/pull/177 also altering the man page!

In that case - it seems that the only part of this issue is updating the writelto instructions in the README.

It wasn't mentioned in the original issue, but the notes about rsync seem out of date as well since gcp is now used for the original transfer of files.

kieranjol commented 5 years ago

I was going to make a new issue about this but I see that @libbyhopfauf beat me to it. I've only looked over a colleagues shoulder who is using ltopers, but it looked like -t is mandatory but it's not reflected in the docs. So I guess

2.) In the "Writing to LTO" section, the command should be updated (you are not prompted to do anything when just running "writelto"): writelto -t ${tapeserial} /Volumes/${sourcedirectory}

is still a valid issue?

Edit: Actually -t and the source directory are both mandatory?

retokromer commented 4 years ago

Has this been resolved in https://github.com/amiaopensource/ltopers/commit/ed8ccabe9e5e09bf578e3cf2f043dc48b84fd4fb?

dericed commented 9 months ago

Yes, I think so.