Re: [Hamlib-developer] rig.h changes

[G0GJV <g0...@go...>]

Mike

winpthreads.h hasn't made it into the overnight zip.

When I add it manually, I find that it creates some serious clashes with Qt!

Why do you need to expose pthreads to the application builder? It looks like it is just 
for

pthread_mutex_t mutex_set_transaction;

Can't you put this somewhere else, and have a void * to point to it in rig.h? This would 
remove the need for a large #include, and would compile easily everywhere.

Mike G0GJV

> Thanks  Mike - I'll give it a try
> 
> Mike
> 
> > Try the 20220827 download when it's available 
tomorrow.http://n0nb.users.sourceforge.net
> > 
> > You'll need to add the new winpthreads.h file to your project files.
> > Mike W9MDB
> > 
> >  
> > 
> >     On Wednesday, August 24, 2022 at 03:47:06 PM CDT, G0GJV
> <g0...@go...>
> > wrote:  
> >  
> >  I'll try direct
> > 
> > Mike G0GJV
> > 
> > > Can you send me your MSVC project?
> > > Mike W9MDB
> > > 
> > >Â  
> > > 
> > >Â  Â  On Wednesday, August 24, 2022 at 05:26:28 AM CDT, G0GJV
> > <g0...@go...>
> > > wrote:Â  
> > >Â  
> > >Â  Mike
> > > 
> > > This doesn't work when compiling a program that uses hamlib with MSVC.
> > > 
> > > MSVC DOES NOT have pthread.h
> > > 
> > > Mike G0GJV
> > > 
> > > > Branch: refs/heads/master
> > > >  Home:  https://github.com/Hamlib/Hamlib
> > > >  Commit: 6f967a68d5a6c0fda3e0d4092dde54eb6ffd4c32
> > > >      
https://github.com/Hamlib/Hamlib/commit/6f967a68d5a6c0fda3e0d4092dde54eb6ffd4c32
> > > >  Author: Mike Black W9MDB <mdb...@ya...>
> > > >  Date:  2022-08-23 (Tue, 23 Aug 2022)
> > > > 
> > > >  Changed paths:
> > > >    M include/hamlib/rig.h
> > > >    M security/security.c
> > > >    M src/misc.h
> > > > 
> > > >  Log Message:
> > > >  -----------
> > > >  Make pthread.h a default include and remove
> > config.h from being included in rig.h
> > > > https://github.com/Hamlib/Hamlib/milestone/14