Skip to content

Conversation

@kristofvandam
Copy link

@kristofvandam kristofvandam commented Jan 4, 2026

  • Adds iproute2 to the wrapped defguard-service
  • Adds update-desktop-database to the wrapped defguard-client
  • Tauri Builds on the GTK3/WebKitGTK4.1 stack, there was a mismatch with GTK4/WebKitGTK4.1, (GTK4 should use WebKitGTK6.0, but that isn't supported yet for tauri.
  • No Makefile so we can overwrite/rename postInstall with installPhase.
  • Formatting

With these changes in place, it fixes issue #728.

* Adds `iproute2` to the wrapped `defguard-service`
* Adds `update-desktop-database` to the wrapped `defguard-client`
* Tauri Builds on the GTK3/WebKitGTK4.1, there was a mismatch with
  GTK4/WebKitGTK4.1, (GTK4 should use WebKitGTK6.0, but that isn't
  supported [yet](tauri-apps/tauri#14684) for tauri.
* No `Makefile` so we can overwrite/rename `postInstall` with
  `installPhase`.
* Formatting

With these changes in place, it fixes issue [DefGuard#728](DefGuard#728).
@teon teon moved this to Analysis in Backlog & Roadmap Jan 5, 2026
@teon teon moved this from Analysis to Refined & Ready in Backlog & Roadmap Jan 5, 2026
@wojcik91 wojcik91 moved this from Refined & Ready to In Progress in Backlog & Roadmap Jan 7, 2026
@wojcik91 wojcik91 moved this from In Progress to Review in Backlog & Roadmap Jan 7, 2026
@wojcik91
Copy link
Contributor

wojcik91 commented Jan 8, 2026

Than you so much for your contribution, I've been searching for a way to handle those missing runtime dependencies.

I also had some other fixes prepared in a separate PR, so I've incorporated your solution there: #747

Again, thank you. I'm glad to finally be able to get the NixOS module to work :)

@wojcik91 wojcik91 closed this Jan 8, 2026
@github-project-automation github-project-automation bot moved this from Review to Ready to release in Backlog & Roadmap Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready to release

Development

Successfully merging this pull request may close these issues.

3 participants