Community Packages

Please read this before reporting a bug:
https://wiki.archlinux.org/title/Bug_reporting_guidelines

Do NOT report bugs when a package is just outdated, or it is in the AUR. Use the 'flag out of date' link on the package page, or the Mailing List.

REPEAT: Do NOT report bugs for outdated packages!
Tasklist

FS#60545 - [v2ray] Minor systemd service unit improvement

Attached to Project: Community Packages
Opened by Zhuoyun Wei (wzyboy) - Tuesday, 23 October 2018, 08:14 GMT
Last edited by Felix Yan (felixonmars) - Tuesday, 06 November 2018, 10:57 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Felix Yan (felixonmars)
Architecture All
Severity Very Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

Currently the v2ray.service file uses `ExecStart=/usr/bin/env` to set environment variables. This leads to `env` being shown in journalctl. It may be better to use `Environment=` to set environment variables in systemd service file.

Additional info:
* package version(s)
* config and/or log files etc.


Steps to reproduce:
This task depends upon

Closed by  Felix Yan (felixonmars)
Tuesday, 06 November 2018, 10:57 GMT
Reason for closing:  Fixed
Additional comments about closing:  4.0.2-1
Comment by Zhuoyun Wei (wzyboy) - Sunday, 04 November 2018, 04:26 GMT
  • Field changed: Percent Complete (100% → 0%)
/usr/lib/systemd/system/v2ray.service:15: Invalid environment assignment, ignoring: v2ray.location.asset=/etc/v2ray

Also, "v2ray@.service" should be updated as well.
Comment by Felix Yan (felixonmars) - Sunday, 04 November 2018, 05:45 GMT
Should be really fixed now in 4.0-2.
Comment by Weirch Sodora (sodora) - Sunday, 04 November 2018, 14:50 GMT
Hi Felix Yan,

Refer to https://www.v2ray.com/en/configuration/env.html
The environment variables should be v2ray.location.asset or V2RAY_LOCATION_ASSET.
But it is set to v2ray_location_asset in 4.0-2.

The path for 'ExecStart=' in v2ray.service is '/usr/bin/v2ray/v2ray'.
It should be '/usr/bin/v2ray'.
Comment by Felix Yan (felixonmars) - Sunday, 04 November 2018, 18:22 GMT
I got that name from a github issue: https://github.com/v2ray/v2ray-core/issues/702

But yeah the doc suggested a better name. Thanks for letting me know!

Both addressed in 4.0.2-1.

Loading...