Red Hat Bugzilla – Bug 1259666
fix ftbfs on aarch64 in condor
Last modified: 2015-09-23 00:08:30 EDT
Created attachment 1069725 [details]
patch + spec changeset
Description of problem:
Condor defines uint64_t type but this is conflicting with definition in stdint.h and build fails.
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1. do build on aarch64 (use arm-koji)
In file included from /usr/lib/gcc/aarch64-redhat-linux/5.1.1/include/stdint.h:9:0,
/usr/include/stdint.h:40:19: error: conflicting declaration 'typedef long int int64_t'
typedef long int int64_t;
Fix is not perfect but solved problem.
Thank you very much for the bug report!
To me, this appears to be a cmake issue - cmake was supposed to determine whether this typedef already existed (I suspect the typedef in condor is for old compilers for windows / solaris).
Is there any way to pull CMakeCache.txt from the failed build?
Will do a build and attach it.
Created attachment 1071611 [details]
Ah - I see the issue:
CMake properly detects int64_t already detects. *However*, this particular file doesn't include the config.h because the header read_user_log.h is meant to be part of the public API.
Probably this header needs to be generated separately by cmake.
Will file a ticket with upstream HTCondor.
For now, the proposed patch looks fine.
> For now, the proposed patch looks fine.
Can we get it applied and pushed for F-23+ then?
pushed to rawhide
condor-8.3.6-4.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-16124
condor-8.3.6-4.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update condor'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-16124
condor-8.3.6-4.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.