Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding quad-precision build support #9

Merged
merged 11 commits into from
Aug 22, 2024
14 changes: 10 additions & 4 deletions recipe/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,11 @@ if [[ "$CONDA_BUILD_CROSS_COMPILATION" == 1 ]]; then

cmake \
-GNinja \
-DSLEEF_BUILD_TESTS=no \
-DSLEEF_BUILD_SHARED_LIBS=yes \
-DSLEEF_BUILD_TESTS=OFF \
-DSLEEF_BUILD_SHARED_LIBS=ON \
-DSLEEF_BUILD_SCALAR_LIB=ON \
-DSLEEF_BUILD_QUAD=ON \
-DSLEEF_BUILD_GNUABI_LIBS=ON \
-DCMAKE_INSTALL_PREFIX=$BUILD_PREFIX \
-DCMAKE_PREFIX_PATH=$BUILD_PREFIX \
-DCMAKE_INSTALL_LIBDIR=lib \
Expand All @@ -43,8 +46,11 @@ fi

cmake ${CMAKE_ARGS} \
-GNinja \
-DSLEEF_BUILD_TESTS=no \
-DSLEEF_BUILD_SHARED_LIBS=yes \
-DSLEEF_BUILD_TESTS=OFF \
-DSLEEF_BUILD_SHARED_LIBS=ON \
-DSLEEF_BUILD_SCALAR_LIB=ON \
-DSLEEF_BUILD_QUAD=ON \
-DSLEEF_BUILD_GNUABI_LIBS=ON \
-DCMAKE_INSTALL_PREFIX=$PREFIX \
-DCMAKE_PREFIX_PATH=$PREFIX \
-DCMAKE_INSTALL_LIBDIR=lib \
Expand Down
8 changes: 7 additions & 1 deletion recipe/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ source:
sha256: 441dcf98c0f22e5d5e553d007f3b93e89eb58e4c66e340da8af5e7f67d1dc24c

build:
number: 1
number: 2
run_exports:
# SO name changes at minor rev bumps.
- {{ pin_subpackage('sleef', max_pin='x') }}
Expand All @@ -20,17 +20,23 @@ build:
requirements:
build:
- {{ compiler('c') }}
- {{ compiler('cxx') }}
- {{ stdlib("c") }}
- cmake
- ninja # [unix]
- make # [unix]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SwayamInSync I was wondering why is make needed here, as the recipe on Unix uses ninja as generator?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for catching that, make sense, I would've just put that there for my local use, will push a refactor commit for this

- llvm-openmp # [osx]
- libgomp # [linux]
host:
- openssl
- llvm-openmp # [osx]

test:
commands:
- test -f $PREFIX/include/sleef.h # [unix]
- test -f $PREFIX/lib/libsleef${SHLIB_EXT} # [unix]
- test -f $PREFIX/lib/libsleefquad${SHLIB_EXT} # [linux]
- test -f $PREFIX/include/sleefquad.h # [linux]
- if not exist %PREFIX%\\Library\\include\\sleef.h exit 1 # [win]

about:
Expand Down
Loading