There's also many misspellings, including at least one command. Line 306: sensible-brownser. I'm not sure exactly how that happened, since that specific function is ripped straight out of the original quickget.

Temporary files should be created with mktemp, a temp directory is entirely unnecessary.

Temporary files should be created with mktemp, a temp directory is entirely unnecessary.

Thanks for mktemp mention 👍

I've provided what I believe to be a fix. Are lines 156-158 of quickget just for debug? It looks that way, as it just prints out the URL, hash, etc, which doesn't need to be presented to the end user.

My intent by creating a TMPDIR in quickget was to provide a consistent place for any temporary files and the guaranty that they will get deleted no matter how the script is ended. I agree using mktemp for the directory is a better solution but just making temporary files that don;t get deleted is not.

Looks like you method of passing associative arrays back from a function works with less bash restrictions than the way I did it. Thanks

I notice that your refactor still has a test for Bash 4+. Version 5 has been circulating for some years now and is fairly standard. Have your tested your refactored code on Bash 4? I wonder if you should consider moving that test to 5+

I am not opposed to moving to 5+ but I think this should follow quickemu

I think it is wrong to test a script on Bash 5 and then just let people on Bash 4 go ahead and use it without any warning.

This is what I originally did with qqX:

if [[ ! "$(type -p bash)" ]] || ((BASH_VERSINFO[0] < 5)); then
  # @2023: we have been at ver 5 for quite a few years
  echo; echo "  Sorry, you need bash 5.0 or newer to run this script."; echo
  echo "  Your version: "; echo
  bash --version
  echo; sleep 10; exit 1
fi

But writing this has made me I think that I want to improve this a bit further, also for myself.

I don't like just telling people to update and kicking them out of the door either.

Basically, I just copy and pasted @ flexiondotorg 's code and gave it a bit more UX info ...

So, on reflection, I am now doing this with qqX for the new release:

if [[ ! "$(type -p bash)" ]] || ((BASH_VERSINFO[0] < 5)); then
  # @2023: we have been at ver 5 for quite a few years
  echo; echo "  Sorry, you probably need Bash 5.0 or newer to run this script."; echo
  echo "  qqX has only been tested on up-to-date versions of Bash ...."; echo
  echo "  Your version: "; echo
  bash --version
  echo
  read -rp "  Press [enter] to try anyway   [e]  to  exit and update  >  "    UpdateBash
  if [[ $UpdateBash ]]; then echo;  exit 1; 
  else  echo; echo "  I understand the risks and have made backups" ; echo ; read -rp "  [enter] to confirm  [e] to exit > " UpdateBash ; fi
  echo
  [[ $UpdateBash ]] &&  exit 1
fi

I think this works better.

Also given that we/you are refactoring/restructuring pretty much most of quickget, we shouldn't ignore this bit just because it is not fixed in quickemu. Two wrongs don't make a right ...

Paste this into a script and set the value to 6. See what you think.