[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1055077: checkinstall: FTBFS on riscv64: Please check the INSTW_ROOTPATH and be sure that it does exist please !



Package: checkinstall
Version: 1.6.2+git20170426.d24a630-4
Tags: patch
User: debian-riscv@lists.debian.org
Usertags: riscv64
X-Debbugs-Cc: debian-riscv@lists.debian.org

Dear maintainer,

checkinstall fails to build from source on riscv64:

| gcc -Wall -DVERSION=\"0.7.0beta7\" -DLIBDIR=\"/<<PKGBUILDDIR>>/debian/testtmp/lib\" -o test-installwatch test-installwatch.c -ldl
| PREFIX=/<<PKGBUILDDIR>>/debian/testtmp /<<PKGBUILDDIR>>/debian/testtmp/bin/installwatch ./test-installwatch
| 
| INFO : Using a default root directory : /tmp/tmp.Ik8H2pmjZz
| 
| Please check the INSTW_ROOTPATH and be sure that it does exist please !
| given value : /tmp/tmp.Ik8H2pmjZz
| make[2]: *** [Makefile:50: test] Error 1
| make[2]: Leaving directory '/<<PKGBUILDDIR>>/installwatch'
| make[1]: *** [debian/rules:22: override_dh_auto_test] Error 2
| make[1]: Leaving directory '/<<PKGBUILDDIR>>'
| make: *** [debian/rules:7: build-arch] Error 2
| dpkg-buildpackage: error: debian/rules build-arch subprocess returned exit status 2

The full build log is available there:
https://buildd.debian.org/status/fetch.php?pkg=checkinstall&arch=riscv64&ver=1.6.2%2Bgit20170426.d24a630-4&stamp=1698621687&raw=0

After investigation, it appears that the definition of _STAT_VERand
_MKNOD_VER is not correct for riscv64. Please find below a patch to fix
that:

--- checkinstall-1.6.2+git20170426.d24a630/debian/patches/0016-Define-_STAT_VER-_MKNOD_VER-locally-dropped-in-glibc.patch
+++ checkinstall-1.6.2+git20170426.d24a630/debian/patches/0016-Define-_STAT_VER-_MKNOD_VER-locally-dropped-in-glibc.patch
@@ -25,7 +25,7 @@
  #endif	
  
 +#ifndef _STAT_VER
-+ #if defined (__aarch64__)
++ #if defined (__aarch64__) || defined(__riscv)
 +  #define _STAT_VER 0
 + #elif defined (__s390x__)
 +  # define _STAT_VER 1
@@ -36,7 +36,7 @@
 + #endif
 +#endif
 +#ifndef _MKNOD_VER
-+ #if defined (__aarch64__)
++ #if defined (__aarch64__) || defined(__riscv)
 +  #define _MKNOD_VER 0
 + #elif defined (__x86_64__)
 +  #define _MKNOD_VER 0

Would it be possible to include it in the next upload? Thanks in
advance.

Regards
Aurelien


Reply to: