Skip to content

Conversation

@Algiane
Copy link
Member

@Algiane Algiane commented Dec 11, 2022

Find Mmg private headers

Update ParMmg to find Mmg headers after commit f3c20cb1b8e638208880efc9 of Mmg.
Linked to:

It also include some of the propositions of improvement of @prudhomm to use ParMmg as submodule (not all of them for now).

This commit also includes some of the proposition of @prudhomme to use Mmg and ParMmg as submodules of a super project.
Start also to fix issue when VTK is found but vtkMPIController.h header is not (doesn't work for now).
@Algiane Algiane added part: compilation compilation related kind: enhancement enhancement to an existing feature part: cmake cmake part priority: high linked to blocking behaviour or to a close deadline labels Dec 11, 2022
@Algiane Algiane self-assigned this Dec 11, 2022
#define _LIBPARMMGTYPES_H

#include "mmg/mmg3d/libmmgtypes.h"
#include "libmmgtypes.h"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
#include "libmmgtypes.h"
#include "mmg/common/libmmgtypes.h"

That's the single diff I have to apply to make everything work, last I checked.

@Algiane
Copy link
Member Author

Algiane commented Dec 12, 2022

Thanks a lot for your responsiveness and for your help. As the last issue is in ParMmg and not Mmg I will merge the current PR and solve the issue of header inclusion in ParMmg later (that I still don't understand because mmg/common is added to ParMmg include directories).

Best Regards,

@prj-
Copy link
Contributor

prj- commented Dec 12, 2022

No problem, we can iterate once the Mmg branch is merged. Worst case scenario, you can't wrap things up on the ParMmg side, then I'll just need to keep the ParMmg fork. At least I'll be able to redirect the trafic on the official Mmg repository.

@Algiane
Copy link
Member Author

Algiane commented Dec 12, 2022

PR will be merged to follow the merge of the PR MmgTools/mmg#187 of Mmg but we may still have some header inclusions to fix in ParMmg.

@Algiane Algiane closed this Dec 12, 2022
@Algiane Algiane reopened this Dec 12, 2022
@Algiane Algiane merged commit efc4eb5 into develop Dec 12, 2022
@Algiane Algiane deleted the feature/Find-Mmg-private-headers branch January 26, 2023 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind: enhancement enhancement to an existing feature part: cmake cmake part part: compilation compilation related priority: high linked to blocking behaviour or to a close deadline

Projects

Development

Successfully merging this pull request may close these issues.

3 participants