mrklein / openfoam-os-x

Patches for OpenFOAM(R) to build it on OS X
93 stars 33 forks source link

Request: A new patch for swak4Foam (develop branch) #78

Closed abzrg closed 2 years ago

abzrg commented 2 years ago

Current swk4Foam patch seems to work only on older versions of swak4Foam that only are compatible with earlier versions of OpenFOAM. I think we need a newer patch for the development branch of swak4Foam.

Installing with the following commands

hg clone http://hg.code.sf.net/p/openfoam-extend/swak4Foam swak4Foam; cd $_
hg update develop
./AllwmakeAll > log.make 2>&1

I get this log. Although installation finished with errors, the exit code was zero and I am able to run for example the command funkySetFields but I get the following linking error:

dyld[64117]: symbol not found in flat namespace '__ZN4Foam10DLListBase13endConstIter_E'
[1]    64117 abort      funkySetFields
$ /opt/homebrew/opt/llvm@13/bin/llvm-cxxfilt '__ZN4Foam10DLListBase13endConstIter_E'
Foam::DLListBase::endConstIter_

If I can help on anything, please let me know.

mrklein commented 2 years ago

Hi,

I see two problems here: swak4Foam and OpenFOAM 8. I do not use any of them.

I can try to update swak4Foam patch, yet, I won't be able neither to test it, nor to use it, and whole work will be for the sake of updating swak4Foam patch.

And I do not know if I want to deal with all this version checks (for example, one needs to check for curl if there is no wget; bear flags in version 3 have been changed, so one needs to check version and update command prefix correspondingly; in OpenFOAM 9 TableFile.H was removed, so there is a need to update CommonValueExpressionDriver.H; and all this from quick look at initial compilation log) and conditional compilation. Also soon patch will be broken any way, since it is for develop branch, and all the work becomes useless.

abzrg commented 2 years ago

Hi,

Yes, you are exactly right. I'll try to study your patch, and I'll see what I can do. If I succeed in that, I will inform you. So, I'll close this issue.