search

gap-actions / setup-gap

Other
1 stars 5 forks source link

Add support for building on macOS #19

Closed ssiccha closed 3 years ago

ssiccha commented 3 years ago

Fixes #17.

ssiccha commented 3 years ago

@fingolfin What do you think? Not pretty but it does the job. Unfortunately we can't use if conditionals for steps of composite actions, if I'm not mistaken. At least I didn't get it to work.

Also, it looks like we don't need to install gmp, readline, and zlib via brew. Or should we keep it just to be safe?

fingolfin commented 3 years ago

We can do w/o readline but installing GMP and zlib saves precious build time

ssiccha commented 3 years ago

We can do w/o readline but installing GMP and zlib saves precious build time

Here brew says:

Warning: gmp 6.2.1 is already installed and up-to-date.
To reinstall 6.2.1, run:
  brew reinstall gmp
Warning: readline 8.1 is already installed and up-to-date.
To reinstall 8.1, run:
  brew reinstall readline