Package: ircd-hybrid
Version: 1:7.2.2-2
Severity: minor
Tags: patch

Hi,

I've found the following problems with the
ircd-hybrid/servlink-path-notice template:
* the "error" type seems to be more appropriate for this template. See
  also the recent discussion on debian-devel about note templates
  displayed with a low or medium priority.
* the following line in your config file:
    if [ -f "/etc/ircd-hybrid/ircd.conf" ]; then
  tests the existance of the ircd.conf file, so the adjective "possibly"
  is pointless
* the following line in your config file:
    if egrep -q '[\t ]*[^#]+servlink_path' /etc/ircd-hybrid/ircd.conf;
  tests if there is a servlink_path directive, so the if-then
  construction of your sentence is pointless.

A patch with a suggested rephrasing is attached.

May I take the opportunity of this bug to ask you to send a request for
review of the French translation to debian-l10n-french?

Cheers,

-- 
Thomas Huriaux
--- ircd-hybrid.templates.orig  2006-09-23 18:11:57.000000000 +0200
+++ ircd-hybrid.templates       2006-09-23 18:13:01.000000000 +0200
@@ -1,8 +1,8 @@
 Template: ircd-hybrid/servlink-path-notice
-Type: note
-_Description: ircd-hybrid should not configure servlink_path variable
- If your possibly existing ircd.conf that you may wish to import contains a
- servlink_path directive, either change it to:
+Type: error
+_Description: servlink_path variable not configurable
+ Your existing ircd.conf that you may wish to import contains a
+ servlink_path directive. Either change it to:
  .
  servlink_path = "/usr/lib/ircd-hybrid/servlink";
  .

Attachment: signature.asc
Description: Digital signature

Reply via email to