Skip to content

Check for missing header files#232

Open
Pan7 wants to merge 2 commits into
pooler:masterfrom
Pan7:includes
Open

Check for missing header files#232
Pan7 wants to merge 2 commits into
pooler:masterfrom
Pan7:includes

Conversation

@Pan7

@Pan7 Pan7 commented May 26, 2021

Copy link
Copy Markdown

More support for non Unix-like systems.

@pooler

pooler commented May 26, 2021

Copy link
Copy Markdown
Owner

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

@Pan7

Pan7 commented May 26, 2021

Copy link
Copy Markdown
Author

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

@pooler

pooler commented May 26, 2021

Copy link
Copy Markdown
Owner

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

@Pan7

Pan7 commented May 26, 2021

Copy link
Copy Markdown
Author

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

pooler commented May 26, 2021

Copy link
Copy Markdown
Owner

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