Skip to content
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

/etc/network/if-pre-up.d/wireless-tools | requires net-tools #2074

Closed
Fourdee opened this Issue Sep 15, 2018 · 4 comments

Comments

Projects
None yet
2 participants
@Fourdee
Copy link
Collaborator

commented Sep 15, 2018

However, not listed as a APT dep:
#2070 (comment)

Appears only required, when all other methods fail, and, should not effect WiFi usage:
#2070 (comment)

@MichaIng

This comment has been minimized.

Copy link
Owner

commented Sep 15, 2018

Appears only required, when all other methods fail, and, should not effect WiFi usage:

Indeed it appears, if any of the settings failed to be applied. All failed settings are not applied then, as ifconfig does not bring up the interface.
But the cases, where impotent settings cannot be applied without interface up, seem to be rare, as we never faced this issue since the quite long time net-tools are not pre-installed any more and got removed on update.

Bug report done: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=908886

@MichaIng

This comment has been minimized.

Copy link
Owner

commented Sep 16, 2018

Bug solved on current version: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=908886#10

  • Download available on unstable/sid repo so far only: https://packages.debian.org/sid/amd64/wireless-tools/download
  • Only change that I could bind is indeed the ifconfig line being replaced by: /sbin/ip link set dev "$IFACE" up, which we can do ourself on v6.16 patch:
    sed -i '\|^[[:blank:]]ifconfig "$IFACE" up$|c\\t/sbin/ip link set dev "$IFACE" up' /etc/network/if-pre-up.d/wireless-tools

PR up: #2076

@Fourdee

This comment has been minimized.

Copy link
Collaborator Author

commented Sep 16, 2018

@MichaIng

Bug solved on current version: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=908886#10

Legend, great work 👍

@Fourdee

This comment has been minimized.

Copy link
Collaborator Author

commented Sep 18, 2018

@MichaIng

Works a charm, no issues in testing 👍 . Marking as completed.

@Fourdee Fourdee closed this Sep 18, 2018

@Fourdee Fourdee referenced this issue Sep 19, 2018

Merged

v6.16 #2080

Fourdee added a commit that referenced this issue Sep 19, 2018

Merge pull request #2080 from Fourdee/testing
v6.16
(19/09/18)

**Changes / Improvements / Optimizations:**

DietPi-Config | WiFi: Added support for applying up-to 5 SSIDs: #368

DietPi-Drive_Manager | Samba Mount: Now mounts with user and group of 'DietPi'. This matches group permissions for DietPi applications that will prevent permission failure, when samba is used for userdata location: #2067

DietPi-Software | Pi-hole: You can now install Pi-hole again on Debian Jessie system, as support got re-enabled with Pi-hole v4.0 FTLDNS update.

**Bug Fixes:**

General | Resolved an issue where 1st run setup password prompt, would run twice. Once on 1st run and once after updates are completed.

General | Resolved an external issue where in very rare cases WiFi interfaces were not initiated successfully: #2074

DietPi-Config | Sparky SBC: Resolved an issue where Piano DAC firmware was not being fully installed.

DietPi-Config | Resolved an issue with selected WiFi SSID not being correctly applied: #2070 (comment)

DietPi-Drive_Manager | RockPro64: Resolved an issue where mtdblock devices would show up in the list. As far as well can tell, these are not currently required for EMMC/SD/USB devices. To ensure the list only shows actual devices, we have disabled them from showing up in view: #2067 (comment)

DietPi-Postboot | Resolved an issue where user scripts were not being executed '/var/lib/dietpi/postboot.d/*'.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.