[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Taylor Braun-Jones changed: What|Removed |Added Flags|needinfo?(taylor@braun-jone |needinfo- needinfo-

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner changed: What|Removed |Added Status|POST|CLOSED

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner changed: What|Removed |Added Flags|

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #27 from Taylor Braun-Jones --- Ankur, I'd be willing to co-maintain the package with you. But it appears that the Fedora Policy requires that a new ticket be opened at this point (if I understand things

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner changed: What|Removed |Added Status|CLOSED |POST

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner changed: What|Removed |Added Status|ON_QA |CLOSED

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner changed: What|Removed |Added Assignee|projects...@smart.ms

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner changed: What|Removed |Added Status|CLOSED |ON_QA

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner changed: What|Removed |Added Flags|fedora-review+ | -- You are

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner changed: What|Removed |Added Whiteboard||NotReady --

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Taylor Braun-Jones changed: What|Removed |Added Flags|needinfo?(taylor@braun-jone |

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner changed: What|Removed |Added Status|POST|CLOSED

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner projects...@smart.ms changed: What|Removed |Added CC|

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #21 from Raphael Groner projects...@smart.ms --- No builds found so far at koji neither bodhi. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner projects...@smart.ms changed: What|Removed |Added Status|ASSIGNED|POST -- You

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #19 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Taylor Braun-Jones tay...@braun-jones.org changed: What|Removed |Added Flags||fedora-cvs?

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #17 from Taylor Braun-Jones tay...@braun-jones.org --- (In reply to Raphael Groner from comment #16) Still some open issues I would like to see fixed in the final package: - ASL 2.0 is meant for vera.ctest but not for

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner projects...@smart.ms changed: What|Removed |Added Flags|fedora-review? |fedora-review+

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #15 from Taylor Braun-Jones tay...@braun-jones.org --- (In reply to Raphael Groner from comment #13) I did however hit a couple failing unit tests due to incompatibility with Lua 5.3. I have simply disabled Lua support on

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #13 from Raphael Groner projects...@smart.ms --- (In reply to Taylor Braun-Jones from comment #12) … I'm not able to reproduce the rawhide error that you're are seeing. Since what you were seeing was an internal compiler error, I

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #14 from Raphael Groner projects...@smart.ms --- Created attachment 1011857 -- https://bugzilla.redhat.com/attachment.cgi?id=1011857action=edit licensecheck.txt -- You are receiving this mail because: You are on the CC list for

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner projects...@smart.ms changed: What|Removed |Added See Also|https://bugzilla.redhat.com |

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #8 from Raphael Groner projects...@smart.ms --- Tried with fedora-review but mock build fails for rawhide. Relevant log snippet: /usr/bin/cmake -E cmake_progress_report /builddir/build/BUILD/vera++-1.3.0/build/CMakeFiles 36 [ 63%]

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #10 from Taylor Braun-Jones tay...@braun-jones.org --- (In reply to Raphael Groner from comment #8) Tried with fedora-review but mock build fails for rawhide. Thanks. Yes, I need to look into that as soon as f22, epel6, and

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #11 from Raphael Groner projects...@smart.ms --- Okay, for now this is FTBFS. Each package must be build for rawhide to comply with policy. Unfortunately, we must wait for upstream or any available patch to fix gcc5. While this is

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #9 from Raphael Groner projects...@smart.ms --- Created attachment 1011553 -- https://bugzilla.redhat.com/attachment.cgi?id=1011553action=edit c++: internal compiler error: Killed (program cc1plus) build.log as mock gives me

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner projects...@smart.ms changed: What|Removed |Added See Also|

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #12 from Taylor Braun-Jones tay...@braun-jones.org --- Thanks, I didn't realize that was the policy (nor did the two reviewers of my recently approved ceres-solver package request which also has problems building with gcc-5.0) I'm

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #7 from Taylor Braun-Jones tay...@braun-jones.org --- Looking into the details of vera++, the Fedora Tcl guidelines, and the %{name}-plugins subpackage option: * A tcl-%{name} package does not apply because vera++ includes a

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #6 from Taylor Braun-Jones tay...@braun-jones.org --- (In reply to Raphael Groner from comment #4) Thanks for the fixes. So let's look deeper into the project details. (In reply to Taylor Braun-Jones from comment #3) (In

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #4 from Raphael Groner projects...@smart.ms --- Thanks for the fixes. So let's look deeper into the project details. (In reply to Taylor Braun-Jones from comment #3) (In reply to Raphael Groner from comment #2) * MUST: Remove

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #5 from Raphael Groner projects...@smart.ms --- * There's also a manpage generated that you should include into the package.

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #2 from Raphael Groner projects...@smart.ms --- Informal review of your spec file. My comments are in order of the spec file lines. * MUST: Remove the Group: line. This tag is deprecated since Fedora 17.

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 Raphael Groner projects...@smart.ms changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug 1208738] Review Request: vera++ - A tool for verification, analysis and transformation of C++ source code

https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #3 from Taylor Braun-Jones tay...@braun-jones.org --- (In reply to Raphael Groner from comment #2) * MUST: Remove the Group: line. This tag is deprecated since Fedora 17. https://fedoraproject.org/wiki/