Skip to content
This repository has been archived by the owner on Feb 16, 2020. It is now read-only.

Minor Fixes #1

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

Minor Fixes #1

wants to merge 3 commits into from

Conversation

zellkou
Copy link

@zellkou zellkou commented Dec 15, 2018

  1. Due to pulling wireguard from unstable it will support any version of debian so I removed the 9
  2. Pulled some variables from another script I wrote for server setups. I find it a much cleaner way to check
    distro and version.
  3. Added a check for Ubuntu that was missing.
  4. Fixed placement of "-y" in apt commands
  5. Attempted to mitigate for the edge case of already having unstable repository enabled. By checking for
    "/sources.list.d/unstable.list" and "/preferences.d/unstable"

Thanks for writing this I know it will come in very handy.

@l-n-s
Copy link
Owner

l-n-s commented Dec 15, 2018

Thanks for your interest, I'll have a look soon

jimzrt
jimzrt reviewed Dec 20, 2018
View reviewed changes
wireguard-install.sh Outdated
elif [ -e /etc/debian_version ]; then
DISTRO=$( lsb_release -a 2>/dev/null| grep "Distributor ID" | awk '{print $3}' )
elif [ $id == debian ]; then
DISTR0="Debian"
Copy link

Choose a reason for hiding this comment

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

this is 0 the number and not O the letter!

Copy link
Author

Choose a reason for hiding this comment

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

With much shame and embarrassment I have fixed the typo.

@devimauz devimauz mentioned this pull request Jan 24, 2020
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jimzrt jimzrt jimzrt left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zellkou @l-n-s @jimzrt