[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 Alexander Kurtakov changed: What|Removed |Added Blocks|652183 (FE-JAVASIG) |

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 Tomas Repik changed: What|Removed |Added Blocks|177841 (FE-NEEDSPONSOR) | Referenced Bugs:

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #22 from Fedora Update System --- HdrHistogram-2.1.8-1.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report. -- You are

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 Fedora Update System changed: What|Removed |Added Status|ON_QA

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #20 from Fedora Update System --- HdrHistogram-2.1.8-1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-3cc7c1c581 -- You are receiving this mail

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 Fedora Update System changed: What|Removed |Added Status|POST

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #19 from jiri vanek --- (In reply to Tomas Repik from comment #17) > --- SPECS/old/HdrHistogram.spec 2016-03-03 15:58:04.925801810 +0100 > +++ SPECS/HdrHistogram.spec 2016-03-07 12:37:58.916005527 +0100

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #18 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/HdrHistogram -- You are receiving this mail because: You are on the CC list for the bug. You are

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #17 from Tomas Repik --- --- SPECS/old/HdrHistogram.spec2016-03-03 15:58:04.925801810 +0100 +++ SPECS/HdrHistogram.spec2016-03-07 12:37:58.916005527 +0100 @@ -1,6 +1,6 @@ Name: HdrHistogram

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #16 from jiri vanek --- Yah, you are right with flags. About the c#15 - was there some change against the original one i approved? If so, please post diff! I short time I should be able to sponsor you. --

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #15 from Tomas Repik --- Spec URL: https://trepik.fedorapeople.org/HdrHistogram/HdrHistogram.spec SRPM URL: https://trepik.fedorapeople.org/HdrHistogram/HdrHistogram-2.1.8-1.fc23.src.rpm * Mon Mar 07 2016 Tomas

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #14 from Tomas Repik --- (In reply to jiri vanek from comment #12) > Not sure how your other requests, but this oen was missing fedora-review=? > If your other requestsd are missing it, set it for your own good.

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 Tomas Repik changed: What|Removed |Added Flags|needinfo?(tre...@redhat.com | |)

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 jiri vanek changed: What|Removed |Added Status|ASSIGNED|POST

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 jiri vanek changed: What|Removed |Added CC||tre...@redhat.com

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #10 from Tomas Repik --- (In reply to jiri vanek from comment #9) > 48: %jpackage_script org.fedoraproject.helloworld.HelloWorld "" "" %{name} > helloworld true Done. > Also I would go with %{_bindir} Done.

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #9 from jiri vanek --- Hello! You tried to install this package+launcher and it worked? I did not, but: cp HistogramLogProcessor %{buildroot}%{_libdir}/HistogramLogProcessor Can never work. Even ig, it is

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #8 from Tomas Repik --- Spec URL: https://trepik.fedorapeople.org/HdrHistogram.spec SRPM URL: https://trepik.fedorapeople.org/HdrHistogram-2.1.8-2.fc23.src.rpm * Wed Mar 02 2016 Tomas Repik

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #7 from jiri vanek --- I can agree on it. But Still I would say - upstream have an launcher => include launcher. No subpackage needed. -- You are receiving this mail because: You are on the CC list for the

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #6 from Tomas Repik --- Well, first of all thanks for the review, I appreciate it. > Thoughts? For me it's more like a library than a real runable application although it would make sense as well. I could try

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 jiri vanek changed: What|Removed |Added Flags|needinfo?(jva...@redhat.com | |)

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 Tomas Repik changed: What|Removed |Added Flags|

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #3 from jiri vanek --- [!]: %check is present and all tests pass. Unless there is some reason I cant see, i probably insists execution of tests during build. there is even junit in In original sources. [!]:

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 --- Comment #2 from jiri vanek --- Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated [ ] = Manual review needed = MUST items = Generic: [x]: Package is

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 jiri vanek changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 gil cattaneo changed: What|Removed |Added Blocks||177841

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 gil cattaneo changed: What|Removed |Added Blocks||1010003

[Bug 1305496] Review Request: HdrHistogram - A High Dynamic Range Histogram

https://bugzilla.redhat.com/show_bug.cgi?id=1305496 gil cattaneo changed: What|Removed |Added Blocks||652183 (FE-JAVASIG)