Summary: | app-vim/gentoo-syntax - improve initd skel | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Sergey S. Starikoff <Ikonta> |
Component: | Current packages | Assignee: | Vim Maintainers <vim> |
Status: | RESOLVED FIXED | ||
Severity: | enhancement | CC: | bkohler, mgorny, yngwin |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: | https://github.com/gentoo/gentoo-syntax/pull/21 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Sergey S. Starikoff
2017-01-08 16:56:05 UTC
I don't understand what's obsolete about this, can you explain? A skel file isn't supposed to be a detailed example. (In reply to Ben Kohler from comment #1) > I don't understand what's obsolete about this, can you explain? A skel file > isn't supposed to be a detailed example. Skel file isn't supposed to be a detailed example, but as for me, it is supposed to be up to date and corresponds with some minimal-typical example. For recent versions of OpenRC it means, that the only probably needed section is depend. But we need to declare some variables. Header of example in man 8 openrc-run (possibly with empty values) looks sufficiently: #!/sbin/openrc-run # Copyright 1999-2017 Gentoo Foundation # Distributed under the terms of the GNU General Public License v2 # $Id$ command=/usr/bin/foo command_args="${foo_args} --bar" pidfile=/var/run/foo.pid name="FooBar Daemon" description="FooBar is a daemon that eats and drinks" depend() { } Please see/test the linked PR. I didn't include pidfile= since I'm not sure if it's really needed and if it's not, people will keep adding it unnecessarily. The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/proj/gentoo-syntax.git/commit/?id=8cbe8adba2433766276be8e8a09a827fce811aa9 commit 8cbe8adba2433766276be8e8a09a827fce811aa9 Author: Michał Górny <mgorny@gentoo.org> AuthorDate: 2018-08-20 15:54:44 +0000 Commit: Michał Górny <mgorny@gentoo.org> CommitDate: 2018-08-20 20:47:04 +0000 newinitd: Update for the modern declarative syntax Bug: https://bugs.gentoo.org/605078 plugin/newinitd.vim | 22 +++++++++++++--------- 1 file changed, 13 insertions(+), 9 deletions(-) The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=daddf63e4386056e4f71945f9d5ed3247ac1e6e6 commit daddf63e4386056e4f71945f9d5ed3247ac1e6e6 Author: Michał Górny <mgorny@gentoo.org> AuthorDate: 2018-08-20 20:50:21 +0000 Commit: Michał Górny <mgorny@gentoo.org> CommitDate: 2018-08-20 21:06:43 +0000 app-vim/gentoo-syntax: Bump to 20180820 Closes: https://bugs.gentoo.org/637104 Closes: https://bugs.gentoo.org/637100 Closes: https://bugs.gentoo.org/605078 app-vim/gentoo-syntax/Manifest | 1 + .../gentoo-syntax/gentoo-syntax-20180820.ebuild | 42 ++++++++++++++++++++++ 2 files changed, 43 insertions(+) |