[PR #173] [MERGED] Expanded multiarch support. #432

New issue

Closed

opened 2026-05-05 12:15:43 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 12:15:43 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jmcnamara/libxlsxwriter/pull/173
Author: @Alexhuszagh
Created: 4/27/2018
Status: ✅ Merged
Merged: 4/27/2018
Merged by: @jmcnamara

Base: master ← Head: master

📝 Commits (1)

055516e Bug fix for #166, adding multiarch support and build toolchains for i686 on x86_64 systems.

📊 Changes

2 files changed (+29 additions, -6 deletions)

View changed files

📝 CMakeLists.txt (+22 -6)
➕ cmake/i686-toolchain.cmake (+7 -0)

📄 Description

On Linux systems this:

Includes an i686 generator to allow building 32-bit libraries on x86_64.
Updates the CMake documentation to reflect these changes.

For example, to build a 32-bit library on a x86_64 Linux, you may do:

mkdir build && cd build
cmake .. -DBUILD_TESTS=ON -DCMAKE_BUILD_TYPE=Release \
    -DCMAKE_TOOLCHAIN_FILE=../cmake/i686-toolchain.cmake
make
make test
make install

It would technically be possible to use an option to pass to CMake, however, this would not work if libxlsxwriter is used as a dependency in another project, while a toolchain with the proper flags for libxlsxwriter (CMAKE_C_FLAGS) and potentially for other projects that use libxlsxwriter (CMAKE_CXX_FLAGS and CMAKE_ASM_FLAGS) would prevent any discontinuities between the binary architecture of the libxlsxwriter target and that of the parent project.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/jmcnamara/libxlsxwriter/pull/173 **Author:** [@Alexhuszagh](https://github.com/Alexhuszagh) **Created:** 4/27/2018 **Status:** ✅ Merged **Merged:** 4/27/2018 **Merged by:** [@jmcnamara](https://github.com/jmcnamara) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`055516e`](https://github.com/jmcnamara/libxlsxwriter/commit/055516e8010391c57acf2e6c51fad550361a8aa8) Bug fix for #166, adding multiarch support and build toolchains for i686 on x86_64 systems. ### 📊 Changes **2 files changed** (+29 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `CMakeLists.txt` (+22 -6) ➕ `cmake/i686-toolchain.cmake` (+7 -0) </details> ### 📄 Description On Linux systems this: - Includes an i686 generator to allow building 32-bit libraries on x86_64. - Updates the CMake documentation to reflect these changes. For example, to build a 32-bit library on a x86_64 Linux, you may do: ```bash mkdir build && cd build cmake .. -DBUILD_TESTS=ON -DCMAKE_BUILD_TYPE=Release \ -DCMAKE_TOOLCHAIN_FILE=../cmake/i686-toolchain.cmake make make test make install ``` It would technically be possible to use an option to pass to CMake, however, this would not work if libxlsxwriter is used as a dependency in another project, while a toolchain with the proper flags for libxlsxwriter (CMAKE_C_FLAGS) and potentially for other projects that use libxlsxwriter (CMAKE_CXX_FLAGS and CMAKE_ASM_FLAGS) would prevent any discontinuities between the binary architecture of the libxlsxwriter target and that of the parent project. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>