Skip to content
This repository was archived by the owner on Feb 24, 2024. It is now read-only.

Chained including not supported #6

Open
RipperJ opened this issue Mar 23, 2022 · 2 comments
Open

Chained including not supported #6

RipperJ opened this issue Mar 23, 2022 · 2 comments
Assignees
@RipperJ
Labels
bug Something isn't working

Comments

@RipperJ
Copy link

RipperJ commented Mar 23, 2022

If top.cpp includes "a.hpp", and "a.hpp" include another "b.hpp" from another folder, it seems that b.hpp will not be found even if we set that path to it in "--include [b.hpp's path]".
@FastAlien FastAlien self-assigned this Mar 29, 2022
@FastAlien FastAlien added the bug Something isn't working label Mar 29, 2022
FastAlien added a commit that referenced this issue Mar 29, 2022
@FastAlien
Remove redundant empty lines if the parsed file doesn't have system i…
2185961 
…ncludes

Add unit test for scenario from issue #6.
@FastAlien
Copy link
Owner

Hello @RipperJ. Thank you for reporting the issue. Unfortunately, I couldn't reproduce the problem. I've added a new unit test for your scenario and I've found a minor bug which I've fixed. Could you please verify if files located in test/data/headerIncludeHeaderInSubdirectory (https://github.com/FastAlien/cpp-merge/tree/master/test/data/headerIncludeHeaderInSubdirectory) are matching your scenario? Changes have been merged into the master branch.

@FastAlien FastAlien assigned RipperJ and unassigned FastAlien Mar 29, 2022
@pedroflor
Copy link

Same problem here...
I'm trying to use multiple include paths with no success.

Unfortunately I can't show output of my "include" paths because of NDA.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@RipperJ RipperJ

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@FastAlien @pedroflor @RipperJ