GeezerVison

Thanks for your contribution.

• An issue would be nice.
• And some test.
• We haven't switched to C++11 yet so that this simplecpp library will be as portable as possible. However I think we can start supporting C++11. I think C++11 should be widely supported by now. :-)

It might be a few weeks before I circle back to push changes upstream.

It might be a few weeks before I circle back to push changes upstream.

ok thanks understood.. for your information we have a PR to start using C++11 so if you get back in few weeks I will hopefully not complain about you using C++11 at least.

@danmar I have updated this PR, but I am struggling to figure out how to make a test to exercise this code. I tried to decipher how other tests were working, but could not identify how the "cache" would need to be setup to test the new functionality. Any guidance would be greatly appreciated.

Thanks,
Hans

@glankk Thank you for the C++11 updates! This is great work!

This PR is adjacent to some code that you recently refactored. It would be great to have your input.

Thanks,
Hans

I suggest we emulate gcc's darwin-specific -F option, and possibly -iframework as well. See https://gcc.gnu.org/onlinedocs/gcc/Darwin-Options.html. The -F paths are interleaved with -I and searched left-to-right, so the order of interleaved normal include paths and frameworks should be preserved. For this we could change the DUI::includePaths from being a vector of path strings to being a vector of structs with a path string and, on darwin, a framework flag that determines what search rules to use.

I will go on vacation for a week. Please ping me later if you want review..

@glankk This is likely a great suggestion, but it is increasing the scope of effort beyond what I can contribute in the short term.

Do you have the bandwidth to make the changes that you recommend, if so, please take whatever is useful from my suggestion and incorporate it.

The behavior should not be limited to builds made on Apple systems. It should be platform-agnostic and controlled by an explicit DUI option or the presence of __APPLE__ in the provided preprocessor macros (or both?).

The tests would probably be best implemented in Python.

And shouldn't this be done for system headers only? I doubt you want to look up random local includes.

This pull request should be closed as it is superseded by:

• feat: Ordered search paths for -I, -isystem, -F, -iframework + Darwin framework support (with legacy -I back-compat) #511

For additional context, @hjmjohnson and I have been working together on this.

Comprehensive solution addressed in #511.