[GH-ISSUE #4] Loading native library in a cross-platform way. #3

New issue

Closed

opened 2026-05-05 10:58:32 -06:00 by gitea-mirror · 3 comments

gitea-mirror commented

2026-05-05 10:58:32 -06:00

Owner

Originally created by @pauldotknopf on GitHub (Jul 14, 2018).
Original GitHub issue: https://github.com/qmlnet/qmlnet/issues/4

LibGit2Sharp is currently doing it properly: e618156c15/LibGit2Sharp/Core/NativeMethods.cs

The problem is there is no single PInvoke dll name that will work across platform. So, either we have different package (Qt.NetCore.Mac, etc), or we manually search and load the libs our self.

Another option is to use a 3rd party for DLL loading: https://github.com/Firwood-Software/AdvanceDLSupport

AdvanceDLSupport looks interesting, but it would require us to drop SWIG, which I kinda like the idea of. It would give us more control without having to understand the cryptic SWIG definition language.

Using SWIG would make sense if we intended to generate wrappings for the entire Qt layer, which we don't intend on doing. This project is for QML integration only, .NET should be used for everything else.

Removing SWIG would take some time. We are almost at a good/usable point with this project though, I'd like to focus on getting it buttoned up.

Originally created by @pauldotknopf on GitHub (Jul 14, 2018). Original GitHub issue: https://github.com/qmlnet/qmlnet/issues/4 LibGit2Sharp is currently doing it properly: https://github.com/libgit2/libgit2sharp/blob/e618156c150d0bf2c318f5e6fa30b468f63ee45f/LibGit2Sharp/Core/NativeMethods.cs The problem is there is no single `PInvoke` dll name that will work across platform. So, either we have different package (`Qt.NetCore.Mac`, etc), or we manually search and load the libs our self. Another option is to use a 3rd party for DLL loading: https://github.com/Firwood-Software/AdvanceDLSupport `AdvanceDLSupport` looks interesting, but it would require us to drop SWIG, which I kinda like the idea of. It would give us more control without having to understand the cryptic SWIG definition language. Using SWIG would make sense if we intended to generate wrappings for the entire Qt layer, which we don't intend on doing. This project is for QML integration *only*, .NET should be used for everything else. Removing SWIG would take some time. We are almost at a good/usable point with this project though, I'd like to focus on getting it buttoned up.

gitea-mirror closed this issue

2026-05-05 10:58:33 -06:00

gitea-mirror commented

2026-05-05 10:58:35 -06:00

Author

Owner

@devmil commented on GitHub (Jul 14, 2018):

AdvancedDLSupport looks really promising!
I would propose to try to switching to the final Interop way first before finishing up the rest.
Especially as all the string special treatment might not be needed when we use AdvancedDLSupport.
I also think that the Qt datatypes are hidden in our case as much as possible. All the user code is inside QML or .NET.

@devmil commented on GitHub (Jul 14, 2018): AdvancedDLSupport looks really promising! I would propose to try to switching to the final Interop way first before finishing up the rest. Especially as all the string special treatment might not be needed when we use AdvancedDLSupport. I also think that the Qt datatypes are hidden in our case as much as possible. All the user code is inside QML or .NET.

gitea-mirror commented

2026-05-05 10:58:37 -06:00

Author

Owner

@pauldotknopf commented on GitHub (Jul 14, 2018):

After getting the unit test project to work, I'm starting to agree.

It will take a lot of time to get swig removed though :(

I'm going to start looking into it now.

@pauldotknopf commented on GitHub (Jul 14, 2018): After getting the unit test project to work, I'm starting to agree. It will take a lot of time to get swig removed though :( I'm going to start looking into it now.

gitea-mirror commented

2026-05-05 10:58:39 -06:00

Author

Owner

@pauldotknopf commented on GitHub (Jul 14, 2018):

Closing. Keep track with #6

@pauldotknopf commented on GitHub (Jul 14, 2018): Closing. Keep track with #6

gitea-mirror referenced this issue

2026-05-05 11:20:01 -06:00

[PR #5] [MERGED] Got travis build working. #163

gitea-mirror referenced this issue

2026-05-05 11:20:01 -06:00