-
Notifications
You must be signed in to change notification settings - Fork 244
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
[Releases][MedApplication] Adding Med Application support for windows. #12049
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot @roigcarlo. I hope @philbucher is also ok with this :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job 👏🏻
@@ -362,6 +362,9 @@ class MedModelPartIO::MedFileHandler | |||
// read only by default, unless other settings are specified | |||
med_access_mode open_mode; | |||
|
|||
// Fix to allow windows conversion from whatever eldritch format is using to something convertible to c_str() | |||
std::string med_file_mame{rFileName.string()}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mame? 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a f**k, now I see it now. I`ll Fix it later :P
@@ -371,7 +374,9 @@ class MedModelPartIO::MedFileHandler | |||
// basic checks if the file is compatible with the MED library | |||
med_bool hdf_ok; | |||
med_bool med_ok; | |||
const med_err err = MEDfileCompatibility(rFileName.c_str(), &hdf_ok, &med_ok); | |||
|
|||
const med_err err = MEDfileCompatibility(med_file_mame.c_str(), &hdf_ok, &med_ok); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe rFileName.string().c_str()
would also work 🤷♂️
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not it does not :D
📝 Description
This allows the MedApplication to be compiled in windows. Also added the necessary files to be able to release it as a pip package.
Ping @Marco1410 because you use salome
Also ping @philbucher @sunethwarna you can find how to compile the med in win in the dockerfile.
🆕 Changelog
Med compilable in win