Your message dated Tue, 18 Oct 2011 15:19:13 +0000
with message-id <[email protected]>
and subject line Bug#622365: fixed in tofrodos 1.7.9.debian.1-1
has caused the Debian Bug report #622365,
regarding Fix FTBFS of tofrodos for GNU/Hurd
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 this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
622365: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=622365
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: tofrodos
Version: 1.7.8.debian.1-2
Severity: important
Tags: patch
Usertags: hurd

Attached is a patch to enable successful build of tofrodos on
GNU/Hurd. It should be applied after the FTBFS_kfreebsd.diff patch.
The only question is if I've missed a free() somewhere. In my opinion it
does not make sense to add a free() after a return statement, does it?

Making this package build enables latex2rtf to be built that is on the
old-package list:
http://ftp-master.debian.org/users/twerner/pre-lenny.txt

Additionally, it makes debian-cd installable since it depends on this
package. 
diff -ur ./tofrodos-1.7.8.debian.1/src/config.h tofrodos-1.7.8.debian.1.new/src/config.h
--- ./tofrodos-1.7.8.debian.1/src/config.h	2011-04-12 16:10:02.000000000 +0200
+++ tofrodos-1.7.8.debian.1.new/src/config.h	2011-04-12 16:18:06.000000000 +0200
@@ -102,7 +102,7 @@
 #endif
 #endif
 
-#if defined(__FreeBSD__) || defined(__OpenBSD__)	/* seems to work like Linux... */
+#if defined(__FreeBSD__) || defined(__OpenBSD__) || defined(__GNU__)	/* seems to work like Linux... */
 #if !defined(LINUX)
 #define	LINUX
 #endif
diff -ur ./tofrodos-1.7.8.debian.1/src/tofrodos.c tofrodos-1.7.8.debian.1.new/src/tofrodos.c
--- ./tofrodos-1.7.8.debian.1/src/tofrodos.c	2008-04-08 06:57:05.000000000 +0200
+++ tofrodos-1.7.8.debian.1.new/src/tofrodos.c	2011-04-12 16:22:22.000000000 +0200
@@ -14,7 +14,9 @@
 #include <stdlib.h>	/* EXIT_SUCCESS, mkstemp() in some systems */
 #include <string.h>	/* strrchr(), strlen(), strcpy(), strcat() */
 #include <sys/stat.h>	/* stat() */
-
+#ifdef __GNU__
+#include <limits.h>
+#endif
 #if defined(UNIX)
 #if defined(MAXPATHLEN_HEADER)
 #include MAXPATHLEN_HEADER
