Skip to content

LVPN-9381: Replace gokogiri,ratago with call to xsltproc#1158

Merged
devzbysiu merged 5 commits intorelease/4.2.1from
LVPN-9381-current-libxml-dependency-is-missing
Oct 29, 2025
Merged

LVPN-9381: Replace gokogiri,ratago with call to xsltproc#1158
devzbysiu merged 5 commits intorelease/4.2.1from
LVPN-9381-current-libxml-dependency-is-missing

Conversation

@devzbysiu
Copy link
Copy Markdown
Contributor

Instead of using xml and xslt libraries to transform config for openvpn, use xsltproc binary. Those libraries require oldish libxml2 which is updated on newer distributions and the old one is not available for installation.

Changes:

  • remove libxml2 from dependencies, add xsltproc - for deb, rpm and snap
  • remove github.com/jbowtie/gokogiri/xml, github.com/jbowtie/ratago/xslt libs
  • delegate xsl transformation to xsltproc binary

This addresses #1142

Signed-off-by: Bartosz Zbytniewski <bartosz.zbytniewski@nordsec.com>
Signed-off-by: Bartosz Zbytniewski <bartosz.zbytniewski@nordsec.com>
@devzbysiu devzbysiu force-pushed the LVPN-9381-current-libxml-dependency-is-missing branch from 9d361b8 to 54f00e2 Compare October 22, 2025 11:02
Signed-off-by: Bartosz Zbytniewski <bartosz.zbytniewski@nordsec.com>
@devzbysiu devzbysiu marked this pull request as ready for review October 23, 2025 05:02
@devzbysiu devzbysiu marked this pull request as draft October 23, 2025 07:04
Signed-off-by: Bartosz Zbytniewski <bartosz.zbytniewski@nordsec.com>
Signed-off-by: Bartosz Zbytniewski <bartosz.zbytniewski@nordsec.com>
@devzbysiu devzbysiu force-pushed the LVPN-9381-current-libxml-dependency-is-missing branch from 2d50573 to fc0a150 Compare October 23, 2025 09:54
@devzbysiu devzbysiu marked this pull request as ready for review October 23, 2025 10:40
@TheNexter
Copy link
Copy Markdown

This patch is very important, on ubuntu 25.10, nordvpn is not possible to install

@devzbysiu devzbysiu merged commit 1c0d195 into release/4.2.1 Oct 29, 2025
13 checks passed
@devzbysiu devzbysiu deleted the LVPN-9381-current-libxml-dependency-is-missing branch October 29, 2025 08:07
@ruffsl ruffsl mentioned this pull request Dec 8, 2025
13 tasks
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.

5 participants