Skip to content

Conversation

@Oidaho
Copy link

@Oidaho Oidaho commented Aug 10, 2025

Pull Request

Description

PR in addition to issue #448

This change more logically structures comments and package groups in .lst files with the necessary packages for HyDe.

The main innovation is the addition of the necessary packages for users to work with VPNs of almost any caliber "from under the hood", without having to configure and/or fix extensions for Network Manager yourself.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My commit message follows the commit guidelines.
  • My change requires a change to the documentation. (Wiki)
  • I have updated the documentation accordingly.
  • I have added a changelog entry.
  • I have added necessary comments/documentation to my code.
  • I have added tests to cover my changes.
  • I have tested my code locally and it works as expected.
  • All new and existing tests passed.

Screenshots

The ability to select multiple VPN connections:
image

Available VPN protocols for every taste:
image

Additional tabs in the Connection viewer:
image

Openconnect Authentication Window:
image

Additional context

This change is niche, I agree. However, it will help most people who use HyDe as a workstation.

I've advised a lot of people on this project, and they all use VPNs one way or another: To access certain resources or to the corporate network on a remote work schedule.

I have tried to choose the most popular and best protocols that are available today. Other VPN protocols for NM can be described in pkg_extra.lst

Oidaho added 3 commits August 10, 2025 13:49
This commit adds the necessary packages for
Network Manager to work with various VPN protocols,
as well as some graphical packages that NM Applet
requires for authentication windows.
This commit adds the system packages necessary
for some git commands to work, as well as to
clarify the documentation for some commands.
@Oidaho Oidaho changed the title Network Manager VPN protocols feat: Network Manager VPN protocols Aug 10, 2025
@amit-0i
Copy link
Member

amit-0i commented Aug 10, 2025

  1. we keep core packages as minimal as possible, so anything extra goes in pkg_extra.lst

  2. i dont think less and man-db packages are required in a dotfiles repository like this. and as far as i remember, less comes with minimal arch installation.

@Oidaho
Copy link
Author

Oidaho commented Aug 10, 2025

  1. we keep core packages as minimal as possible, so anything extra goes in pkg_extra.lst
2. i dont think `less` and `man-db` packages are required in a dotfiles repository like this. and as far as i remember, `less` comes with minimal arch installation.

Ok, then I'll move the VPN block to pkg_extra.lst

No, the arch minimal profile does not provide less. And it's weird. At some point, I wanted to check the GPG signatures for my commits and found that the git show and git log commands simply did not work due to the lack of a less package. So, I think that less is worthy of being in `pkg_core.lst'.

But man-db is exactly what dotfiles needs, otherwise you can get confused in the variety of fairly fresh and non-trivial packages.

@Oidaho Oidaho force-pushed the vpn-protocols-for-nm branch from 5b2a0c0 to 80fe84c Compare August 10, 2025 06:24
@kRHYME7
Copy link
Contributor

kRHYME7 commented Aug 10, 2025

But man-db is exactly what dotfiles needs, otherwise you can get confused in the variety of fairly fresh and non-trivial packages

One can use a browser. We can have this in our extra packages.

@kRHYME7
Copy link
Contributor

kRHYME7 commented Aug 12, 2025

Resolve conflicts please

@Oidaho
Copy link
Author

Oidaho commented Aug 29, 2025

I found another cool thing today:
The git rebase --help command requires the man package, similar to the less package in the git show or git log command

Tonight I will return man to the list of system dependencies.

@kRHYME7
Copy link
Contributor

kRHYME7 commented Aug 29, 2025

man is not really required for HyDE to run or feature so I don't think we should add it.

@Oidaho
Copy link
Author

Oidaho commented Aug 29, 2025

man is not really required for HyDE to run or feature so I don't think we should add it.

Okay, whatever you say

But it still haunts me that some of the git functions don't work w\o this package. They're not important, but the idealist inside me is sounding the alarm :)

@kRHYME7
Copy link
Contributor

kRHYME7 commented Aug 29, 2025

You can Install it in your local. We have tons of apps here with optional dependency, following this idea we will have tons of dependencies here.

@Hareekshith
Copy link

Hareekshith commented Oct 2, 2025

@Oidaho
How is the outline of the pop-up removed?
I have something like this!
image

@Oidaho
Copy link
Author

Oidaho commented Oct 2, 2025

@Oidaho How is the outline of the pop-up removed? I have something like this!

Perhaps this problem could arise due to incorrect windowrules.

In Hyprland, it is possible to customize the noborder in windowrules for specific applications.

@kRHYME7
Copy link
Contributor

kRHYME7 commented Oct 2, 2025

@Hareekshith GTK3 theming issue. This can be solved by modifying the gtk theme. Not all theme has this. I'm no good in css so I can't help.

@github-actions
Copy link
Contributor

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants