Thank you for the patch. The problem has already been fixed in the master
branch, but in a different way.
Regards,
- Mark
$ git show 872cbb62b0142eabeb5d4b5dbb0bac558d94fc1a
commit 872cbb62b0142eabeb5d4b5dbb0bac558d94fc1a
Author: Will Newton <will.new...@linaro.org>
Date: Wed Aug 20 15:07:34 2014 +0100
src/common/knote.c: Portably print unitptr_t values
The format string for uintptr_t values is defined in inttypes.h
which is part of C99. Use that instead of having to special case
every 64bit architecture.
diff --git a/src/common/knote.c b/src/common/knote.c
index 3142f95..40d2232 100644
--- a/src/common/knote.c
+++ b/src/common/knote.c
@@ -14,6 +14,7 @@
* OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
*/
+#include <inttypes.h>
#include <stdlib.h>
#include <string.h>
#include <sys/types.h>
@@ -120,11 +121,7 @@ knote_lookup(struct filter *filt, uintptr_t ident)
ent = RB_FIND(knt, &filt->kf_knote, &query);
pthread_rwlock_unlock(&filt->kf_knote_mtx);
-#if defined(__x86_64__) || defined(__mips64)
- dbg_printf("id=%lu ent=%p", ident, ent);
-#else
- dbg_printf("id=%u ent=%p", ident, ent);
-#endif
+ dbg_printf("id=%" PRIuPTR " ent=%p", ident, ent);
return (ent);
}
On Tue, Sep 2, 2014 at 1:25 PM, Fernando Seiti Furusato <ferse...@br.ibm.com
> wrote:
> Source: libkqueue
> Severity: normal
> Tags: patch
> User: debian-powe...@lists.debian.org
> Usertags: ppc64el
>
> Dear Maintainer,
>
> The package libkqueue fails to build from source on ppc64el, because the
> specification
> to use unsigned long in knote.c file only covers x86_64.
>
> Do you think it can be considered to use the more generic preprocessor
> macro LP64 instead?
>
> Attached is a patch with the modification to do so.
>
> Thanks!
>
>
> -- System Information:
> Debian Release: jessie/sid
> APT prefers unstable
> APT policy: (500, 'unstable')
> Architecture: ppc64el (ppc64le)
>
> Kernel: Linux 3.13-1-powerpc64le (SMP w/4 CPU cores)
> Locale: LANG=en_US.utf8, LC_CTYPE=en_US.utf8 (charmap=UTF-8)
> Shell: /bin/sh linked to /bin/dash
>
