Skip to content

Conversation

@MKlimenko
Copy link

@MKlimenko MKlimenko commented Apr 1, 2018

Existing msc solutions are extremly outdated, updated them.

By the way, why does the ERB protocol (erb.c) is located in src folder, not the src/rcv?

Why does the ERB protocol (erb.c) is located in src folder, not ther src/rcv?
@justprosh
Copy link

justprosh commented Apr 5, 2018

https://github.com/MKlimenko/RTKLIB/blob/8d6ff6244add54eb7970c9521a9b8fda99907c0a/src/rtkcmn.c#L133
Please replace ifndef WIN32 with ifdef WIN32, it is broke the build

@MKlimenko
Copy link
Author

@justprosh, that's a problem, because there is no such thing as sys/*** headers in MSVC compiler

@MKlimenko
Copy link
Author

@justprosh what about converting it to the #ifndef _MSC_VER?

@sjfergus
Copy link

My apologies in advance if this is the wrong location to ask (newbie here), but I'm very interested in whether you were ever able to get the console projects to ever successfully compile for Windows. Most particularly, I'm trying to get convbin created under version b27 (too many issues still with b28+). Thanks in advance for any feedback.

@MKlimenko
Copy link
Author

@sjfergus MSVC-builds are a bit buggy, so I've switched to BCC with clang. Also, I've made a PowerShell build-script, we're planning on releasing our RTKLib fork in a month or so.

If it's urgent, feel free to contact me via e-mail.

@sjfergus
Copy link

That's definitely encouraging. I'm definitely interested in the position you efforts. For what it is worth, my friends and I have found Emlid's version b27 based on RTKLIB 2.4.3 to be the last version with reliable/accurate results. B28+ seems to have a number of outstanding issues currently.

Unfortunately, I am not well versed in C/C++ to the same degree as I am in C#, or I'd offer to help where I could. I appreciate what you can share when it is ready to be shared. Thank you for the reply.

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.

3 participants