Bug 58216

Summary:	syslog-ng init script should do configuration check
Product:	Gentoo Linux	Reporter:	Eldad Zack (RETIRED) <eldad>
Component:	Current packages	Assignee:	Mr. Bones. (RETIRED) <mr_bones_>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	High
Version:	unspecified
Hardware:	All
OS:	All
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	syslog-ng.rc.1.13.patch syslog-ng.init.1.13.patch syslog-ng.init.1.13.patch

Description Eldad Zack (RETIRED) gentoo-dev

2004-07-24 12:13:18 UTC

syslog-ng -s (configfile) provide with a configuration check.

Comment 1 Eldad Zack (RETIRED) gentoo-dev

2004-07-24 12:14:54 UTC

Created attachment 36085 [details, diff]
syslog-ng.rc.1.13.patch

Added the syslog-ng -s check, both with reload and start.

I've also added SYSLOGNG_CONF instead of /etc/syslog-ng/syslog-ng.conf
everywhere.

Comment 2 Eldad Zack (RETIRED) gentoo-dev

2004-07-25 05:26:47 UTC

Created attachment 36109 [details, diff]
syslog-ng.init.1.13.patch

Learned the hard way that using variables messes with depcache.
This is a much smaller patch.

Comment 3 Eldad Zack (RETIRED) gentoo-dev

2004-07-25 05:27:59 UTC

Created attachment 36110 [details, diff]
syslog-ng.init.1.13.patch

former patch was reversed.

Comment 4 Mr. Bones. (RETIRED) gentoo-dev

2004-07-25 17:36:13 UTC

So.... Explain to me what this patch actually buys us over the current behavior please?

Comment 5 Eldad Zack (RETIRED) gentoo-dev

2004-07-25 23:07:22 UTC

suppose you misconfigure syslog-ng and try to restart it - it will stop but won't start. 

Also, if it fails to start, it tells you why.

Comment 6 Mr. Bones. (RETIRED) gentoo-dev

2004-08-04 18:05:51 UTC

Added to CVS.  No rev bump or anything since this is pretty minor.  Thanks for the bug report and patch.