[ debian-hurd-Patches-301183 ] FTBFS and Debian issues - qt-x11-free-3.3.3
Patches item #301183, was opened at 2005-02-13 16:42
You can respond by visiting:
http://alioth.debian.org/tracker/?func=detail&atid=410472&aid=301183&group_id=30628
Category: several
Group: needs-work
Status: Open
Resolution: None
Priority: 5
Submitted By: Barry deFreese (bddebian-guest)
Assigned to: Nobody (None)
Summary: FTBFS and Debian issues - qt-x11-free-3.3.3
Initial Comment:
Author: Barry deFreese <bddebian@comcast.net>
Source: qt-x11-free
Status: posix issues fixed. Debian stuff needs work.
Categories: posix
Strip-Level: -p0
I am pasting in the posix fixes. I have posted my build log and my hacked up debian/rules, and debian/control files here:
http://www.bddebian.com/projects/qt/
diff -urp qt-x11-free-3.3.3.old/src/tools/qdir.cpp qt-x11-free-3.3.3/src/tools/qdir.cpp
--- qt-x11-free-3.3.3.old/src/tools/qdir.cpp 2005-02-13 05:11:10.000000000 -0500
+++ qt-x11-free-3.3.3/src/tools/qdir.cpp 2005-02-08 15:14:04.000000000 -0500
@@ -38,6 +38,10 @@
#include "qplatformdefs.h"
#include "qdir.h"
+#ifndef PATH_MAX
+# define PATH_MAX 4096
+#endif
+
#ifndef QT_NO_DIR
#include <private/qdir_p.h>
#include "qfileinfo.h"
diff -urp qt-x11-free-3.3.3.old/src/tools/qdir_unix.cpp qt-x11-free-3.3.3/src/tools/qdir_unix.cpp
--- qt-x11-free-3.3.3.old/src/tools/qdir_unix.cpp 2005-02-13 05:11:22.000000000 -0500
+++ qt-x11-free-3.3.3/src/tools/qdir_unix.cpp 2005-02-08 15:14:04.000000000 -0500
@@ -38,6 +38,10 @@
#include "qplatformdefs.h"
#include "qdir.h"
+#ifndef PATH_MAX
+# define PATH_MAX 4096
+#endif
+
#ifndef QT_NO_DIR
#include "qdir_p.h"
diff -urp qt-x11-free-3.3.3.old/src/tools/qfileinfo_unix.cpp qt-x11-free-3.3.3/src/tools/qfileinfo_unix.cpp
--- qt-x11-free-3.3.3.old/src/tools/qfileinfo_unix.cpp 2005-02-13 05:11:34.000000000 -0500
+++ qt-x11-free-3.3.3/src/tools/qfileinfo_unix.cpp 2005-02-08 15:14:04.000000000 -0500
@@ -41,6 +41,10 @@
#include "qdatetime.h"
#include "qdir.h"
+#ifndef PATH_MAX
+# define PATH_MAX 4096
+#endif
+
#include <limits.h>
----------------------------------------------------------------------
>Comment By: Michael Banck (mbanck)
Date: 2006-08-06 16:09
Message:
Logged In: YES
user_id=2380
And some more:
<youpi> Mmm, they're writing ::getcwd()
<youpi> shouldn't we write ::get_current_dir_name() too ?
<youpi> else the patch seems fine
----------------------------------------------------------------------
Comment By: Michael Banck (mbanck)
Date: 2006-08-06 16:07
Message:
Logged In: YES
user_id=2380
Some comments to abeaumont's patch:
<youpi> I'd say PATH_CHUNK_SIZE should be a power of 2 :)
<youpi> why should it be #if defined(Q_OS_HURD) ? shouldn't it be #if !defined(PATH_MAX) ?
<youpi> else the algorithm looks sane
<youpi> (for ReadLink)
<youpi> canonicalPath is strange: r should be initialized
<youpi> because in case tmp is null, r is not defined to null
----------------------------------------------------------------------
You can respond by visiting:
http://alioth.debian.org/tracker/?func=detail&atid=410472&aid=301183&group_id=30628
Reply to: