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

Minor Fixes#1

Open
zellkou wants to merge 3 commits intol-n-s:masterfrom
zellkou:master
Open

Minor Fixes#1
zellkou wants to merge 3 commits intol-n-s:masterfrom
zellkou:master

Conversation

@zellkou
Copy link
Copy Markdown

@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
Copy Markdown
Owner

l-n-s commented Dec 15, 2018

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

Comment thread 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
Copy Markdown

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
Copy Markdown
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.

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.

3 participants