Skip to content
This repository has been archived by the owner on Nov 11, 2021. It is now read-only.

Add bash script to check driver on some distros. #54

Merged
merged 13 commits into from
May 6, 2020
Merged

Add bash script to check driver on some distros. #54

merged 13 commits into from
May 6, 2020

Conversation

aquadzn
Copy link
Contributor

@aquadzn aquadzn commented Apr 29, 2020

No description provided.

@aquadzn
Copy link
Contributor Author

aquadzn commented Apr 29, 2020

done 👍

Copy link
Collaborator

@agentcobra agentcobra left a comment

Choose a reason for hiding this comment

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

look OK for me

@NicolasGuilloux
Copy link
Owner

There is some stuff that I dislike in the script. It tries to install the driver evenif the vainfo which is very instrusive if we widespread this script for all community.

A better approach would be to test vainfo, and if it is not optimal, it checks if all the appropriate drivers are installed. If not, it asks to actually install them and recheck.

Moreover, a small warning about why we use sudo would be great aswell. Many users of the scripts we made are not good Linux users, and I guess they will start the command several times and enter their password several times aswell. Limiting that would be great :)

scripts/check_driver.sh Outdated Show resolved Hide resolved
scripts/check_driver.sh Outdated Show resolved Hide resolved
scripts/check_driver.sh Show resolved Hide resolved
scripts/check_driver.sh Outdated Show resolved Hide resolved
scripts/check_driver.sh Outdated Show resolved Hide resolved
scripts/check_driver.sh Outdated Show resolved Hide resolved
scripts/check_driver.sh Outdated Show resolved Hide resolved
Copy link
Contributor Author

@aquadzn aquadzn left a comment

Choose a reason for hiding this comment

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

Good Missing vainfo Distro not found
good vainfo distro

@NicolasGuilloux @agentcobra 😃

scripts/check_driver.sh Outdated Show resolved Hide resolved
scripts/check_driver.sh Show resolved Hide resolved
scripts/check_driver.sh Show resolved Hide resolved
scripts/check_driver.sh Outdated Show resolved Hide resolved
scripts/check_driver.sh Outdated Show resolved Hide resolved
ImgBotApp and others added 2 commits April 30, 2020 21:16
*Total -- 1,216.68kb -> 1,009.54kb (17.03%)

/docs/assets/css/images/overlay.png -- 19.63kb -> 2.76kb (85.95%)
/docs/images/logo.png -- 27.42kb -> 17.03kb (37.89%)
/docs/images/share_folder.png -- 354.91kb -> 248.44kb (30%)
/docs/images/appimage.png -- 22.24kb -> 17.33kb (22.08%)
/docs/images/hamachi_virtualhere.png -- 118.13kb -> 93.61kb (20.76%)
/docs/images/dolphin_share.png -- 53.87kb -> 44.56kb (17.29%)
/docs/images/hamachi_config.png -- 90.20kb -> 76.13kb (15.59%)
/docs/images/hamachi_virtualhere_linux.png -- 54.47kb -> 46.25kb (15.09%)
/docs/images/shadow-tool.png -- 24.08kb -> 20.71kb (14%)
/docs/assets/css/styles/brown-papersq.png -- 17.77kb -> 17.36kb (2.32%)
/docs/assets/fonts/fontawesome-webfont.svg -- 433.96kb -> 425.36kb (1.98%)

Signed-off-by: ImgBotApp <[email protected]>
Reduce images total size by 17%
@NicolasGuilloux NicolasGuilloux merged commit b52288d into NicolasGuilloux:master May 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants