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

Improved support for the CLion IDE #2068

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

Conversation

hanzi
Copy link

@hanzi hanzi commented Nov 20, 2024

Description

When loading the project in the JetBrains CLion, the IDE would not properly resolve includes from the include/ directory.

That's because apparently, CLion does not understand the -iquote include flag when parsing the Makefile, but it does unterstand -iquoteinclude without the space.

I realise that that's probably more of an issue with CLion itself as the syntax with space seems to be valid. But for better or worse, at least this fixed my immediate issue of not being able to properly use the project in CLion.

Because I'm not entirely sure if that syntax will work with other tools as well, I've put it in a clunky ifdef block inside the Makefile.

I have also added CLion's compiler flag to the dummy defines in include/global.h.

Before
image

After
image

Discord contact info

untitled_tino

When loading the project in the JetBrains CLion, the IDE would not properly resolve includes from the `include/` directory.

That's because apparently, CLion does not understand the `-iquote include` flag when parsing the `Makefile`, but it _does_ understand `-iquoteinclude` without the space.

I realise that that's probably more of an issue with CLion itself as the syntax with space seems to be valid. But for better or worse, at least this fixed my immediate issue of not being able to properly use the project in CLion.

I have also added CLion's compiler flag to the dummy defines in `include/global.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.

1 participant