Skip to content

Conversation

@prj-
Copy link
Contributor

@prj- prj- commented Dec 14, 2021

Previous behavior:

$ otool -L petsc/arch-darwin-c-debug-real/lib/libparmmg.dylib
petsc/arch-darwin-c-debug-real/lib/libparmmg.dylib:
	petsc/arch-darwin-c-debug-real/lib/libparmmg.dylib (compatibility version 0.0.0, current version 0.0.0)
	petsc/arch-darwin-c-debug-real/lib/libmmg.5.dylib (compatibility version 5.0.0, current version 5.6.0)

New behavior:

$ otool -L petsc/arch-darwin-c-debug-real/lib/libparmmg.dylib
petsc/arch-darwin-c-debug-real/lib/libparmmg.dylib:
	petsc/arch-darwin-c-debug-real/lib/libparmmg.1.dylib (compatibility version 1.0.0, current version 1.4.0)
	petsc/arch-darwin-c-debug-real/lib/libmmg.5.dylib (compatibility version 5.0.0, current version 5.6.0)

This matches with the current code of Mmg.

@prj-
Copy link
Contributor Author

prj- commented Dec 15, 2021

@Algiane or @lcirrottola, could you please review/merge this? It's a very trivial change.

@Algiane Algiane merged commit 8da49e2 into MmgTools:develop Dec 16, 2021
@Algiane
Copy link
Member

Algiane commented Dec 16, 2021

Yes, of course (sorry, I missed the PR mail).
Thanks!

@prj-
Copy link
Contributor Author

prj- commented Dec 16, 2021

Thank you 😄

@prj- prj- deleted the jolivet/fix-missing-soversion branch February 3, 2022 06:17
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.

2 participants