[patch] Better detection of sizes of numeric types
This patch changes how preprocessor macros are used to detect which numeric types to handle; the existing behaviour assumes that there's only Windows/non-Windows, but there's actually more variability than that and many different architectures fail to build bornagain because of that. This patch relies instead on the actual differenciation between types.
Activity
Roland, thank you very much.
Which architectures, operating systems, compilers do you want to support beyond the four architecture/OS combinations for which we are currently providing binary installers?
Is this patch sufficient? No corrections in our CMake scripts needed? Currently we boldly assume that it's either (gcc or clang) on (Linux or Mac) or MSVC on Windows.
Can you possibly contribute this patch as a merge request?
Edited by Wuttke, JoachimSince I'm working on the Debian packaging, I'd like to support all architectures supported by Debian :-) This means many hardware architectures (different endiannesses, different word sizes), but I think we use GCC by default (although Clang is also supported). GNU/Hurd and GNU/kFreeBSD are not released architectures so you can safely assume Linux for now. You can find the current status of the builds on https://buildd.debian.org/status/package.php?p=bornagain The patch does fix the build for some architectures, but others fail for other reasons (testsuite). This'll be for another issue.
As for the merge request… apparently I don't have the required permissions to create a fork of this repository, so I can't really do that. I'm attaching the output of "git format-patch" if it helps.
mentioned in commit 8b45fe5e
mentioned in merge request !2529 (merged)
mentioned in commit 8cfee812
closed with merge request !2529 (merged)
closed with commit 8b45fe5e
mentioned in commit 3fe5f970
mentioned in merge request !2531 (closed)
mentioned in commit 55fea5ab
mentioned in merge request !2543 (merged)
mentioned in commit dee8fe3d
mentioned in commit a7b763bd
