Skip to content

Fix adapt for server with both curl and busybox version wget#76

Open
Binly42 wants to merge 1 commit into
jeanp413:masterfrom
Binly42:patch-1
Open

Fix adapt for server with both curl and busybox version wget#76
Binly42 wants to merge 1 commit into
jeanp413:masterfrom
Binly42:patch-1

Conversation

@Binly42

@Binly42 Binly42 commented Apr 1, 2023

Copy link
Copy Markdown

NOTE: In my trial last week, it was reported that a parameter (I forgotten ...) in the wget command was not supported by the busybox version wget, and I just temporarily renamed the wget on that server ...

@lex-ibm

lex-ibm commented Jun 8, 2023

Copy link
Copy Markdown
Contributor

Isn't there a better way around this?
In my current setup, curl performs significantly worse than wget (sometimes by a long shot) and here you just moved the priority to prefer curl. This would be detrimental for some of us.
You could maybe try distinguishing if you are running GNU Wget or BusyBox Wget by getting the --version output and checking for "BusyBox".

@daiyam

daiyam commented Apr 2, 2026

Copy link
Copy Markdown
Collaborator

@Binly42 Can you put wget first and update the test with if command -v wget >/dev/null 2>&1 && [[ ! "$(wget --version 2>&1)" =~ BusyBox ]]; then? Thx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants