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

debian/control: remove unneeded dependency on network-manager. #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

menakite
Copy link

It was added in 8021bb1 as an alternative to dhcpcd5, but when dhcpcd5 was removed in 1a37b2b network-manager was left there.

Fixes removing network-manager without uninstalling raspberrypi-net-mods too:

menakite@raspy:~ $ sudo apt remove --purge network-manager
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
The following packages were automatically installed and are no longer required:
  dns-root-data dnsmasq-base libbluetooth3 libcurl3-gnutls libmm-glib0 libndp0 libnetfilter-conntrack3 libnfnetlink0 libnm0 libteamdctl0
Use 'sudo apt autoremove' to remove them.
The following packages will be REMOVED:
  network-manager* raspberrypi-net-mods*
0 upgraded, 0 newly installed, 2 to remove and 0 not upgraded.
After this operation, 16.1 MB disk space will be freed.
Do you want to continue? [Y/n] n
Abort.
menakite@raspy:~ $

It was added in 8021bb1 as an
alternative to dhcpcd5, but when dhcpcd5 was removed in 1a37b2b
network-manager was left there.

Fixes removing network-manager without uninstalling raspberrypi-net-mods
too.
@XECDesign
Copy link
Member

Why would you note remove raspberrypi-net-mods in this situation?

Currently, it's only used to let the user know that their wifi won't work unless they set the regulatory domain through raspi-config. The assumption on our end is that if the user is running Raspberry Pi OS, then they're probably using NetworkManager. If not, then we can't test that our changes work as expected and there's no reason for this package to be installed, right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants