[Bug 187932] Review Request: paraview - Parallel visualization application

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: paraview - Parallel visualization application


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187932





------- Additional Comments From orion at cora.nwra.com  2006-04-10 11:49 EST -------
(In reply to comment #3)
> * it would be nice to have a .desktop file
 
Got an initial version.  Still needs a little more work.  Suggestions?

> rpmlint is also unhappy with the debuginfo package. There are lots of 'objdump
> failed', that I don't know how to solve, but there are also some errors, because
> lots of source files have the executable bit set. It could be possible to chmod
> -x everything ending in .h .c .cxx. Could be done later, however.

Added these to the existing chmod -x command.

> * I would have chosed BSD-like for the licence, but Distributable is ok too.

I thought this was better due to multpiple licenses.
 
> * I don't know how much the -data and other packages are coupled. But if they
> are the specific version release should be required, like
> Requires:       %{name}-data = %{version}-%{release}

Very good - done.

> NEEDSWORK: there are many man pages distributed in the paraview-mpi package, and
> some cmake files that I believe shouldn't be packaged, the man pages refer to
> non existant header files.

I suppose that they might be useful for something, but certainly not as
currently installed.  I'll remove. 

Again, just need the new spec:
http://www.cora.nwra.com/~orion/fedora/paraview.spec
http://www.cora.nwra.com/~orion/fedora/paraview-2.4.3-3.src.rpm

Still looking at the build issue...

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.