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
Fixes #9095: Use a 2 steps download in rudder agent makefile #1076
Fixes #9095: Use a 2 steps download in rudder agent makefile #1076
Conversation
@@ -37,24 +37,31 @@ INSTALL_DIR = $(DESTDIR)$(RUDDER_DIR) | |||
TRY_LDFLAGS=-pie -Wl,-z,relro -Wl,-z,now | |||
TRY_CFLAGS=-fPIE -fstack-protector | |||
|
|||
# Autodetect wget or curl or curl usage and proxy configuration | |||
# GENERIC_GET: do note remove this line |
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.
s/note/not
# If you change this code, change all the places where you can find this tag too. | ||
# It is duplicated because it is the code that is used to manage dependency | ||
# to other repositories, hence allowing deduplication of other code. | ||
# Autodetect wget or curl usage and proxy configuration |
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.
or fetch
GET_OPT = --wget | ||
CPAN_OPT = --wget | ||
endif | ||
endif | ||
GET=get() { $(_GET) "$$1.part" "$$2" && mv "$$1.part" "$$1"; }; get |
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.
Could you add an example of how this will be called to the "doc" comment above?
Commit modified |
6e651b8
to
e46892c
Compare
looks good now |
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/9095