-
Notifications
You must be signed in to change notification settings - Fork 10
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
Various fixes and improvements #54
base: master
Are you sure you want to change the base?
Conversation
Debug version crashes here, release version optimizes away the check.
@JohanMabille or @davidbrochart, can you comment on CI failure?
I have different test failures in a local build:
I am stuck on pretty basic stuff, so help is greatly appreciated. |
e0cb3ea
to
f0f1e9f
Compare
This seems to be a waste of time. |
@@ -12,6 +12,9 @@ jobs: | |||
- bash: echo "##vso[task.prependpath]$CONDA/bin" | |||
displayName: Add conda to PATH | |||
|
|||
- bash: sudo apt-get install libxerces-c | |||
displayName: Add conda to PATH |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi,
Sorry for the late reply. Dependencies should be installed with mamba or added in the environment-dev.yml
so be sure they're built with a compatible runtime., instead of relying on the build machine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried that too, but am not familiar with conda. I can try again.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It turns out my attempt does nothing.
1196935
to
0400b64
Compare
The error was: CMake Error at C:/Libs/xtl-vs19/xtlConfig.cmake:26 (message): File or directory C://include referenced by variable xtensor_zarr_INCLUDE_DIRS does not exist ! Call Stack (most recent call first): C:/Libs/xtensor-zarr-vs19/xtensor-zarrConfig.cmake:49 (set_and_check) CMakeLists.txt:6 (find_package)
So who has write access to this repository to merge these pull requests ? |
The error was: