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

check for all external progs like xxd inc needed version #2

Open
MaxPeal opened this issue Jan 12, 2022 · 1 comment
Open

check for all external progs like xxd inc needed version #2

MaxPeal opened this issue Jan 12, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@MaxPeal
Copy link

MaxPeal commented Jan 12, 2022

check for all external progs like xxd inc needed version.

e.g. on a system without xxd it don't catch the error:
root@DietPi-01:~# bash sdinfo.sh -c 1b534d454332515430615763c7013633

   ___________________  
  |                   | 
  | |               ==| 
  | |               ==| 
  | |   SDInfo      ==| 
  | |   2021-12-24  ==| 
  | |               ==| 
  | |       __    ____| 
  |________/  |__/      

sdinfo.sh: line 416: xxd: command not found
sdinfo.sh: line 422: xxd: command not found
MID : Samsung
OID :
PNM :
PRV : 30
PSN : 1633117127
MDT : 2019/6/1
AID : 956

root@DietPi-01:~#

@OneOfTheInfiniteMonkeys
Copy link
Owner

OneOfTheInfiniteMonkeys commented Jan 22, 2022

I agree, good suggestion.

It has been rather a long time since I wrote POSIX compliant scripts. Looking at the internet circa 1995, exciting!
(Repeat comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants