Bug 366640

Summary:	fix setting of comp-lzo=no to correctly set comp-lzo=no-by-default
Product:	[Plasma] plasma-nm	Reporter:	Thomas Haller <thaller>
Component:	editor	Assignee:	Lukáš Tinkl <lukas>
Status:	RESOLVED FIXED
Severity:	normal	CC:	jgrulich, kde, matthias.nagel
Priority:	NOR
Version:	master
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	http://commits.kde.org/plasma-nm/2503210a9b218f3787a517d7dc19e338d32e4240	Version Fixed In:

Description Thomas Haller 2016-08-11 14:42:17 UTC

Related to bug 365816, plasma-nm should set "comp-lzo=no-by-default" instead of "comp-lzo=no".

Old versions of nm-openvpn and very new versions of nm-openvpn treat "comp-lzo=no" as if comp-lzo is not enabled at all. Only the 1.2.4 release treated comp-lzo=no that way, this however is now reverted again.

See https://bugzilla.gnome.org/show_bug.cgi?id=769177#c7 and https://git.gnome.org/browse/network-manager-openvpn/commit/?id=62bdd278d69cc396479af9d05a0b776d5ad386bf

Reproducible: Always

Comment 1 Jan Grulich 2016-08-17 13:06:23 UTC

Git commit 2503210a9b218f3787a517d7dc19e338d32e4240 by Jan Grulich.
Committed on 17/08/2016 at 12:43.
Pushed by grulich into branch 'master'.

Set correct value for comp-lzo property

M  +1    -1    vpn/openvpn/openvpnadvanced.ui
M  +2    -2    vpn/openvpn/openvpnadvancedwidget.cpp

http://commits.kde.org/plasma-nm/2503210a9b218f3787a517d7dc19e338d32e4240

Comment 2 Jan Grulich 2016-11-06 14:27:42 UTC

*** Bug 372086 has been marked as a duplicate of this bug. ***