14:31 Mathieu Arnold (mat) 

all: Remove all other $FreeBSD keywords.

    135fdee

13:44 ohauer 

- fix broken rc script from last commit.
  (var do not expandinside singel quote, so the port failed to start)
- bump PORTREVISION

PR:		196090 (followup)
Approved by:	maintainer (implicit)

 

13:25 rafan 

- rc script enhancements
- pass maintainership
- bump PORTREVISION

PR:		196090
Submitted by:	geoffroy desvernay <dgeo at centrale-marseille.fr>

 

03:40 rafan 

Don't package runtime rrd directory location

 

00:28 dougb 

Fix what looks like a copy-paste-o that originated in the same script:

command=foo >/dev/null 2>&1
	|
	v
command=foo
command_args='>/dev/null 2>&1'

This is clearly what should have been done, for several reasons.

No PORTREVISION bump because the old version simply ignored everything
after the space, and does not seem to have done any harm. However
it's good to clean these up so that similar errors aren't pasted into
a new script where they might actually matter.

08:57 dougb 

In the rc.d scripts, change assignments to rcvar to use the
literal name_enable wherever possible, and ${name}_enable
when it's not, to prepare for the demise of set_rcvar().

In cases where I had to hand-edit unusual instances also
modify formatting slightly to be more uniform (and in
some cases, correct). This includes adding some $FreeBSD$
tags, and most importantly moving rcvar= to right after
name= so it's clear that one is derived from the other.

00:15 dougb 

Begin the process of deprecating sysutils/rc_subr by
s#. %%RC_SUBR%%#. /etc/rc.subr#

02:24 rafan 

- Make rc variables can be properly set by users. Default variables should be
  handled after load_rc_config per Handbook. For example, if user sets
  name_pidfile in rc.conf, it does not correctly pass to name_flags because
  when we set name_flags, it uses our default name_pidfile, but not user's.
- Bump PORTREVISION as rc scripts changed.

Approved by:    portmgr (pav), Xavier Beaudouin <kiwi at oav.net> (ovs
maintainer)

13:47 rafan 

- Remove MAILGRAPH_GROUP from pre-everything message and mailgraph_group
  from rc script to avoid user confusion. We can not setgid for command
  invoked from rc.subr. [1]
- Change rc script to match our guideline in Porter's Handbook

Noticed by:     Nicolas Rachinsky <nicolas-2007 at rachinsky.de> [1]

13:40 rafan 

- Strip .sh suffix for rc script for upcoming changes