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

Check for missing header files #232

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Check for missing header files #232

wants to merge 2 commits into from

Conversation

Pan7
Copy link

@Pan7 Pan7 commented May 26, 2021

More support for non Unix-like systems.

@pooler
Copy link
Owner

pooler commented May 26, 2021

What systems (requiring the patch) have you tested this on?

@Pan7
Copy link
Author

Pan7 commented May 26, 2021

Trying to compile cpuminer with msvc on a 32 bit machine.

@pooler
Copy link
Owner

pooler commented May 26, 2021

Can you build cpuminer on that system with just this patch? If sys/time.h isn't available, where is gettimeofday() declared?

@Pan7
Copy link
Author

Pan7 commented May 26, 2021

Didn't find a good solution for gettimeofday yet aside from using something like f4exb/sdrangel@f61d1c3#diff-8fb557527cc23eb19b0d8adcfb3294dfe3735cb89f7bc04a2d9ae865301c14fe
There is also a couple of small things missing like:
#ifdef _MSC_VER
#define strncasecmp _strnicmp
#endif
Not sure if this is the best sullution or where to add it.

@pooler
Copy link
Owner

pooler commented May 26, 2021

Compatibility macros should probably go in compat.h.

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.

2 participants