ncbi / gaptools

dbGaP data validation tool repo
Other
11 stars 11 forks source link

macOS support with short options only #19

Closed alanhoyle closed 1 year ago

alanhoyle commented 1 year ago

This allows macOS users to run GaPTools.

macOS's getopt is based on the BSD version which does not support long command line options like --help.

The changes detect darwin* os types, forces using the short command line options (-h, etc.), and prints a warning as appropriate.

This addresses #14.

It also improves the output_files directory in the repo .gitignore, removed a few trailing whitespace characters on a few lines. It also fixes and improves formatting, and adds and fixes links in the README.md .

On some Docker installations, Compose is accessible via docker compose and not docker-compose and this works around this by searching for both and using the one it finds.

davidshao commented 1 year ago

Tested on Mac, and worked as expected. Thanks, Alan.

alanhoyle commented 1 year ago

Something failed about the merge as I don't see the changes reflected in ncbi:master