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

Pixi Windows failing in CI #782

Closed
facontidavide opened this issue Mar 8, 2024 · 6 comments
Closed

Pixi Windows failing in CI #782

facontidavide opened this issue Mar 8, 2024 · 6 comments
Labels
bug Something isn't working

Comments

@facontidavide
Copy link
Collaborator

@tony-p , any suggestion?

I am not sure what the error is

@facontidavide facontidavide added the bug Something isn't working label Mar 8, 2024
@tony-p
Copy link
Contributor

tony-p commented Mar 8, 2024

From looking at the error message my best guess would be something has changed in the path or how to specify the temporary environment variable.

If I remember correctly this was needed on windows as the test executable and library were in different folders compared to the linux build.

When i get a chance I can try take a look if still stuck.

if you aren’t against it, it would probably be more robust to create a pixi.toml file in the repo so it can be used as pixi project and then CI wouldn’t have to try set it all up on the fly.

it would just live alongside the ros and conan files giving another way to interact with the project

@facontidavide
Copy link
Collaborator Author

go ahead with any modification that you consider appropriate

@tony-p
Copy link
Contributor

tony-p commented Mar 20, 2024

I have made a start here #791

This fixes the issue you encountered, but I see it now has linux build issues. I get these issues with my ROS builds and is to do with the sqlite and zeromq not being added to the rpath properly (Conda environments lean very heavily on the rpath of binaries).

Locally on windows I get Json warnings/errors when building and the json tests fail.

I can hopefully look into these very soon unless you know some direct solutions to add to the cmake. I think adding the dependency libraries as public already helps ....

@tony-p
Copy link
Contributor

tony-p commented Mar 21, 2024

Making the ${BTCPP_EXTRA_LIBRARIES} does solve the linking issue on linux. On the PR both windows builds fail due to ambiguous symbol errors for a number of the JSON conversions.

@tony-p
Copy link
Contributor

tony-p commented Mar 26, 2024

@facontidavide The PR above fixes all pixi build related issues. However both the windows and pixi windows builds fail for the new json conversion so have created a new issue #793

@tony-p tony-p mentioned this issue Mar 26, 2024
@facontidavide
Copy link
Collaborator Author

thanks a lot!!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants