Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 766357 - net-vpn/openfortivpn: support for systemd
Summary: net-vpn/openfortivpn: support for systemd
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal minor (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-01-21 07:46 UTC by Alessandro Calorì
Modified: 2022-09-09 07:52 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alessandro Calorì 2021-01-21 07:46:18 UTC
Upstream sources include a systemd service [1] which would be nice to add to the ebuild in order to support always-on VPN's.

[1] https://raw.githubusercontent.com/adrienverge/openfortivpn/master/lib/systemd/system/openfortivpn%40.service.in
Comment 1 Alessandro Calorì 2021-01-21 08:14:46 UTC
The systemd service file needs to be modified:

sed -i 's/@BINDIR@/\/usr\/bin/g' openfortivpn@.service
sed -i 's/@SYSCONFDIR@/\/etc/g' openfortivpn@.service
Comment 2 Petru 2021-10-18 06:09:24 UTC
I've updated this pull request to include these changes. Let me know if it looks good to you. https://github.com/gentoo/gentoo/pull/22560
Comment 3 Larry the Git Cow gentoo-dev 2021-10-20 03:26:21 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=17b9dab6d7d190b40a4cd0b6f01c190d6a84a698

commit 17b9dab6d7d190b40a4cd0b6f01c190d6a84a698
Author:     Petru Ciobanu <scantlight@gmail.com>
AuthorDate: 2021-10-17 00:16:52 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2021-10-20 03:26:11 +0000

    net-vpn/openfortivpn: Substitute BINDIR and SYSCONFDIR in systemd unit file
    
    Those variables needs to be replaced with real location that contains
    the expected executable and configuration.
    
    Bug: https://bugs.gentoo.org/766357
    
    Package-Manager: Portage-3.0.20, Repoman-3.0.3
    Signed-off-by: Petru Ciobanu <scantlight@gmail.com>
    Signed-off-by: Sam James <sam@gentoo.org>

 .../files/systemd_substitute_bin_and_sysconfig_dirs.patch   | 13 +++++++++++++
 net-vpn/openfortivpn/openfortivpn-1.17.1.ebuild             |  4 ++++
 2 files changed, 17 insertions(+)
Comment 4 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2021-10-20 03:26:55 UTC
We should close this bug once the new versions are stabled.