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


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

--- Comment #2 from Ankur Sinha <sanjay.an...@gmail.com> 2011-06-28 13:02:26 
EDT ---
(In reply to comment #1)
> I think that caption doesn't belong into the package description:
> 
> TetGen is written in C++. It can be compiled into either a 
> standalone program invoked from command-line or a library for 
> linking with other programs. All major operating systems, 
> e.g. Unix/Linux, MacOS, Windows, etc, are supported.
> 
> There are no pictures:
> 
> The following pictures respectively illustrate a 3D polyhedral
> domain (left), a boundary conforming Delaunay tetrahedral mesh
> (middle), and its dual - a Voronoi partition (right).
> 
> No need to rm -rf $RPM_BUILD_ROOT.
> 
> It might be a good idea to include the documentation you can download from the
> homepage.
> 
> Your CFLAGS and CXXFLAGS are not worshiped:
> 
> g++ -O0 -c predicates.cxx
> g++ -g -Wall -DSELF_CHECK -o tetgen tetgen.cxx predicates.o -lm


Hi Volker, 

updated packages:

http://ankursinha.fedorapeople.org/tetgen/tetgen.spec

http://ankursinha.fedorapeople.org/tetgen/tetgen-1.4.3-1.fc15.src.rpm

I'll package the docs as a separate package? Or I can always include them
later. 

Thanks,
Ankur

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to