Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 233191 - sci-vizualization/paraview-3.3_pre20080514: add >=OpenFOAM-1.5 support
Summary: sci-vizualization/paraview-3.3_pre20080514: add >=OpenFOAM-1.5 support
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: Markus Dittrich (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-07-28 20:30 UTC by Oliver Borm
Modified: 2008-07-30 21:52 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch for the OpenFOAM-1.5 file support (paraview-3.3_pre20080514-OpenFOAM.patch,353.36 KB, patch)
2008-07-28 20:31 UTC, Oliver Borm
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Oliver Borm 2008-07-28 20:30:47 UTC
The OpenFOAM reader which is actually in the cvs sources from paraview cannot read OpenFAOM cases from the new OpenFOAM-1.5 version. In order to have support for >=OpenFOAM-1.5 in paraview, the provided patch is necessary. This patch was posted on the OpenFOAM mailing list:

http://openfoam.cfd-online.com/cgi-bin/forum/board-auth.cgi?file=/1/4704.html#POST25140

I've tested this patch with paraview-3.3_pre20080514. But I think it should be also working with paraview-3.2.1. 

If this patch is not applied, paraview-3.3_pre20080514 just freezes and paraview-3.2 gives an error message with OpenFOAM-1.5 test cases.

Reproducible: Always
Comment 1 Oliver Borm 2008-07-28 20:31:44 UTC
Created attachment 161581 [details, diff]
Patch for the OpenFOAM-1.5 file support
Comment 2 Wormo (RETIRED) gentoo-dev 2008-07-29 00:16:01 UTC
Thanks for the patch; assigning to maintainer
Comment 3 Markus Dittrich (RETIRED) gentoo-dev 2008-07-29 12:41:37 UTC
Hi Oliver,

Thank you very much for the patch and I'll try to 
have a look at it soon.

Best,
Markus
Comment 4 Markus Dittrich (RETIRED) gentoo-dev 2008-07-29 15:10:14 UTC
Hi Oliver,

Does the author of the patch release it under the same license
as paraview itself? I didn't see any mention of it in the
release notes.

Thanks,
Markus
Comment 5 Oliver Borm 2008-07-29 15:36:27 UTC
Hello Markus,

I don't know exactly under which license he has published his code, but as this reader is derived from the one which is actually in paraview and he stated that he want to include his reader into the official paraview cvs, I assume that he is publishing his code under the paraview license. 

For an exact clarification you may want to contact him.

Best regards,
Oliver
Comment 6 Markus Dittrich (RETIRED) gentoo-dev 2008-07-29 15:52:07 UTC
(In reply to comment #5)
> Hello Markus,
> 
> I don't know exactly under which license he has published his code, but as this
> reader is derived from the one which is actually in paraview and he stated that
> he want to include his reader into the official paraview cvs, I assume that he
> is publishing his code under the paraview license. 
> 
> For an exact clarification you may want to contact him.
> 
> Best regards,
> Oliver
> 

I'll send him an email just to make sure. Once this is settled I'll  
commit the patch to portage.

Thanks,
Markus
Comment 7 Markus Dittrich (RETIRED) gentoo-dev 2008-07-30 21:52:32 UTC
I've just added the patch to paraview-3.3 in portage.
You'll have to re-sync in an hour or so and recompile
paraview to pull in the addition.

Many thanks go to the original patch creator Takuya OSHIMA 
who kindly provided a patch compatible with the original 
paraview license.

Best,
Markus