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

rmlint: fix the build on older Darwin systems #573

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

barracuda156
Copy link

This PR fixes rmlint build for 10.5.8 and 10.6.
It contains two part, which come from:

  1. https://gist.github.com/ingramj/1105106/adfad1a7b1f0575dd7737c296d644856f2c2d074
  2. error: no member named 'st_atim' in 'struct stat' hboetes/mg#7 (comment)

P. S. There is also another problem related to how SCons build system works – trying to build a FAT binary on Leopard fails, for example, because arch flags get ignored. I solve it like this (for Macports): https://github.com/macports/macports-ports/pull/15003/files#diff-d43b1faa4b64984e308f2d41fddfd09fb559868da27208ff41417d636a4c7d20
I don’t know how to make a similar fix in the source code that gonna work without Macports assistance.

@kencu
Copy link

kencu commented Jun 1, 2022

instead of MAC_OS_X_VERSION_10_7 use 1070

@barracuda156
Copy link
Author

@sahib Could you please take a look at this PR?

@cebtenzzre cebtenzzre added the topic-compiling Related to building rmlint from source label Aug 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic-compiling Related to building rmlint from source
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants