From: <es...@th...> - 2004-02-17 06:05:49
|
This is automatically generated email about problems in a man page for which you appear to be responsible. If you are not the right person or list, tell me and I will attempt to correct my database. See http://catb.org/~esr/doclifter/problems.html for details on how and why these patches were generated. Feel free to email me with any questions. Note: This patch does not change the mod date of the manual page. You may wish to do that by hand. Problems with smartd.8: 1. Invalid comment syntax. --- smartd.8-orig 2004-02-16 15:53:10.000000000 -0500 +++ smartd.8 2004-02-16 15:54:27.000000000 -0500 @@ -1,21 +1,21 @@ -\# Copyright (C) 2002-3 Bruce Allen <sma...@li...> -\# -\# $Id: smartd.8.in,v 1.6 2003/10/13 14:31:34 arvoreen Exp $ -\# -\# This program is free software; you can redistribute it and/or modify it -\# under the terms of the GNU General Public License as published by the Free -\# Software Foundation; either version 2, or (at your option) any later -\# version. -\# -\# You should have received a copy of the GNU General Public License (for -\# example COPYING); if not, write to the Free Software Foundation, Inc., 675 -\# Mass Ave, Cambridge, MA 02139, USA. -\# -\# This code was originally developed as a Senior Thesis by Michael Cornwell -\# at the Concurrent Systems Laboratory (now part of the Storage Systems -\# Research Center), Jack Baskin School of Engineering, University of -\# California, Santa Cruz. http://ssrc.soe.ucsc.edu/ -\# +.\" Copyright (C) 2002-3 Bruce Allen <sma...@li...> +.\" +.\" $Id: smartd.8.in,v 1.6 2003/10/13 14:31:34 arvoreen Exp $ +.\" +.\" This program is free software; you can redistribute it and/or modify it +.\" under the terms of the GNU General Public License as published by the Free +.\" Software Foundation; either version 2, or (at your option) any later +.\" version. +.\" +.\" You should have received a copy of the GNU General Public License (for +.\" example COPYING); if not, write to the Free Software Foundation, Inc., 675 +.\" Mass Ave, Cambridge, MA 02139, USA. +.\" +.\" This code was originally developed as a Senior Thesis by Michael Cornwell +.\" at the Concurrent Systems Laboratory (now part of the Storage Systems +.\" Research Center), Jack Baskin School of Engineering, University of +.\" California, Santa Cruz. http://ssrc.soe.ucsc.edu/ +.\" .TH SMARTD 8 "$Date: 2003/10/13 14:31:34 $" smartmontools-5.21 .SH NAME \fBsmartd\fP \- SMART Disk Monitoring Daemon @@ -297,9 +297,9 @@ .nf .B /sbin/chkconfig --del smartd -\# DO NOT MODIFY THIS OR THE FOLLOWING TWO LINES. THIS MATERIAL -\# IS AUTOMATICALLY INCLUDED IN THE FILE smartd.conf.5 -\# STARTINCLUDE +.\" DO NOT MODIFY THIS OR THE FOLLOWING TWO LINES. THIS MATERIAL +.\" IS AUTOMATICALLY INCLUDED IN THE FILE smartd.conf.5 +.\" STARTINCLUDE .SH CONFIGURATION FILE /etc/smartd.conf In the absence of a configuration file, @@ -1170,9 +1170,9 @@ reside in directories like /tmp that are writable by ordinary users and may expose your system to symlink attacks. -\# ENDINCLUDE -\# DO NOT MODIFY THIS OR PREVIOUS/NEXT LINES. THIS DEFINES THE -\# END OF THE INCLUDE SECTION FOR smartd.conf.5 +.\" ENDINCLUDE +.\" DO NOT MODIFY THIS OR PREVIOUS/NEXT LINES. THIS DEFINES THE +.\" END OF THE INCLUDE SECTION FOR smartd.conf.5 .SH NOTES .B smartd Problems with smartctl.8: 1. Invalid comment syntax. --- smartctl.8-orig 2004-02-16 16:01:01.000000000 -0500 +++ smartctl.8 2004-02-16 16:01:15.000000000 -0500 @@ -1,21 +1,21 @@ -\# Copyright (C) 2002-3 Bruce Allen <sma...@li...> -\# -\# $Id: smartctl.8.in,v 1.8 2003/10/13 14:31:34 arvoreen Exp $ -\# -\# This program is free software; you can redistribute it and/or modify it -\# under the terms of the GNU General Public License as published by the Free -\# Software Foundation; either version 2, or (at your option) any later -\# version. -\# -\# You should have received a copy of the GNU General Public License (for -\# example COPYING); if not, write to the Free Software Foundation, Inc., 675 -\# Mass Ave, Cambridge, MA 02139, USA. -\# -\# This code was originally developed as a Senior Thesis by Michael Cornwell -\# at the Concurrent Systems Laboratory (now part of the Storage Systems -\# Research Center), Jack Baskin School of Engineering, University of -\# California, Santa Cruz. http://ssrc.soe.ucsc.edu/ -\# +.\" Copyright (C) 2002-3 Bruce Allen <sma...@li...> +.\" +.\" $Id: smartctl.8.in,v 1.8 2003/10/13 14:31:34 arvoreen Exp $ +.\" +.\" This program is free software; you can redistribute it and/or modify it +.\" under the terms of the GNU General Public License as published by the Free +.\" Software Foundation; either version 2, or (at your option) any later +.\" version. +.\" +.\" You should have received a copy of the GNU General Public License (for +.\" example COPYING); if not, write to the Free Software Foundation, Inc., 675 +.\" Mass Ave, Cambridge, MA 02139, USA. +.\" +.\" This code was originally developed as a Senior Thesis by Michael Cornwell +.\" at the Concurrent Systems Laboratory (now part of the Storage Systems +.\" Research Center), Jack Baskin School of Engineering, University of +.\" California, Santa Cruz. http://ssrc.soe.ucsc.edu/ +.\" .TH SMARTCTL 8 "$Date: 2003/10/13 14:31:34 $" smartmontools-5.21 .SH NAME \fBsmartctl\fP \- Control and Monitor Utility for SMART Disks @@ -714,7 +714,7 @@ Executes TEST immediately. The \'\-C\' option can be used in conjunction with this option to run the short or long (and also for ATA devices, -\# selective or +.\" selective or conveyance) self-tests in captive mode (known as "foreground mode" for SCSI devices). Note that only one test can be run at a time, so this option should only be used once per command @@ -770,14 +770,14 @@ complete. Note that this command can be given during normal system operation (unless run in captive mode \- see the \'\-C\' option below). -\#.I selective -\#\- [ATA ONLY: \fBNOT YET IMPLEMENTED\fP] runs a SMART Selective Self Test. -\#This self-test routine is intended to check specific ranges of disk -\#Logical Block Addresses (LBAs). Each range that is checked is called -\#a "span". Each span is specified by a starting LBA and an ending LBA. -\#Up to 5 spans can be specified. Note that this command can be given -\#during normal system operation (unless run in captive mode \- see -\#the \'\-C\' option below). +.\".I selective +.\"\- [ATA ONLY: \fBNOT YET IMPLEMENTED\fP] runs a SMART Selective Self Test. +.\"This self-test routine is intended to check specific ranges of disk +.\"Logical Block Addresses (LBAs). Each range that is checked is called +.\"a "span". Each span is specified by a starting LBA and an ending LBA. +.\"Up to 5 spans can be specified. Note that this command can be given +.\"during normal system operation (unless run in captive mode \- see +.\"the \'\-C\' option below). .TP .B \-C, \-\-captive Problems with smartd.conf.5: 1. Invalid comment syntax. --- smartd.conf.5-orig 2004-02-17 00:39:50.000000000 -0500 +++ smartd.conf.5 2004-02-17 00:40:15.000000000 -0500 @@ -1,24 +1,24 @@ -\# Copyright (C) 2002-3 Bruce Allen <sma...@li...> -\# -\# $Id: smartd.conf.5.in,v 1.5 2003/10/13 14:31:34 arvoreen Exp $ -\# -\# This program is free software; you can redistribute it and/or modify it -\# under the terms of the GNU General Public License as published by the Free -\# Software Foundation; either version 2, or (at your option) any later -\# version. -\# -\# You should have received a copy of the GNU General Public License (for -\# example COPYING); if not, write to the Free Software Foundation, Inc., 675 -\# Mass Ave, Cambridge, MA 02139, USA. -\# -\# This code was originally developed as a Senior Thesis by Michael Cornwell -\# at the Concurrent Systems Laboratory (now part of the Storage Systems -\# Research Center), Jack Baskin School of Engineering, University of -\# California, Santa Cruz. http://ssrc.soe.ucsc.edu/ -\# +.\" Copyright (C) 2002-3 Bruce Allen <sma...@li...> +.\" +.\" $Id: smartd.conf.5.in,v 1.5 2003/10/13 14:31:34 arvoreen Exp $ +.\" +.\" This program is free software; you can redistribute it and/or modify it +.\" under the terms of the GNU General Public License as published by the Free +.\" Software Foundation; either version 2, or (at your option) any later +.\" version. +.\" +.\" You should have received a copy of the GNU General Public License (for +.\" example COPYING); if not, write to the Free Software Foundation, Inc., 675 +.\" Mass Ave, Cambridge, MA 02139, USA. +.\" +.\" This code was originally developed as a Senior Thesis by Michael Cornwell +.\" at the Concurrent Systems Laboratory (now part of the Storage Systems +.\" Research Center), Jack Baskin School of Engineering, University of +.\" California, Santa Cruz. http://ssrc.soe.ucsc.edu/ +.\" .TH SMARTD.CONF 5 "$Date: 2003/10/13 14:31:34 $" smartmontools-5.21 .SH NAME -\fBsmartd.conf\fP \- SMART Disk Monitoring Daemon Configuration File \- \fBsmartmontools-5.21\fP +\fBsmartd.conf\fP \- SMART Disk Monitoring Daemon Configuration File for \fBsmartmontools-5.21\fP .SH PACKAGE VERSION smartmontools-5.21 "$Date: 2003/10/13 14:31:34 $" @@ -37,9 +37,9 @@ \fB\<CONTROL-C\>\fP in the terminal window where \fBsmartd\fP is running. -\# DO NOT MODIFY THIS OR THE FOLLOWING TWO LINES. WHAT FOLLOWS -\# IS AUTOMATICALLY INCLUDED FROM THE FILE smartd.8 -\# STARTINCLUDE +.\" DO NOT MODIFY THIS OR THE FOLLOWING TWO LINES. WHAT FOLLOWS +.\" IS AUTOMATICALLY INCLUDED FROM THE FILE smartd.8 +.\" STARTINCLUDE .SH CONFIGURATION FILE /etc/smartd.conf In the absence of a configuration file, @@ -910,9 +910,9 @@ reside in directories like /tmp that are writable by ordinary users and may expose your system to symlink attacks. -\# ENDINCLUDE -\# DO NOT MODIFY THIS OR PREVIOUS/NEXT LINES. THIS DEFINES THE -\# END OF THE INCLUDED SECTION FROM smartd.8 +.\" ENDINCLUDE +.\" DO NOT MODIFY THIS OR PREVIOUS/NEXT LINES. THIS DEFINES THE +.\" END OF THE INCLUDED SECTION FROM smartd.8 .PP .SH AUTHOR -- Eric S. Raymond |
From: Bruce A. <ba...@gr...> - 2004-02-17 19:07:50
|
Hi Eric, Thanks for reporting the problems with the man page below. Unfortunately the versions you commented on (eg, CVS version 1.6 from 2003/10/13) is a few months out of data. The current versions of the two man pages are at: http://cvs.sourceforge.net/viewcvs.py/smartmontools/sm5/smartd.8.in?view=markup http://cvs.sourceforge.net/viewcvs.py/smartmontools/sm5/smartctl.8.in?view=markup I think that the problems you pointed out (comment syntax) have been fixed, using a combination of './' and '.ig' syntax. But if there is an automated way to check the syntax of these current versions in CVS, I'd be grateful. Thank you. (And thanks also for the open source movement and advocacy!) Cheers, Bruce On Tue, 17 Feb 2004 es...@th... wrote: > This is automatically generated email about problems in a man page for which > you appear to be responsible. If you are not the right person or list, tell > me and I will attempt to correct my database. > > See http://catb.org/~esr/doclifter/problems.html for details on how and > why these patches were generated. Feel free to email me with any questions. > > Note: This patch does not change the mod date of the manual page. You > may wish to do that by hand. > > Problems with smartd.8: > > 1. Invalid comment syntax. > > --- smartd.8-orig 2004-02-16 15:53:10.000000000 -0500 > +++ smartd.8 2004-02-16 15:54:27.000000000 -0500 > @@ -1,21 +1,21 @@ > -\# Copyright (C) 2002-3 Bruce Allen <sma...@li...> > -\# > -\# $Id: smartd.8.in,v 1.6 2003/10/13 14:31:34 arvoreen Exp $ > -\# > -\# This program is free software; you can redistribute it and/or modify it > -\# under the terms of the GNU General Public License as published by the Free > -\# Software Foundation; either version 2, or (at your option) any later > -\# version. > -\# > -\# You should have received a copy of the GNU General Public License (for > -\# example COPYING); if not, write to the Free Software Foundation, Inc., 675 > -\# Mass Ave, Cambridge, MA 02139, USA. > -\# > -\# This code was originally developed as a Senior Thesis by Michael Cornwell > -\# at the Concurrent Systems Laboratory (now part of the Storage Systems > -\# Research Center), Jack Baskin School of Engineering, University of > -\# California, Santa Cruz. http://ssrc.soe.ucsc.edu/ > -\# > +.\" Copyright (C) 2002-3 Bruce Allen <sma...@li...> > +.\" > +.\" $Id: smartd.8.in,v 1.6 2003/10/13 14:31:34 arvoreen Exp $ > +.\" > +.\" This program is free software; you can redistribute it and/or modify it > +.\" under the terms of the GNU General Public License as published by the Free > +.\" Software Foundation; either version 2, or (at your option) any later > +.\" version. > +.\" > +.\" You should have received a copy of the GNU General Public License (for > +.\" example COPYING); if not, write to the Free Software Foundation, Inc., 675 > +.\" Mass Ave, Cambridge, MA 02139, USA. > +.\" > +.\" This code was originally developed as a Senior Thesis by Michael Cornwell > +.\" at the Concurrent Systems Laboratory (now part of the Storage Systems > +.\" Research Center), Jack Baskin School of Engineering, University of > +.\" California, Santa Cruz. http://ssrc.soe.ucsc.edu/ > +.\" > .TH SMARTD 8 "$Date: 2003/10/13 14:31:34 $" smartmontools-5.21 > .SH NAME > \fBsmartd\fP \- SMART Disk Monitoring Daemon > @@ -297,9 +297,9 @@ > .nf > .B /sbin/chkconfig --del smartd > > -\# DO NOT MODIFY THIS OR THE FOLLOWING TWO LINES. THIS MATERIAL > -\# IS AUTOMATICALLY INCLUDED IN THE FILE smartd.conf.5 > -\# STARTINCLUDE > +.\" DO NOT MODIFY THIS OR THE FOLLOWING TWO LINES. THIS MATERIAL > +.\" IS AUTOMATICALLY INCLUDED IN THE FILE smartd.conf.5 > +.\" STARTINCLUDE > > .SH CONFIGURATION FILE /etc/smartd.conf > In the absence of a configuration file, > @@ -1170,9 +1170,9 @@ > reside in directories like /tmp that are writable by ordinary users > and may expose your system to symlink attacks. > > -\# ENDINCLUDE > -\# DO NOT MODIFY THIS OR PREVIOUS/NEXT LINES. THIS DEFINES THE > -\# END OF THE INCLUDE SECTION FOR smartd.conf.5 > +.\" ENDINCLUDE > +.\" DO NOT MODIFY THIS OR PREVIOUS/NEXT LINES. THIS DEFINES THE > +.\" END OF THE INCLUDE SECTION FOR smartd.conf.5 > > .SH NOTES > .B smartd > Problems with smartctl.8: > > 1. Invalid comment syntax. > > --- smartctl.8-orig 2004-02-16 16:01:01.000000000 -0500 > +++ smartctl.8 2004-02-16 16:01:15.000000000 -0500 > @@ -1,21 +1,21 @@ > -\# Copyright (C) 2002-3 Bruce Allen <sma...@li...> > -\# > -\# $Id: smartctl.8.in,v 1.8 2003/10/13 14:31:34 arvoreen Exp $ > -\# > -\# This program is free software; you can redistribute it and/or modify it > -\# under the terms of the GNU General Public License as published by the Free > -\# Software Foundation; either version 2, or (at your option) any later > -\# version. > -\# > -\# You should have received a copy of the GNU General Public License (for > -\# example COPYING); if not, write to the Free Software Foundation, Inc., 675 > -\# Mass Ave, Cambridge, MA 02139, USA. > -\# > -\# This code was originally developed as a Senior Thesis by Michael Cornwell > -\# at the Concurrent Systems Laboratory (now part of the Storage Systems > -\# Research Center), Jack Baskin School of Engineering, University of > -\# California, Santa Cruz. http://ssrc.soe.ucsc.edu/ > -\# > +.\" Copyright (C) 2002-3 Bruce Allen <sma...@li...> > +.\" > +.\" $Id: smartctl.8.in,v 1.8 2003/10/13 14:31:34 arvoreen Exp $ > +.\" > +.\" This program is free software; you can redistribute it and/or modify it > +.\" under the terms of the GNU General Public License as published by the Free > +.\" Software Foundation; either version 2, or (at your option) any later > +.\" version. > +.\" > +.\" You should have received a copy of the GNU General Public License (for > +.\" example COPYING); if not, write to the Free Software Foundation, Inc., 675 > +.\" Mass Ave, Cambridge, MA 02139, USA. > +.\" > +.\" This code was originally developed as a Senior Thesis by Michael Cornwell > +.\" at the Concurrent Systems Laboratory (now part of the Storage Systems > +.\" Research Center), Jack Baskin School of Engineering, University of > +.\" California, Santa Cruz. http://ssrc.soe.ucsc.edu/ > +.\" > .TH SMARTCTL 8 "$Date: 2003/10/13 14:31:34 $" smartmontools-5.21 > .SH NAME > \fBsmartctl\fP \- Control and Monitor Utility for SMART Disks > @@ -714,7 +714,7 @@ > Executes TEST immediately. The \'\-C\' option can be used in > conjunction with this option to run the short or long (and also for ATA > devices, > -\# selective or > +.\" selective or > conveyance) self-tests in captive mode (known > as "foreground mode" for SCSI devices). Note that only one test can be > run at a time, so this option should only be used once per command > @@ -770,14 +770,14 @@ > complete. Note that this command can be given during normal system > operation (unless run in captive mode \- see the \'\-C\' option below). > > -\#.I selective > -\#\- [ATA ONLY: \fBNOT YET IMPLEMENTED\fP] runs a SMART Selective Self Test. > -\#This self-test routine is intended to check specific ranges of disk > -\#Logical Block Addresses (LBAs). Each range that is checked is called > -\#a "span". Each span is specified by a starting LBA and an ending LBA. > -\#Up to 5 spans can be specified. Note that this command can be given > -\#during normal system operation (unless run in captive mode \- see > -\#the \'\-C\' option below). > +.\".I selective > +.\"\- [ATA ONLY: \fBNOT YET IMPLEMENTED\fP] runs a SMART Selective Self Test. > +.\"This self-test routine is intended to check specific ranges of disk > +.\"Logical Block Addresses (LBAs). Each range that is checked is called > +.\"a "span". Each span is specified by a starting LBA and an ending LBA. > +.\"Up to 5 spans can be specified. Note that this command can be given > +.\"during normal system operation (unless run in captive mode \- see > +.\"the \'\-C\' option below). > > .TP > .B \-C, \-\-captive > Problems with smartd.conf.5: > > 1. Invalid comment syntax. > > --- smartd.conf.5-orig 2004-02-17 00:39:50.000000000 -0500 > +++ smartd.conf.5 2004-02-17 00:40:15.000000000 -0500 > @@ -1,24 +1,24 @@ > -\# Copyright (C) 2002-3 Bruce Allen <sma...@li...> > -\# > -\# $Id: smartd.conf.5.in,v 1.5 2003/10/13 14:31:34 arvoreen Exp $ > -\# > -\# This program is free software; you can redistribute it and/or modify it > -\# under the terms of the GNU General Public License as published by the Free > -\# Software Foundation; either version 2, or (at your option) any later > -\# version. > -\# > -\# You should have received a copy of the GNU General Public License (for > -\# example COPYING); if not, write to the Free Software Foundation, Inc., 675 > -\# Mass Ave, Cambridge, MA 02139, USA. > -\# > -\# This code was originally developed as a Senior Thesis by Michael Cornwell > -\# at the Concurrent Systems Laboratory (now part of the Storage Systems > -\# Research Center), Jack Baskin School of Engineering, University of > -\# California, Santa Cruz. http://ssrc.soe.ucsc.edu/ > -\# > +.\" Copyright (C) 2002-3 Bruce Allen <sma...@li...> > +.\" > +.\" $Id: smartd.conf.5.in,v 1.5 2003/10/13 14:31:34 arvoreen Exp $ > +.\" > +.\" This program is free software; you can redistribute it and/or modify it > +.\" under the terms of the GNU General Public License as published by the Free > +.\" Software Foundation; either version 2, or (at your option) any later > +.\" version. > +.\" > +.\" You should have received a copy of the GNU General Public License (for > +.\" example COPYING); if not, write to the Free Software Foundation, Inc., 675 > +.\" Mass Ave, Cambridge, MA 02139, USA. > +.\" > +.\" This code was originally developed as a Senior Thesis by Michael Cornwell > +.\" at the Concurrent Systems Laboratory (now part of the Storage Systems > +.\" Research Center), Jack Baskin School of Engineering, University of > +.\" California, Santa Cruz. http://ssrc.soe.ucsc.edu/ > +.\" > .TH SMARTD.CONF 5 "$Date: 2003/10/13 14:31:34 $" smartmontools-5.21 > .SH NAME > -\fBsmartd.conf\fP \- SMART Disk Monitoring Daemon Configuration File \- \fBsmartmontools-5.21\fP > +\fBsmartd.conf\fP \- SMART Disk Monitoring Daemon Configuration File for \fBsmartmontools-5.21\fP > .SH PACKAGE VERSION > smartmontools-5.21 "$Date: 2003/10/13 14:31:34 $" > > @@ -37,9 +37,9 @@ > \fB\<CONTROL-C\>\fP in the terminal window where \fBsmartd\fP is > running. > > -\# DO NOT MODIFY THIS OR THE FOLLOWING TWO LINES. WHAT FOLLOWS > -\# IS AUTOMATICALLY INCLUDED FROM THE FILE smartd.8 > -\# STARTINCLUDE > +.\" DO NOT MODIFY THIS OR THE FOLLOWING TWO LINES. WHAT FOLLOWS > +.\" IS AUTOMATICALLY INCLUDED FROM THE FILE smartd.8 > +.\" STARTINCLUDE > > .SH CONFIGURATION FILE /etc/smartd.conf > In the absence of a configuration file, > @@ -910,9 +910,9 @@ > reside in directories like /tmp that are writable by ordinary users > and may expose your system to symlink attacks. > > -\# ENDINCLUDE > -\# DO NOT MODIFY THIS OR PREVIOUS/NEXT LINES. THIS DEFINES THE > -\# END OF THE INCLUDED SECTION FROM smartd.8 > +.\" ENDINCLUDE > +.\" DO NOT MODIFY THIS OR PREVIOUS/NEXT LINES. THIS DEFINES THE > +.\" END OF THE INCLUDED SECTION FROM smartd.8 > > .PP > .SH AUTHOR > > -- > Eric S. Raymond > > > ------------------------------------------------------- > SF.Net is sponsored by: Speed Start Your Linux Apps Now. > Build and deploy apps & Web services for Linux with > a free DVD software kit from IBM. Click Now! > http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click > _______________________________________________ > Smartmontools-support mailing list > Sma...@li... > https://lists.sourceforge.net/lists/listinfo/smartmontools-support > > |
From: Eric S. R. <es...@th...> - 2004-02-17 19:29:42
|
Bruce Allen <ba...@gr...>: > Thanks for reporting the problems with the man page below. Unfortunately > the versions you commented on (eg, CVS version 1.6 from 2003/10/13) is a > few months out of data. The current versions of the two man pages are at: > > http://cvs.sourceforge.net/viewcvs.py/smartmontools/sm5/smartd.8.in?view=markup > http://cvs.sourceforge.net/viewcvs.py/smartmontools/sm5/smartctl.8.in?view=markup > > I think that the problems you pointed out (comment syntax) have been > fixed, using a combination of './' and '.ig' syntax. But if there is an > automated way to check the syntax of these current versions in CVS, I'd be > grateful. I checked, they're both good. You might want to see if you can get an update into the Fedora repository. -- <a href="http://www.catb.org/~esr/">Eric S. Raymond</a> |
From: Bruce A. <ba...@gr...> - 2004-02-17 19:54:30
|
> > I think that the problems you pointed out (comment syntax) have been > > fixed, using a combination of './' and '.ig' syntax. But if there is an > > automated way to check the syntax of these current versions in CVS, I'd be > > grateful. > > I checked, they're both good. Thanks. > You might want to see if you can get an update into the Fedora > repository. Currently smartmontools is part of the kernel-utils RPM, maintained by Arjan van de Ven. I've had some email discussions with Arjan and with Arjan's manager Michael K. Johnson asking them to move smartmontools out of kernel-utils and make it a separate standalone package, which would simplify the process of updating the Fedora version of smartmontools. But it hasn't happened yet. Cheers, Bruce |
From: Eric S. R. <es...@th...> - 2004-02-17 20:47:22
|
Bruce Allen <ba...@gr...>: > Currently smartmontools is part of the kernel-utils RPM, maintained by > Arjan van de Ven. I've had some email discussions with Arjan and with > Arjan's manager Michael K. Johnson asking them to move smartmontools out > of kernel-utils and make it a separate standalone package, which would > simplify the process of updating the Fedora version of smartmontools. Something else where smartmontools could use some work is initial setup of the smartd.conf file. The present default configuration is broken on any SCSI-based system, which is a bit more of an issue than it might seem because it makes Fedora Core 1 think the boot sequence is broken and flip out of graphical-bootup mode (maybe panicking some hapless end-user in the process). To prevent this, the "very silent check" should be commented out. It wouldn't be too difficult to write a postinstall script for the RPM that would customize the smartd.conf file by looking at the contents of /etc/fstab. In fact, here is a proof of concept (tested). Alternatively, is there any reason your device scan doesn't use this information? #!/usr/bin/env python # # Hack the /etc/smartd.conf file to match /etc/fstab # # Proof of concept by Eric S. Raymond, 17 Feb 2004 import sys, re # Snarf the filesystem table fsfp = open("/etc/fstab") fstabdevs = fsfp.readlines() fsfp.close() # Throw out what we don't need. End up with a list of physical device stems fstabdevs = map(lambda x: x.split()[0], fstabdevs) fstabdevs = filter(lambda x: x.find('/dev/') == 0, fstabdevs) fstabdevs = map(lambda x: re.sub("[0-9]*$", "", x), fstabdevs) # Filter /etc/smartd.conf, commenting/uncommenting as needed sc = open("/etc/smartd.conf") while True: line = sc.readline() if not line: break line = re.sub("#\s+", "#", line) if line.startswith("/dev/"): if not line.split()[0] in fstabdevs: line = "#" + line if line.startswith("#/dev/"): if line[1:].split()[0] in fstabdevs: line = line[1:] sys.stdout.write(line) sc.close() # end -- <a href="http://www.catb.org/~esr/">Eric S. Raymond</a> |
From: Guido G. <ag...@si...> - 2004-02-18 21:43:22
|
On Tue, Feb 17, 2004 at 03:34:45PM -0500, Eric S. Raymond wrote: > Something else where smartmontools could use some work is initial > setup of the smartd.conf file. The present default configuration is Debian uses a smard.conf that only has the DEVICESCAN directive enabled, maybe that would be o.k. for fedora too. -- Guido |