search

rui314 / setup-mold

A GitHub Action to install the mold linker
MIT License
114 stars 10 forks source link

note requirement on `wget` package #15

Open jimkring opened 6 months ago

jimkring commented 6 months ago

It's worth noting that this action requires wget to be installed. I'm working with a container that doesn't already have it installed, so I had to install it. Not a big deal, but maybe work mentioning.

rui314 commented 6 months ago

Alternatively, I could make a change to this Action so that it wouldn't try to install mold if wget is not available. Which do you prefer?

jimkring commented 6 months ago

I would probably prefer that mold detects the missing dependency and reports a descriptive message like "The 'wget' package is required to setup mold. Please ensure it's installed on your system (e.g. using your system's package manager) and then try again."

Thanks again for mold -- it's great!