Created attachment 216218 [details]
update version from 3.6 to 4.1
The reason I have not updated the NetworkMgr to the latest because it use sudo now and I am not sure it will be acepted to have /usr/local/etc/sudoers.d/networkmgr.
Doas support was remove in the latest changes and I se that your patch does not refect the changes that was done to NetworkMgr.
If someone can confirm that NetworkMgr can install /usr/local/etc/sudoers.d/networkmgr I will update the port.
(In reply to Eric Turgeon from comment #1)
yeah I just noticed a couple minutes ago about those changes. Maybe it should put on hold I guess until further confirmation.
Created attachment 216224 [details]
with sudo a no doas
this is the update removing doaes and adding sudo as dependency, but again. I totally understood your point Eric.
Created attachment 216237 [details]
forgot to remove the sub_files for the pkg-message
^Triage: Feedback provided (+), set maintainer-feedback accordingly. @Eric If you meant to not accept the proposed patch, use maintainer-approval attachment flag (set to -)
I would also obsolete the patch so its not accidentally committed
(In reply to Miguel Gocobachi from comment #4)
you forgot to update the pkg-plist for the there is a new file installed by NetworkMgr.
Created attachment 216257 [details]
pkg-plist updated and INSTALLS_ICONS added
A commit references this bug:
Date: Mon Jul 6 23:30:44 UTC 2020
New revision: 541374
update net-mgmt/networkmgr to 4.1
Submitted by: Miguel Gocobachi