Community Packages

Please read this before reporting a bug:
https://wiki.archlinux.org/title/Bug_reporting_guidelines

Do NOT report bugs when a package is just outdated, or it is in the AUR. Use the 'flag out of date' link on the package page, or the Mailing List.

REPEAT: Do NOT report bugs for outdated packages!
Tasklist

FS#38890 - [prosody] Couple of suggestion regarding logging

Attached to Project: Community Packages
Opened by x33a (x33a) - Thursday, 13 February 2014, 12:35 GMT
Last edited by Sergej Pupykin (sergej) - Friday, 14 February 2014, 11:17 GMT
Task Type General Gripe
Category Packages
Status Closed
Assigned To Sergej Pupykin (sergej)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

Since prosody logs to syslog by default now, I suggest that the following points be considered:

1. Don't create /var/log/prosody, it clutters up /var/log unnecessarily. If the user decides to log under that directory, he can create the directory himself.

2. Remove prosody.logrotated. Again, with the default configuration, this file is useless.

Since, I had var/log/prosody in NoExtract, I got the following error:

chown: cannot access ‘var/log/prosody’: No such file or directory

If you do decide to create /var/log/prosody, at least check for it's presence before doing a chown over it.

- chown prosody:prosody var/log/prosody
+
+ if [ -e /var/log/prosody ]; then
+ chown prosody:prosody var/log/prosody
+ fi


Package version: Prosody 0.9.3-1
This task depends upon

Closed by  Sergej Pupykin (sergej)
Friday, 14 February 2014, 11:17 GMT
Reason for closing:  Fixed
Comment by Sergej Pupykin (sergej) - Friday, 14 February 2014, 09:58 GMT
/var/log is FHS directory, see "man hier".

About last suggestion: I've changed chown to

[ -d var/log/prosody ] && chown prosody:prosody var/log/prosody

in svn trunk.
Comment by x33a (x33a) - Friday, 14 February 2014, 10:33 GMT
> /var/log is FHS directory, see "man hier".

I was referring to /var/log/prosody. It is packaged with prosody. But with the default configuration, i.e., logging to syslog, this directory remains empty.

> [ -d var/log/prosody ] && chown prosody:prosody var/log/prosody

Thanks!

Loading...