Bug 213641

Summary:	www/otrs: Update to 5.0.13, Minor improvements
Product:	Ports & Packages	Reporter:	knezour
Component:	Individual Port(s)	Assignee:	freebsd-ports-bugs (Nobody) <ports-bugs>
Status:	Closed Overcome By Events
Severity:	Affects Only Me	CC:	joneum, m.tsatsenko, miwi
Priority:	---	Keywords:	needs-patch, needs-qa
Version:	Latest	Flags:	bugzilla: maintainer-feedback? (m.tsatsenko)
Hardware:	Any
OS:	Any
URL:	https://www.otrs.com/release-notes-otrs-5-patch-level-13/

Description knezour 2016-10-20 10:00:56 UTC

OTRS 5.0.13 is available.

Additionally, following may be done to improve the port:

There is new bin/otrs.Daemon.pl daemon which can be integrated into our rc.d infrastructure.

Parameters to the bin/otrs.SetPermissions.pl has been changed, not all mentioned in the pkg-message are used.

Following pkgs have to be installed manually on my system to keep OTRS happy
p5-Schedule-Cron-Events
p5-Excel-Writer-XLSX
p5-HTML-Truncate

I would try to find time to provide some patches, but someone may have time to fix earlier.

Comment 1 m.tsatsenko 2016-10-31 23:18:24 UTC

Hello, thanks for informing me.
I try to have a look at update and rc.d script when I have some spare time, however as far I remember OTRS has bundled versions of the modules you have mentioned. So if you had to install them manually you most likely doing something wrong.

Comment 2 m.tsatsenko 2016-12-09 21:42:49 UTC

Hello,
I just submitted bug 215182 which has patch to update to 5.0.14, pkg-message is also updated.
However creating rc.d script requires a running instance of otrs which I don't have at the moment, but as always patches are welcome!

Comment 3 knezour 2016-12-14 06:32:41 UTC

Hi,

this is working for me as /usr/local/etc/rc.d/otrs, but it is my first try on rc scripting, so it may not be the best possible way
--------------
#!/bin/sh

# PROVIDE: otrs

. /etc/rc.subr

name="otrs"
rcvar=otrs_enable

load_rc_config $name
: ${otrs_enable:=no}

otrs_user=otrs

command_interpreter=/usr/local/bin/perl
command=/usr/local/otrs/bin/otrs.Daemon.pl

start_postcmd="su -m otrs -c '/usr/local/otrs/bin/Cron.sh start'"
stop_precmd="su -m otrs -c '/usr/local/otrs/bin/Cron.sh stop'"
status_cmd="su -m otrs -c '/usr/local/otrs/bin/otrs.Daemon.pl status'"
start_cmd="su -m otrs -c '/usr/local/otrs/bin/otrs.Daemon.pl start'"
stop_cmd="su -m otrs -c '/usr/local/otrs/bin/otrs.Daemon.pl stop'"

run_rc_command "$1"

Comment 4 m.tsatsenko 2017-01-07 22:28:20 UTC

(In reply to knezour from comment #3)
Hello
Unfortunately the script you have provided is far from complete and I can not approve it. Please have a look at rc.d scripting chapter in the porters handbook: https://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/rc-scripts.html

Comment 5 knezour 2017-01-08 15:07:28 UTC

Hello, 

in fact it is done by the Porters Handbook, but it is my first take on rc scripting so I am aware that it is probably far from ideal.

I will check Handbook chapter on rc scripting once more, if you can pinpoint what is wrong or may be done better, I would highly appreciate it. Simple list of missing things or errors is fine, I will look up the rest.

Comment 6 Martin Wilke freebsd_committer

2017-05-06 14:13:32 UTC

Hi,

any update here?

Comment 7 knezour 2017-05-06 14:26:55 UTC

No, some guidance still requested :)