Bug 373609 - [patch] rename comp-lzo value "No by default" in openvpn GUI
Summary: [patch] rename comp-lzo value "No by default" in openvpn GUI
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_networkmanagement (other bugs)
Version First Reported In: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Lukáš Tinkl
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-12-13 11:49 UTC by Thomas Haller
Modified: 2024-12-23 18:27 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments
[patch] openvpn: rename comp-lzo setting "No by default" in the UI (1.19 KB, application/mbox)
2016-12-13 11:49 UTC, Thomas Haller
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Haller 2016-12-13 11:49:07 UTC
Created attachment 102764 [details]
[patch] openvpn: rename comp-lzo setting "No by default" in the UI

Quote:


    openvpn: rename comp-lzo setting "No by default" in the UI
    
    To express a "comp-lzo no" setting in nm-openvpn, it must be stored as
    "comp-lzo=no-by-default". That is to preserve backward compatiblity,
    where older versions would wrongly set "comp-lzo=no" to mean LZO
    compression disabled [1].
    
    This "no-by-default" term is only relevant for the connection, the UI should
    call it "no", like openvpn's manual page.
    
    [1] https://git.gnome.org/browse/network-manager-openvpn/commit/?id=62bdd278d69cc396479af9d05a0b776d5ad386bf
Comment 1 Jan Grulich 2016-12-13 13:55:06 UTC
Git commit c6d6d380f9133e1286a9022e9895d28b5c7dcb45 by Jan Grulich, on behalf of Thomas Haller.
Committed on 13/12/2016 at 13:54.
Pushed by grulich into branch 'master'.

openvpn: rename comp-lzo setting "No by default" in the UI

To express a "comp-lzo no" setting in nm-openvpn, it must be stored as
"comp-lzo=no-by-default". That is to preserve backward compatiblity,
where older versions would wrongly set "comp-lzo=no" to mean LZO
compression disabled [1].

This "no-by-default" term is only relevant for the connection, the UI should
call it "no", like openvpn's manual page.

[1] https://git.gnome.org/browse/network-manager-openvpn/commit/?id=62bdd278d69cc396479af9d05a0b776d5ad386bf

M  +1    -1    vpn/openvpn/openvpnadvanced.ui

https://commits.kde.org/plasma-nm/c6d6d380f9133e1286a9022e9895d28b5c7dcb45