@@ -443,8 +445,12 @@
 static int openandconvert_preamble ( char * filename )
 {
 	struct stat statbuf ;
+#ifndef __GNU__
 	char		realfilepath[MAXPATHLEN+1] ;
 	int			len ;
+#else
+	char		*realfilepath ;
+#endif
 
 	/* get the file information */
 	if (lstat( filename, &statbuf )) {
@@ -461,16 +467,24 @@
 		/* eg, #define S_ISLNK(x) (((x) & S_IFMT) == S_IFLNK) */
 		/* or something like that. */
 
+#ifndef __GNU__
 		if ((len = readlink( filename, realfilepath, (sizeof(realfilepath) - 1) )) != -1) {
 			/* got to null terminate the string - there is always space because */
 			/* we passed readlink() the size of the buffer less 1. */
 			realfilepath[len] = '\0' ;
+#else
+			realfilepath = realpath (filename, NULL);
+			if ((realfilepath) != NULL) {
+#endif
 			if (verbose) {
 				emsg( VERBOSE_SYMLINKSRC, filename, realfilepath );
 			}
 			return openandconvert( realfilepath );
 		}
 		emsg( EMSG_SYMLINK, filename );
+#ifdef __GNU__
+		free (realfilepath);
+#endif
 		return -1 ;
 	}
 	/* If we reach here, "filename" is not a symbolic link */

--- End Message ---
--- Begin Message ---
Source: tofrodos
Source-Version: 1.7.9.debian.1-1

We believe that the bug you reported is fixed in the latest version of
tofrodos, which is due to be installed in the Debian FTP archive:

tofrodos_1.7.9.debian.1-1.diff.gz
  to main/t/tofrodos/tofrodos_1.7.9.debian.1-1.diff.gz
tofrodos_1.7.9.debian.1-1.dsc
  to main/t/tofrodos/tofrodos_1.7.9.debian.1-1.dsc
tofrodos_1.7.9.debian.1-1_amd64.deb
  to main/t/tofrodos/tofrodos_1.7.9.debian.1-1_amd64.deb
tofrodos_1.7.9.debian.1.orig.tar.gz
  to main/t/tofrodos/tofrodos_1.7.9.debian.1.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.
Alexander Reichle-Schmehl <[email protected]> (supplier of updated tofrodos 
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: SHA256

Format: 1.8
Date: Tue, 18 Oct 2011 16:46:35 +0200
Source: tofrodos
Binary: tofrodos
Architecture: source amd64
Version: 1.7.9.debian.1-1
Distribution: unstable
Urgency: low
Maintainer: Alexander Reichle-Schmehl <[email protected]>
Changed-By: Alexander Reichle-Schmehl <[email protected]>
Description: 
 tofrodos   - Converts DOS <-> Unix text files, alias tofromdos
Closes: 616439 622365
Changes: 
 tofrodos (1.7.9.debian.1-1) unstable; urgency=low
 .
   * NOT RELEASED YET
   * Fix Vcs-Svn header in debian/control
   * New upstream release
   * Add recommended targets build-arch and build-indep to debian/rules
   * Enable LFS support (Closes: #616439), thanks to Lionel Elie Mamane for
     the hint
   * Bump standards to 3.9.2 (no changes needed)
   * Apply patch from Guillem Jover to fix build on linux arches (drop
     patches/FTBFS_kfreebsd.diff, new patch is
     patches/FTBFS_non-linux.diff). Also thanks to  Svante Signell  for his
     inital work! (Closes: #622365)
Checksums-Sha1: 
 b997fe729ccf90528dc31921808aa8aa8759d257 1924 tofrodos_1.7.9.debian.1-1.dsc
 161e5ccabbfd3e474d0deccbf2b2494773c3b811 39961 
tofrodos_1.7.9.debian.1.orig.tar.gz
 45bb6c6fb44cba3191ea3d318eb1d799391b43f2 6719 tofrodos_1.7.9.debian.1-1.diff.gz
 be5317fcddcf7dbafc3bba91086bbdb560ceaab7 21876 
tofrodos_1.7.9.debian.1-1_amd64.deb
Checksums-Sha256: 
 eb4595e07e1f07640dafcc76e527165effda45aa0e76854a41022ff6bbea3ec9 1924 
tofrodos_1.7.9.debian.1-1.dsc
 269b57f1c2d42293ed46e54f9a88bff0c1626004565ac2b78fbc5f7d62d58673 39961 
tofrodos_1.7.9.debian.1.orig.tar.gz
 d0144fd303dce17ddfe2dc23faffdd998e1c8f703be41006fb8b79d22019351f 6719 
tofrodos_1.7.9.debian.1-1.diff.gz
 d5b3c16d9f06c7d53971a39ce51d710855b06ffa8ea33bae33db09c31d6362b9 21876 
tofrodos_1.7.9.debian.1-1_amd64.deb
Files: 
 1f7d2dbdd8dce5367cae081000da9f59 1924 utils optional 
tofrodos_1.7.9.debian.1-1.dsc
 471349607dda268de169993532b31650 39961 utils optional 
tofrodos_1.7.9.debian.1.orig.tar.gz
 130252367525faaa9dbdf16c82ab2c75 6719 utils optional 
tofrodos_1.7.9.debian.1-1.diff.gz
 c55f31d0e26b424a73fe7c517cdba837 21876 utils optional 
tofrodos_1.7.9.debian.1-1_amd64.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)

iQIcBAEBCAAGBQJOnZH3AAoJEMJLZaJnLIsSbqMQAIiPfDy+BrEla1YuuApcaO4d
Hf+eFM8J6zK5oBcDLxZ1C40UpYDWdPyTktYdpkshXg8cozQjZAJi0zEdo3gRi74F
FOZ2qKsMoohVFBN49AQQbDdAFqVVPBrPMw2KUHaTXQAme241xeGSzXWETFx7K7eV
1YrIrHqHfaJ22HhbxIkj+fA6xv1yCkPnm7utMFcJQNS6tOkdjneNcOPALYDHTvon
IjIdEcuJvE78Ar/I/pdqZtGumn2CFzcsY3FhuLiam6EUT7JM6hf/14veISBRBotX
KHm5Qh59IiN1ES8UCqy9E9vNL5rdnUdXBSMk6BiLRB1NJcp2beugC/JvtRds7mAA
tnrw1DKhy/o1SlB5kremRnZoQyG6mpiRRMBC3oNPS80e44Ex6f53Tn/POq+c++kt
QpIjjs6g3sPLimpfiTb9XaqnfTsLvcgPH24IAu2TUt3jYxwVAAXvpisneoHslMEw
xlmWtQxvWOQhBiautjlNueWE8B/6ViZubJPdDXU9fuRheH2/kjTxhfxOmlETPjBo
vY0fr912vDhWN9UUtTLGiWwuTc5eA1VYkCigFgARDvHXNOqhFYw+TOnydUiHdbQB
Sh0mARh9zXynFg8f+uBjmdU/JRsH6AdIOEx41+3M35quVl9RqzCAvcfPaVGkcVBC
cgeLPceruWfAnXp4K5tf
=lUB6
-----END PGP SIGNATURE-----



--- End Message ---

Reply via email to