Source: mjpegtools
Version: 1:2.0.0+debian-2
Severity: important
Tags: patch
User: debian-h...@lists.debian.org
Usertags: hurd
Control: forwarded -1 https://sourceforge.net/p/mjpeg/patches/53/

Hi,

currently [1] mjpegtools does not build on GNU/Hurd.

The problem is the unconditional usage of PATH_MAX and MAXPATHLEN,
neither of which is defined on Hurd.

I fixed the issues in a clean way (hopefully) and sent the patches
upstream [2]; since those patches can sound intrusive and they have no
feedback yet (reported them few minutes ago), I am asking to locally
apply in Debian the attached patch, which does a simplier ifndef/define
job (and should not break things).

Would it be possible to upload it sometime soon? I just discovered
mjpegtools is a build dependency for a couple of sources involved in
the currently ongoing libav9 transition, which then cannot be rebuilt
on Hurd due to the lack of mjpegtools.

[1] 
https://buildd.debian.org/status/fetch.php?pkg=mjpegtools&arch=hurd-i386&ver=1%3A2.0.0%2Bdebian-2&stamp=1351795704
[2] https://sourceforge.net/p/mjpeg/patches/53/

Thanks,
-- 
Pino
--- a/lavtools/editlist.c
+++ b/lavtools/editlist.c
@@ -26,6 +26,9 @@
 #include "editlist.h"
 #include <math.h>
 
+#ifndef PATH_MAX
+# define PATH_MAX 4096
+#endif
 
 /* Since we use malloc often, here the error handling */
 
--- a/lavtools/png2yuv.c
+++ b/lavtools/png2yuv.c
@@ -51,6 +51,10 @@ png2yuv
 #include "subsample.h"
 #include "colorspace.h"
 
+#ifndef PATH_MAX
+# define PATH_MAX 4096
+#endif
+
 #define DEFAULT_CHROMA_MODE Y4M_CHROMA_420JPEG
 
 typedef struct _parameters 
--- a/mplex/main.cpp
+++ b/mplex/main.cpp
@@ -73,6 +73,10 @@ lround(double x)
 };
 #endif
 
+#if	!defined(MAXPATHLEN)
+# define MAXPATHLEN 4096
+#endif
+
 
 
 class FileOutputStream : public OutputStream

Reply via email to