Skip to content

Check if busybox exists #2

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.

Sign up for GitHub

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

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

Check if busybox exists #2

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
3ab0d04
Preconfigure SSH on device
jbruechert Jul 9, 2017
b9cb07e
Check if busybox exists and install it when needed
jbruechert Jul 10, 2017
5eb5be4
Mark busybox executable
jbruechert Jul 10, 2017
3d01ae7
Don't create an unnecessary new folder
jbruechert Jul 10, 2017
5c0be56
Install busybox applets to /sbin
jbruechert Jul 11, 2017
a03965b
Use tabs to match coding style
jbruechert Jul 11, 2017
fd76886
Correct architecture names
jbruechert Jul 11, 2017
890a9da
Make if a little more efficient
jbruechert Jul 11, 2017
c2f6c2e
Correct test parameter
jbruechert Jul 15, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions halium-install
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,14 +48,16 @@ check_busybox()
export device_architecture=$(do_shell "uname -m")
export busybox_exists=$(do_shell "if [ -d /sbin/busybox ]; then echo true; else echo false; fi")

if [ $busybox_exists = "true" ]; then
if [ $busybox_exists = true ]; then
echo 'Using already installed busybox'
else
if [ $device_architecture = x86_64 ]; then
wget https://github.com/meefik/busybox/raw/master/app/src/main/assets/intel/static/bin/busybox
elif [ $device_architecture = armv7l ]; then
wget https://github.com/meefik/busybox/raw/master/app/src/main/assets/arm/static/bin/busybox
elif [ $device_architecture = arm64 ]; then
elif [ $device_architecture = aarch64 ]; then
wget https://github.com/meefik/busybox/raw/master/app/src/main/assets/arm/static/bin/busybox
elif [ $device_architecture = armv8l ]; then
Copy link
Copy Markdown
Collaborator

@z3ntu z3ntu Jul 11, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you should do if device_architecture is armv7l or aarch64 or armv8l and not three seperate branches that do exactly the same.

Copy link
Copy Markdown
Member Author

@jbruechert jbruechert Jul 11, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm new to bash and didn't know it can do this. Will of course correct :)

Copy link
Copy Markdown
Member Author

@jbruechert jbruechert Jul 11, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Better now?

Copy link
Copy Markdown
Collaborator

@z3ntu z3ntu Jul 11, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep looks good :)

wget https://github.com/meefik/busybox/raw/master/app/src/main/assets/arm/static/bin/busybox
fi

Expand Down