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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Tomas Repik <trepik@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |needinfo?(jvanek@xxxxxxxxxx
                   |                            |)



--- Comment #4 from Tomas Repik <trepik@xxxxxxxxxx> ---
(In reply to jiri vanek from comment #3)
> [!]: %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.

Tests are executed during build, using maven-surefire-plugin. Only
HistogramPerfTest is not run, but I don't think it's necessary. As [1] tells:
"Optional %check section
Runs projects integration tests. Unit test are usually run in %build section,
so if there are no integration tests available, this section is omitted"

> [!]: Package functions as described.
> 
> There is quite a lot interesting things in the source tarball. One of them
> is launcher. Others are web based histrograms. I would expect them to work
> in installed application. Rightnow they are just missing.

I can't see any more files needed to be installed. Can you be more specific?

[1] https://fedorahosted.org/released/javapackages/doc/#_example_rpm_project

-- 
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
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]