ninenines / erlang.mk

A build tool for Erlang that just works.
https://erlang.mk
ISC License
578 stars 241 forks source link

autopatching is overly verbose (missing $(verbose)) #981

Closed artman41 closed 1 year ago

artman41 commented 1 year ago

Somewhere in the spree of commits you merged @essen, a change to remove autopatching for rabbitmq also removed the $(verbose) preceding the bash

Before https://github.com/ninenines/erlang.mk/blob/a1d7135f1349735569016adcb3e185be58c44042/core/deps.mk#L768-L788

After https://github.com/ninenines/erlang.mk/blob/e1e3d15a1b211fc60b83557a2ea54ea3741f3e80/core/deps.mk#L841-L847

This results in a lot of debug info being printed to the terminal during the deps step as all the autopatching is executed

essen commented 1 year ago

Oops. Can you please send a PR adding the $(verbose) back?

artman41 commented 1 year ago

@essen done 👍

essen commented 1 year ago

Thanks!