Skip to content

Fix qtapp and consapp compilation to recent options, QT and Mac compile#138

Closed
drf5n wants to merge 9 commits intortklibexplorer:demo5from
drf5n:dfq5
Closed

Fix qtapp and consapp compilation to recent options, QT and Mac compile#138
drf5n wants to merge 9 commits intortklibexplorer:demo5from
drf5n:dfq5

Conversation

@drf5n
Copy link

@drf5n drf5n commented Mar 1, 2023

This solves these issues for my non-windows Mac & Homebrew setup.

#135

#127

It also has a workaround for the interaction between RTK's lock() function and Mac's mutex lock() templates.

It also makes use of the pull in

#107

@pkkrusty
Copy link

pkkrusty commented Mar 22, 2023

Lots of open PRs on this fork and original rtklib. Recent and many months old. Are the original admins no longer actively involved in the projects? If not, anyone have recommendations for a fork that is more actively maintained?

*edit there are quite a few commits in the last 6 months, but not really any interaction with folks submitting PRs.

Repository owner deleted a comment from gaofenzi Mar 22, 2023
@rtklibexplorer
Copy link
Owner

It's true that I haven't been good about keeping up with the PR requests, especially the most recent ones. I will try and review these in the near future. Regarding the Qt builds, support for them was dropped in the original (2.4.3) RTKLIB and I have not been supporting them either although the original code is still present in the repo in case anyone else wants to resurrect it. I do actively maintain the Windows GUI and CUI apps and the linux CUI apps.

rtklibexplorer added a commit that referenced this pull request May 17, 2023
 - Fix qtapp compilation for new Qt versions (Qt 6.x)
 - Make rtknavi_qt and rtkpost_qt compile on mac
 - Add INHIBIT_RTK_LOCK_MACROS for lock-mutex workaround
 - Update the rtkget URL_LIST for CORS stations
@rtklibexplorer
Copy link
Owner

Merged most of these changes into the code with the 5/17/2023 commit. Removing the libraries from the gcc makefiles breaks my linux builds so I did not include these.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants