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

Fix build #4

Merged
merged 34 commits into from
Jun 24, 2024
Merged

Fix build #4

merged 34 commits into from
Jun 24, 2024

Conversation

sgouache
Copy link
Collaborator

No description provided.

cedric-le-cam and others added 30 commits June 7, 2024 12:44
Must be considered a work in progress.

Added:
 - a helper class to handle the plugins that have a common interface
 - a test skeleton to check that the handle works

The build lacks automation for now. The required dlls to let the test work need to be copied where the test executable resides.
At the current state, the goal is to find an appropriate way to pass the path to the plugin dll to the test executable.
Update: replace dynamic library run-time loading with loading at executable's launch time for the test.
Some options are left aside for now.
Support for multifile should be automatically handled by the glob options of ListObjects.
Stephane Gouache and others added 4 commits June 19, 2024 12:00
* Add plugin load test

* Fix windows compilation

* Fix compilation on windows

* Add plugin load test

* Delay test discovery until test execution

* Fix windows

* Try debugging lib loading

* Fix typo

* Try loading dll with relative path

* Try debug library loading

* Fix typo

* Restore test discovery at build time

* Cleanup

* Fix typo

---------

Co-authored-by: Stephane Gouache <[email protected]>
@sgouache sgouache merged commit 85e2f02 into main Jun 24, 2024
16 checks passed
@sgouache sgouache deleted the fix_build branch June 24, 2024 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants