Summary: | rc/sh/openrc-run.sh: Wrong usage of cgroup_add_service | ||
---|---|---|---|
Product: | Gentoo Hosted Projects | Reporter: | Patrick Lauer <patrick> |
Component: | OpenRC | Assignee: | OpenRC Team <openrc> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Patrick Lauer
2017-11-29 13:13:12 UTC
I can't figure out from git history when this changed - 2391ddff772ce541bc2237c89a2792a0b85e076c adds: + if [ -d "$1" ]; then + cgroup="$1/$RC_SVCNAME" with that the syntax in openrc-run.sh makes sense. But I don't see it removed, and I don't see it in the current file, so I'm very wtf Ah, it was undone in the merge commit of itself - 88cdcc145eb6343b94626ab3910a5eb1f8654212 merges the branch with the chance, had a conflict, and accidentally drops the important bit: -cgroup_set_limits() +cgroup_add_service() { - if [ -d "$1" ]; then - cgroup="$1/$RC_SVCNAME" This has been fixed in the following commit: http://github.com/openrc/openrc/commit/cee3919 |