Your message dated Tue, 19 Dec 2006 06:02:03 +0000 with message-id <[EMAIL PROTECTED]> and subject line Bug#403546: fixed in cedar-backup2 2.9.0-1 has caused the attached Bug report to be marked as done.
This means that you claim that the problem has been dealt with. If this is not the case it is now your responsibility to reopen the Bug report if necessary, and/or fix the problem forthwith. (NB: If you are a system administrator and have no idea what I am talking about this indicates a serious mail system misconfiguration somewhere. Please contact me immediately.) Debian bug tracking system administrator (administrator, Debian Bugs database)
--- Begin Message ---Package: cedar-backup2 Version: 2.8.1-1 Severity: wishlist Tags: patch ide-scsi is obsolete on 2.6 kernels, cdrecord (presently known as wodim in Debian by the way) can use recorders just like ==== wodim dev=/dev/hdc ... ==== and cannot (as far as I can see) use it the only way cedar-backup2 can use, unlesss ide-scsi is configured. To this end I've made the following trivial modifications which seem to fix the problem for me: Index: CedarBackup2/writer.py =================================================================== --- CedarBackup2/writer.py (revision 1011) +++ CedarBackup2/writer.py (working copy) @@ -105,7 +105,7 @@ raise ValueError("Backup device is not writable by the current user.") return device -def validateScsiId(scsiId): +def validateScsiId(scsiId, device): """ Validates a SCSI id string. SCSI id must be a string in the form C{[ATA:|ATAPI:]scsibus,target,lun}. @@ -115,6 +115,9 @@ @return: SCSI id as a string, suitable for assignment to C{CdWriter.scsiId}. @raise ValueError: If the SCSI id string is invalid. """ + if scsiId is None: + return device + pattern = re.compile(r"^\s*(?:ATA:|ATAPI:)?\s*[0-9][0-9]*\s*,\s*[0-9][0-9]*\s*,\s*[0-9][0-9]*\s*$") if not pattern.search(scsiId): pattern = re.compile(r"^\s*IO.*Services(\/[0-9][0-9]*)?\s*$") @@ -432,7 +435,7 @@ @raise IOError: If device properties could not be read for some reason. """ self._device = _validateDevice(device, unittest) - self._scsiId = validateScsiId(scsiId) + self._scsiId = validateScsiId(scsiId, device) self._driveSpeed = validateDriveSpeed(driveSpeed) self._media = MediaDefinition(mediaType) if not unittest: Index: CedarBackup2/config.py =================================================================== --- CedarBackup2/config.py (revision 1011) +++ CedarBackup2/config.py (working copy) @@ -4567,8 +4567,8 @@ raise ValueError("Store section media type must be filled in.") if self.store.devicePath is None: raise ValueError("Store section device path must be filled in.") - if self.store.deviceScsiId is None: - raise ValueError("Store section SCSI id must be filled in.") +# if self.store.deviceScsiId is None: +# raise ValueError("Store section SCSI id must be filled in.") def _validatePurge(self): """ With that patch if there is no target_scsi_id tag in the configuration file a string between target_device tags is used for dev=... cdrecord parameter. My Python knowledge however is next to None, and the patch breaks some of the unittests. I wish I had more time to look into that. -- System Information: Debian Release: 4.0 APT prefers unstable APT policy: (500, 'unstable') Architecture: i386 (i686) Shell: /bin/sh linked to /bin/bash Kernel: Linux 2.6.18.1 Locale: LANG=ru_RU.UTF-8, LC_CTYPE=ru_RU.UTF-8 (charmap=UTF-8) Versions of packages cedar-backup2 depends on: ii python 2.4.4-2 An interactive high-level object-o ii python-support 0.5.6 automated rebuilding support for p Versions of packages cedar-backup2 recommends: ii cdrecord 9:1.1.0-1 Dummy transition package for wodim ii cedar-backup2-doc 2.8.1-1 local and remote backups to CD-R/C ii eject 2.1.4-2.1 ejects CDs and operates CD-Changer ii mkisofs 9:1.1.0-1 Dummy transition package for genis ii ssh 1:4.3p2-7 Secure shell client and server (tr -- no debconf information
--- End Message ---
--- Begin Message ---Source: cedar-backup2 Source-Version: 2.9.0-1 We believe that the bug you reported is fixed in the latest version of cedar-backup2, which is due to be installed in the Debian FTP archive: cedar-backup2-doc_2.9.0-1_all.deb to pool/main/c/cedar-backup2/cedar-backup2-doc_2.9.0-1_all.deb cedar-backup2_2.9.0-1.diff.gz to pool/main/c/cedar-backup2/cedar-backup2_2.9.0-1.diff.gz cedar-backup2_2.9.0-1.dsc to pool/main/c/cedar-backup2/cedar-backup2_2.9.0-1.dsc cedar-backup2_2.9.0-1_all.deb to pool/main/c/cedar-backup2/cedar-backup2_2.9.0-1_all.deb cedar-backup2_2.9.0.orig.tar.gz to pool/main/c/cedar-backup2/cedar-backup2_2.9.0.orig.tar.gz A summary of the changes between this version and the previous one is attached. Thank you for reporting the bug, which will now be closed. If you have further comments please address them to [EMAIL PROTECTED], and the maintainer will reopen the bug report if appropriate. Debian distribution maintenance software pp. Kenneth J. Pronovici <[EMAIL PROTECTED]> (supplier of updated cedar-backup2 package) (This message was generated automatically at their request; if you believe that there is a problem with it please contact the archive administrators by mailing [EMAIL PROTECTED]) -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Format: 1.7 Date: Mon, 18 Dec 2006 22:59:17 -0600 Source: cedar-backup2 Binary: cedar-backup2 cedar-backup2-doc Architecture: source all Version: 2.9.0-1 Distribution: unstable Urgency: low Maintainer: Kenneth J. Pronovici <[EMAIL PROTECTED]> Changed-By: Kenneth J. Pronovici <[EMAIL PROTECTED]> Description: cedar-backup2 - local and remote backups to CD-R/CD-RW media cedar-backup2-doc - local and remote backups to CD-R/CD-RW media (documentation) Closes: 403448 403546 403662 Changes: cedar-backup2 (2.9.0-1) unstable; urgency=low . * New upstream release. - Provide way to configure dev=/dev/cdrw and the like (closes: #403546). - Fix, clean up and reorganize parts of user manual (closes: #403448, #403662). Files: b568453414cf9c46eddd6b7e25b78f6e 642 admin optional cedar-backup2_2.9.0-1.dsc c1a74eea176068309191ff3a6a178724 1256376 admin optional cedar-backup2_2.9.0.orig.tar.gz ba5235fed7fe23dfce398e511a745ff6 8349 admin optional cedar-backup2_2.9.0-1.diff.gz 7a63846df1556967ea1dcaa41b31fa17 161116 admin optional cedar-backup2_2.9.0-1_all.deb e5235a3cdfd7234a93bc3c94e3cc2568 776826 doc optional cedar-backup2-doc_2.9.0-1_all.deb -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.6 (GNU/Linux) iD8DBQFFh30a8On2ujzZUQQRAsS7AJ9fiP/KDO6HOs6iUQ3kGBNN9zorSQCg5gnY W7IJBsElAnI8j/HVnQqmnAg= =5DyU -----END PGP SIGNATURE-----
--- End Message ---
