Skip to content
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

better way of running the bluebomb helper script. #115

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

sctech-tr
Copy link

Description

this pr basically fits the entire downloading and running process into one command, so the users don't have to run 3 commands.

the old commands are:

wget https://wii.hacks.guide/assets/files/bluebomb-helper.sh
chmod +x bluebomb-helper.sh
./bluebomb-helper.sh

and the new command is:

wget -qO- https://wii.hacks.guide/assets/files/bluebomb-helper.sh | sh

if i did anything wrong, please notify me because this is my first time contributing here.

_pages/en_US/bluebomb.md Outdated Show resolved Hide resolved
Co-authored-by: thepikachugamer <[email protected]>
@sctech-tr
Copy link
Author

sctech-tr commented Sep 16, 2024

i did it
i have no idea why sh fails, it's just a symlink to bash (it may be different in other systems, so nice idea)

@danny8376
Copy link

i did it i have no idea why sh fails, it's just a symlink to bash (it may be different in other systems, so nice idea)

not always, like debian/ubuntu use dash, not bash for sh, you should never expect sh to work like bash in any case

@sctech-tr
Copy link
Author

how about the $SHELL variable?

sctech-tr

This comment was marked as off-topic.

@sctech-tr
Copy link
Author

it even works on iSH shell
image

@danny8376
Copy link

danny8376 commented Sep 17, 2024

it even works on iSH shell

The script is just written with bash in mind
so... just make it run with bash
if they don't have bash and still want to run it, they need to figure it out
getting weird error cuz the user using weird shell is the least you want

@sctech-tr
Copy link
Author

image
%96 of the people who want to homebrew their wii use windows, they are not going to be linux masters. only the remaining %4 of the people can probably figure out how to run it in bash

@danny8376
Copy link

figure what? bash is literally in most distro, you literally just need to type bash, like what originally was
if they don't have bash, they most likely using niche distro which they probably know what's going
if they're not using bash as their main shell, yes, they likely know how to fix it
but you just make what originally will globally works into something that need extra work, no benefit
it absolutely make no sense when you literally need bash to run, then rely on something like $SHELL to introduce uncertainty

@sctech-tr
Copy link
Author

sctech-tr commented Oct 13, 2024

niche distro

you don't automatically be a "niche" distro by just not using bash by default. so, by this logic, arch is a niche distro as well. arch uses zsh, which is superior in most cases compared to bash.

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