cvs commit: ports/net-im/prosody Makefile
ports/net-im/prosody/files prosody.in
Doug Barton
dougb at FreeBSD.org
Fri Jun 11 22:23:11 UTC 2010
On 06/11/10 14:53, David E. Thiel wrote:
> On Fri, Jun 11, 2010 at 01:29:36PM -0700, Doug Barton wrote:
>> There is a problem with the change here in that the literal /usr/local
>> should be replaced with %%PREFIX%%. However the whole script needs some
>> work.
>
> Thanks Doug, I should have caught that.
No worries. :)
>> 4. Simplify the status and stop methods down to one function that takes
>> an argument.
>
> This works mostly well, but I notice that the "restart" command
> doesn't work,
What output does it give?
> and was going to add a restart_cmd telling it to issue a
> prosodyctl stop followed by prosodyctl start.
That's probably reasonable.
> I kind of think it would
> look cleaner to have start_cmd, stop_cmd and restart_cmd rather than
> prosody_cmd and restart_cmd. Thoughts?
The idea behind prosody_cmd is that it can handle all of the normal
arguments in one method, rather than needing to duplicate essentially
the same thing numerous times. That's what functions are for. :)
The way start is handled currently is actually sort of clever, which is
why I did not suggest changing it. However I _think_ the following will
work, but it should be tested:
start_cmd="prosody_cmd start"
stop_cmd="prosody_cmd stop"
restart_cmd="$start_cmd $stop_cmd"
status_cmd="prosody_cmd status"
This will also require removing command_args. I've updated the version
at http://people.freebsd.org/~dougb/prosody.in.txt accordingly.
If the restart_cmd above does not work, then the solution you suggested
is fine. I should also point out that I don't particularly care if you
and/or the maintainer decide to use the prosody_cmd idea or not. I have
no authority to do anything other than suggest, although in this case
_I_ happen to think it's a rather clever suggestion. :)
Doug
--
... and that's just a little bit of history repeating.
-- Propellerheads
Improve the effectiveness of your Internet presence with
a domain name makeover! http://SupersetSolutions.com/
More information about the cvs-ports
mailing list