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

Updating Hamlib to v4+ #867

Merged
merged 1 commit into from
Aug 2, 2021
Merged

Updating Hamlib to v4+ #867

merged 1 commit into from
Aug 2, 2021

Conversation

vsonnier
Copy link
Collaborator

@vsonnier vsonnier commented Jan 3, 2021

Hamlib 4+ has been relreased : https://github.com/Hamlib/Hamlib/releases/tag/4.1

This is the update of Hamlib to v4+, in particular Windows binaries.

So far I modified CMakeLists.txt to accomodate the lib name change (-4 instead of -2) and added the Windows binaries, and one constant in a Cubic .cpp

So far it compiles (with warnings) both for Win32 and Win64 using VS2019 v16.5

Edit :

  • 4.0 : runs without apparent issues, but what should I know ? I don't have Rigs to test with, so I opened this PR :)
  • 4.1 : print 2021-03-03:18:22:20.712339: rig_mW2power called then segfault. :(
  • 4.2 : same as 4.0 :)

I'm using in both cases the Win Dlls from the Hamlib releases .

Vincent

@vsonnier vsonnier mentioned this pull request Jan 3, 2021
@vsonnier vsonnier requested a review from cjcliffe January 3, 2021 11:51
@vsonnier vsonnier force-pushed the vso_update_to_hamlib_v40 branch from 150d4fd to d2dedbb Compare March 3, 2021 06:22
@vsonnier vsonnier changed the title Updated Hamlib to v4.0 Updated Hamlib to v4.1 Mar 3, 2021
@vsonnier
Copy link
Collaborator Author

vsonnier commented Mar 3, 2021

Updated for v4.1, changed title and such.

@vsonnier vsonnier force-pushed the vso_update_to_hamlib_v40 branch from d2dedbb to 50d7a36 Compare March 3, 2021 06:25
@vsonnier vsonnier changed the title Updated Hamlib to v4.1 Updated Hamlib to v4+ Mar 3, 2021
@vsonnier vsonnier changed the title Updated Hamlib to v4+ Updating Hamlib to v4+ Mar 3, 2021
@vsonnier vsonnier mentioned this pull request Apr 23, 2021
@vsonnier vsonnier force-pushed the vso_update_to_hamlib_v40 branch from 50d7a36 to 619a16f Compare May 17, 2021 18:56
@vsonnier
Copy link
Collaborator Author

Hamlib v4.2 has just been released: it runs under Win32 / Win64 without crashing this time.

@@ -114,7 +114,7 @@ void RigThread::run() {
std::cout << "Rig thread starting." << std::endl;

rig = rig_init(rigModel);
strncpy(rig->state.rigport.pathname, rigFile.c_str(), FILPATHLEN - 1);
strncpy(rig->state.rigport.pathname, rigFile.c_str(), HAMLIB_FILPATHLEN - 1);
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only change I made on CubicSDR source is here:

@vsonnier vsonnier force-pushed the vso_update_to_hamlib_v40 branch from 619a16f to b2b178e Compare August 2, 2021 12:28
@vsonnier vsonnier merged commit 3a08df7 into master Aug 2, 2021
@vsonnier vsonnier deleted the vso_update_to_hamlib_v40 branch November 21, 2022 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants