Package: cvs
Version: 1:1.12.13-3
Severity: important
Tags: patch

Hi.

Please find attached a tiny patch, which fixes a rather strange (IMHO)
behaviour in lib/chdir-long.c:
 - an error is issued, telling the world that this file has not to be
   compiled, when PATH_MAX is not defined;

instead of:
 -  putting (that part of) the file between #ifdef/#endif

which is done once this (dbs) patch is applied.

I also checked that the package still builds on (linux-)i386.

Cheers,

-- 
Cyril Brulebois
diff -ruN cvs-1.12.13-old/lib/chdir-long.c cvs-1.12.13/lib/chdir-long.c
--- cvs-1.12.13-old/lib/chdir-long.c	2005-09-19 21:12:18.000000000 +0000
+++ cvs-1.12.13/lib/chdir-long.c	2006-07-20 02:47:32.000000000 +0000
@@ -39,9 +39,7 @@
 # define O_DIRECTORY 0
 #endif
 
-#ifndef PATH_MAX
-# error "compile this file only if your system defines PATH_MAX"
-#endif
+#ifdef PATH_MAX
 
 struct cd_buf
 {
@@ -269,6 +267,8 @@
 }
 #endif
 
+#endif /* PATH_MAX */
+
 /*
 Local Variables:
 compile-command: "gcc -DTEST_CHDIR=1 -DHAVE_CONFIG_H -I.. -g -O -W -Wall chdir-long.c libcoreutils.a"

Reply via email to