[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 Miroslav Suchý changed: What|Removed |Added Status|NEW |CLOSED

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 Miroslav Suchý changed: What|Removed |Added CC|

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 --- Comment #15 from Michael Schwendt (Fedora Packager Sponsors Group) bugs.mich...@gmx.net --- %files %{_docdir}/%{name}/ReadMe.md %license LICENSE %{_libdir}/*.so.* In this package, a directory entry for %{_docdir}/%{name} is missing.

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 --- Comment #14 from Arnaud Gelas arnaudge...@gmail.com --- Hi Michael, Thanks for the review! I have just uploaded new spec/srpm there: https://dl.dropboxusercontent.com/u/15135983/statismo.spec

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 --- Comment #12 from Michael Schwendt bugs.mich...@gmx.net --- Consider pointing the fedora-review at this ticket: fedora-review -b 1171624 It performs a lot of package review tests on the src.rpm and any built rpms and presents a list of

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 --- Comment #13 from Michael Schwendt bugs.mich...@gmx.net --- Antonia Hah! Antonio, of course! ;-) Been making awesome typos recently. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 --- Comment #11 from Arnaud Gelas arnaudge...@gmail.com --- Antonio, Thank you very much for your (really appreciated) guidance. I'll follow these guidelines and instructions. Here is the latest verison for reference:

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 --- Comment #9 from Arnaud Gelas arnaudge...@gmail.com --- It should be fixed in the latest version: https://dl.dropboxusercontent.com/u/15135983/statismo.spec https://dl.dropboxusercontent.com/u/15135983/statismo-0.10.1-3.fc20.src.rpm

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 --- Comment #10 from Antonio Trande anto.tra...@gmail.com --- **good** %files %doc ReadMe.md %license LICENSE %{_libdir}/*.so.* **change** %files doc %doc ReadMe.md %license LICENSE %{_docdir}/%{name}/ **change** %files devel

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 --- Comment #7 from Arnaud Gelas arnaudge...@gmail.com --- I have fixed these issues in the latest version https://dl.dropboxusercontent.com/u/15135983/statismo.spec https://dl.dropboxusercontent.com/u/15135983/statismo-0.10.1-2.fc20.src.rpm

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 Antonio Trande anto.tra...@gmail.com changed: What|Removed |Added CC|

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 --- Comment #5 from Arnaud Gelas arnaudge...@gmail.com --- - I have replaced 'basic' cmake command by provided %cmake macro - I have set most of variables returned by 'cmake -LAH' - Note there is no files coming from vtk nor itk in this

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 --- Comment #6 from Antonio Trande anto.tra...@gmail.com --- - gcc-c++ BR is not needed. See http://fedoraproject.org/wiki/Packaging:Guidelines#Exceptions_2 - List LICENSE file with %license macro - doc sub-package does not install the

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 Antonio Trande anto.tra...@gmail.com changed: What|Removed |Added Blocks||177841

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 --- Comment #2 from Antonio Trande anto.tra...@gmail.com --- RPM buildings fail in rawhide, there are some fixes to do. Make at least a scratch build test before to post a new release. See

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 --- Comment #3 from Arnaud Gelas arnaudge...@gmail.com --- Antonio, I have fixed errors on rawhide, see http://koji.fedoraproject.org/koji/taskinfo?taskID=8321945 Best -- You are receiving this mail because: You are on the CC list for

[Bug 1171624] Review Request: statismo - C++ Framework for building Statistical Image And Shape Models

https://bugzilla.redhat.com/show_bug.cgi?id=1171624 --- Comment #4 from Antonio Trande anto.tra...@gmail.com --- - RPM build system provides a %cmake macro; see the output of command 'rpm --eval %cmake' to expand it. - Use 'cmake -LAH' to list all the variables from the CMake including whose