Skip to content

🐛 Further installation fixes#540

Closed
burgholzer wants to merge 2 commits intomainfrom
further-installation-fixes
Closed

🐛 Further installation fixes#540
burgholzer wants to merge 2 commits intomainfrom
further-installation-fixes

Conversation

@burgholzer
Copy link
Copy Markdown
Member

@burgholzer burgholzer commented Jan 27, 2024

Description

The goal of this PR is to identify and fix why

are all having problems with the static libraries deployed with v2.2.2. of mqt-core.
I have the slight feeling that shared libraries might actually be necessary to get this working.

Currently known issues:

  • Due to some unknown reason, some of the symbols are undefined and cannot be found in the shared Python module library.
  • Due to some also unknown reason, the Windows wheels are installed in a Debug configuration which causes incompatibilities when trying to compile them together with Release build libraries.

Checklist:

  • The pull request only contains commits that are related to it.
  • I have added appropriate tests and documentation.
  • I have made sure that all CI jobs on GitHub pass.
  • The pull request introduces no new warnings and follows the project's style guidelines.

@burgholzer burgholzer added bug Something isn't working python Anything related to Python code packaging Anything related to Python packaging labels Jan 27, 2024
@burgholzer burgholzer added this to the MQT Core milestone Jan 27, 2024
@burgholzer
Copy link
Copy Markdown
Member Author

Closing this for now as it seems the preferred course of action is to go with shared libraries, so there is no need to keep this open.

@burgholzer burgholzer closed this Feb 5, 2024
@burgholzer burgholzer deleted the further-installation-fixes branch August 15, 2025 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working packaging Anything related to Python packaging python Anything related to Python code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant