-
Notifications
You must be signed in to change notification settings - Fork 558
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
allow customising 'go' executable with GOEXE var #2841
Conversation
b3acbf5
to
0750440
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all lgtm except for a small change request
thanks for this PR
Signed-off-by: Dmitry S <[email protected]>
Signed-off-by: Dmitry S <[email protected]>
* Makefile: date format of log should not show signatures Signed-off-by: Vincent Batts <[email protected]> * Makefile: add the ability to DEBUG the `make` build Signed-off-by: Vincent Batts <[email protected]> --------- Signed-off-by: Vincent Batts <[email protected]>
859011f
to
321133a
Compare
Signed-off-by: Dmitry S <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #2841 +/- ##
=======================================
Coverage 29.47% 29.47%
=======================================
Files 151 151
Lines 9678 9678
=======================================
Hits 2853 2853
Misses 6386 6386
Partials 439 439 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
addressed your feedback, please take a look @cpanato |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you
Summary
Allow customising the
go
executable path with theGOEXE
variable setting - for example, If one needs to build a debugging version ofgo
etc.go
is the default value which is used ifGOEXE
is not set.Release Note
NONE
Documentation
No documentation changes are needed.