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
Hostap update for Orange Pi #106
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix the following before submitting any code for review:
- spaces not tabs in this repo
- extra or missing newlines
- comments have a space after
#
and begin with a capital - misaligned indentations
scripts/hostapd/install
Outdated
@@ -9,6 +9,16 @@ if ! [ "$(which hostapd)" ] || ! [ "$(which dnsmasq)" ]; then | |||
sudo apt-get install hostapd dnsmasq radvd -y | |||
fi | |||
|
|||
#if ! [ "$(which dnsmasq)" ]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why? Old version on Orange Pi's?
scripts/hostapd/install
Outdated
if ! [ "$(which hostapd)" ] || ! [ "$(which dnsmasq)" ]; then | ||
sudo apt-get install hostapd dnsmasq radvd -y | ||
fi | ||
sudo apt-get install hostapd dnsmasq radvd dnsmasq iptables -y |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# Install packages to run IEEE 802.11 Access Point
and dnsmasq
is doubled
scripts/hostapd/install
Outdated
sudo echo "" | sudo tee --append /etc/dhcpcd.conf > /dev/null | ||
sudo echo "denyinterfaces wlan0" | sudo tee --append /etc/dhcpcd.conf > /dev/null | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please do not have random newlines everywhere.
scripts/install2
Outdated
@@ -11,13 +11,18 @@ BOARD_REVISION="?" | |||
CJDNS_BUILD_CMD="sudo Seccomp_NO=1 NO_NEON=1 ./do" | |||
BOARD_HARDWARE=$(cat /proc/cpuinfo | grep Hardware | awk '{print $3}' | head -n 1) | |||
|
|||
ALLOW_HOSTAP=false #Flag for prompt of hostap |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SUPPORT_HOSTAP
? The comment for the flag should be whether the board supports hostapd, the prompt is just the effect of the flag being true.
scripts/install2
Outdated
@@ -61,6 +67,9 @@ if [[ $BOARD_FAMILY == "Orange Pi" ]]; then | |||
wget http://apt.armbian.com/pool/main/l/linux-4.10.0-sun8i/linux-image-dev-sun8i_5.26_armhf.deb | |||
sudo dpkg -i linux-image-dev-sun8i_5.26_armhf.deb | |||
rm -rf linux-image-dev-sun8i_5.26_armhf.deb | |||
ALLOW_HOSTAP=true #downgrading kernel, allow hostapd |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The downgrade is still needed here?
scripts/install2
Outdated
@@ -61,6 +67,9 @@ if [[ $BOARD_FAMILY == "Orange Pi" ]]; then | |||
wget http://apt.armbian.com/pool/main/l/linux-4.10.0-sun8i/linux-image-dev-sun8i_5.26_armhf.deb | |||
sudo dpkg -i linux-image-dev-sun8i_5.26_armhf.deb | |||
rm -rf linux-image-dev-sun8i_5.26_armhf.deb | |||
ALLOW_HOSTAP=true #downgrading kernel, allow hostapd | |||
else | |||
ALLOW_HOSTAP=false #Zero is not downgraded from broken kernel, dont allow hostap |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't
Have to finish testing it still but i think that some of the change here will help with stretch as well