vaporup / ssh-tools

Making SSH more convenient
GNU General Public License v3.0
889 stars 27 forks source link

Even on Linux systems you may want to use perl #10

Closed ericcurtin closed 3 years ago

ericcurtin commented 4 years ago

If the date binary doesn't exist but the perl one does

vaporup commented 4 years ago

Thanks for the patch. Sorry for not replying earlier, I didn't get a notification from Github :-(

Just out of curiosity, on which Linux Distro was the date binary missing?

ericcurtin commented 4 years ago

Struggling to remember, I think it may have been a docker container. They start off fairly barebones with very little.

ericcurtin commented 4 years ago

I would nearly do it this way, ignoring platform:

if perl exists use perl way else if date exists use date way else error message stating you need date for perl

Or else just use perl everywhere, less dependencies to track, it's very rare that perl isn't installed, that's only the case is very embedded systems, android, RTOS's that kind of case I think? Which is rare