Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
syslog.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
8 Mar 2009 at 20:13 UTC
Updated:
24 Jul 2013 at 09:01 UTC
Jump to comment: Most recent, Most recent file



Comments
Comment #2
xanoI forgot to mention that. The patch was made outside of the Drupal root because of lack of CVS access. You will need to apply it using p1, IIRC.
Comment #3
xanoNeed to add OS X to the paragraph about UNIX/Linux.
Comment #4
xanoFollow-up patch. OS X is now mentioned as well.
Comment #5
keith.smith commentedIMO, if the individual operating system sections are to be separated out like this, they should not be as h2 level headings (where they will conflict with the help topic), but should probably be list items with a bolded section at the front describing the OS.
Comment #6
xanoThe headings are wrong in Garland, since the help topic should be a <h1>.
Comment #8
xanoThis patch fixes a bug in system_watchdog() that caused a PHP error if the hook was invoked for the first time.
I suggest we change this as well:
The strings we have now don't add anything to the codes themselves.
Comment #9
dries commentedThis looks good to me but I don't have a MS Windows environment to test this on.
Comment #11
xanoComment #12
brianV commentedThe Linux portions look good to me. However, I can't test the Windows portions.
Comment #14
xanoTagging this WTF because Windows users should *not* be able to select Linux facilities.
Comment #15
dries commentedCommitted to CVS HEAD. Thanks!
Comment #16
cburschkaI'd open a separate issue, but this is titled "cleanup" so it fits.
The help text contains a link to http://www.rt.com/man/syslog.5.html, which is a 404. This should be replaced with a more durable man link.
Comment #17
xanoWhat about http://www.linuxmanpages.com/man2/syslog.2.php?
Comment #18
moshe weitzman commentedpls open new issue.
Comment #20
xano