Bugzilla Bug 202685

Since the update to version 3.2.1 of Paraview I'm not able to use the native
OpenFOAM Readers in Paraview. So please add the patches to the ebuild.
Thank's!

Reproducible: Always

Created an attachment (id=138810) [details]
OpenFOAM Reader patch

To activate the build in capabilities of Paraview for OpenFOAM Files, this
patch is needed at compile time. Maybe this can be activated with an USE Flag,
so that my ebuild from bug 104257 has an postprocessing tool. The built-in
capabilities of Paraview were selected as default in version 2.6.2.

Created an attachment (id=138812) [details]
Patch for the OpenFOAM Reader

This patch improves the built-in capabilities from Paraview for the OpenFOAM
Datastructure very much. I would highly recommend to add this patch to the
ebuild, if the openfoam support is compiled with.

Hi Oliver,

Thanks much for your patches and I'll try to have a look at them 
as soon as I find some time.

Best,
Markus

Markus,

In the meantime are you aware that Oliver put an ebuild for OpenFoam on the
sunrise overlay? I used the ebuild and it built the beast (and the beast seems
to be alive and kicking...)... Maybe time to try to include OpenFoam to the
main gentoo tree...

Daniel

Hi Daniel,

I am aware of it, but haven't had time to even look at it yet :(
As soon as I find some time ..... I'll have a closer look.

cheers,
Markus

Hi Oliver,

I just had some time to look at your patches. 
Since they are quite extensive, would you be
able to comment on where they come from.
It almost seems to me that this is something
that should be reviewed and merged by upstream
rather than by us.

Best,
Markus

Hello Markus,

this patches came from the OpenFOAM Mailing list:
http://openfoam.cfd-online.com/cgi-bin/forum/show.cgi?tpc=1&post=17466#POST17466

and since the post from yesterday, it seems that these patches were sent
upstream but are not yet in the CVS. I fully agree with you, that these patches
should be merged into the official tree. Nevertheless, I would highly recommend
to use these patches in version 3.2.1 of the paraview ebuild and I hope we
don't need these patches in further versions of paraview.

Oliver

Hi Oliver,

Thanks much for the link! This indeed looks like a solid
patch hence it should be fine as part of our paraview ebuild
until upstream merges it into their trunk.

I've added it to the pareview-3.2.1 ebuild and it should
have hit the mirrors by now. You'll need to re-sync and 
re-emerge paraview to pull it in.

Lacking any OpenFOAM dataset I have not yet tested it
but the option is there. Please give it a spin and let me know
if it works as expected.

Thanks,
Markus

Markus,

Are you sure you put the patch in the gentoo tree?
When I emerged, I got the new ebuild but not the patch...

Daniel

Daniel,

The patch is on the mirrors not in the files/ directory
due to its size (patches in files/ should be < 30k 
uncompressed). Just look for the line

epatch "${DISTDIR}"/${P}-OpenFOAM.patch.bz2

in the ebuild.

best,
Markus

Hello Markus,

I have had a quick look on the new paraview and it seems to work as expected.
So I think we can close this bug.

Oliver

BTW: Why don't you call the new ebuild not paraview-3.2.1-r1 or something like
that?

The present version works just fine for everybody
not caring about OpenFOAM. Hence, there is no reason
to force a multi-hour compile onto people for a feature
they most likely don't care about. An -r1 in this case
would be a real bug fix/enhancement that matters for
a significant fraction of people. If paraview would
compile in 5 minutes this would have been a
different story.

Best,
Markus