[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 Hanns-Joachim Uhl changed: What|Removed |Added Blocks||1451303

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 --- Comment #14 from Richard W.M. Jones rjo...@redhat.com --- I can't upload the sources at the moment ... $ fedpkg -v new-sources camlp4-87c6a6b07818acbbef6ced00cc8f4e09b533e055.tar.gz Uploading: dc6e69fd7a66639fe1d1393c87ada6fa

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 --- Comment #12 from Richard W.M. Jones rjo...@redhat.com --- New Package SCM Request === Package Name: ocaml-camlp4 Short Description: Pre-Processor-Pretty-Printer for OCaml Upstream URL: https://github.com/ocaml/camlp4

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? --

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

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

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 --- Comment #13 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 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Flags|

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Flags|

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 Michel Alexandre Salim mic...@michel-slm.name changed: What|Removed |Added Flags|fedora-review?

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 --- Comment #8 from Richard W.M. Jones rjo...@redhat.com --- (In reply to Michel Alexandre Salim from comment #6) Reviewing now; while I'm at it, the license field should probably be changed to just LGPLv2 ? Here are the commit logs for

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 --- Comment #9 from Richard W.M. Jones rjo...@redhat.com --- Spec URL: http://oirase.annexia.org/reviews/camlp4/ocaml-camlp4.spec SRPM URL: http://oirase.annexia.org/reviews/camlp4/ocaml-camlp4-4.02.0-0.2.git87c6a6b0.fc20.src.rpm -- You are

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 Bug 1119081 depends on bug 1119049, which changed state. Bug 1119049 Summary: ocamlopt.opt segfaults when compiling ounit https://bugzilla.redhat.com/show_bug.cgi?id=1119049 What|Removed |Added

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 --- Comment #5 from Richard W.M. Jones rjo...@redhat.com --- ARM builds now work. There is a Koji scratch build here: http://koji.fedoraproject.org/koji/taskinfo?taskID=7143902 -- You are receiving this mail because: You are on the CC list

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 --- Comment #6 from Michel Alexandre Salim mic...@michel-slm.name --- Reviewing now; while I'm at it, the license field should probably be changed to just LGPLv2 ? Here are the commit logs for that file on Github:

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 --- Comment #7 from Michel Alexandre Salim mic...@michel-slm.name --- Apart from the license, there are a couple more issues: - duplicate ocaml(runtime) and ocaml-runtime in Requires: -- the manually added requirement is probably unnecessary?

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 Michel Alexandre Salim mic...@michel-slm.name changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Flags|needinfo?(rjo...@redhat.com |

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 --- Comment #1 from Richard W.M. Jones rjo...@redhat.com --- rpmlint output with my comments inline: ocaml-camlp4.src: W: spelling-error Summary(en_US) Pre - Pr, Re, Pres ocaml-camlp4.src: W: spelling-error %description -l en_US runtime - run

[Bug 1119081] Review Request: ocaml-camlp4 - Pre-Processor-Pretty-Printer for OCaml

https://bugzilla.redhat.com/show_bug.cgi?id=1119081 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Depends On||1119049 ---