Bug 195716 - [new port] java/GCViewer: classic little tool for java garbage collector log analysis
Summary: [new port] java/GCViewer: classic little tool for java garbage collector log ...
Status: Closed Feedback Timeout
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-12-05 16:53 UTC by Michael Zhilin
Modified: 2015-05-17 15:42 UTC (History)
4 users (show)

See Also:


Attachments
SHAR (2.64 KB, application/x-shar)
2014-12-05 16:53 UTC, Michael Zhilin
no flags Details
Port test logs (535.27 KB, text/x-log)
2014-12-05 16:54 UTC, Michael Zhilin
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Zhilin freebsd_committer freebsd_triage 2014-12-05 16:53:01 UTC
Created attachment 150238 [details]
SHAR

Hello,

Please have a look at adding port for GCViewer tool. This little tool visualizes Java garbage collector logs providing performance metrics like throughput, timings of gc pauses, speed of collections. So it's widely used for performance analysis/monitoring of Java applications. 

This port makes java source code compilation instead of binary downloads. I'll checked build/installation steps, validated port by portlint & porttools. Also of course I've checked that it works functionally. :)
Comment 1 Michael Zhilin freebsd_committer freebsd_triage 2014-12-05 16:54:12 UTC
Created attachment 150239 [details]
Port test logs
Comment 2 Pawel Pekala freebsd_committer freebsd_triage 2014-12-30 20:59:18 UTC
Few ideas for improvement:

- port tries to fetch files during build phase, this is forbidden:

http://people.freebsd.org/~pawel/buildlogs/GCViewer-1.34-10amd64.log

- lines in Makefile should be broken around 80 characters for easier reading

XPORTNAME=	GCViewer

Please don't use capital letters in port names.

X#!/bin/sh
X#
X# $FreeBSD$
X
XJAVA_VERSION="%%JAVA_VERSION%%" "%%LOCALBASE%%/bin/java" -jar "%%JAVAJARDIR%%/gcviewer.jar" "$@"

Use exec, please see:

https://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/dads-sh-exec.html
Comment 3 Jason Helfman freebsd_committer freebsd_triage 2014-12-31 17:08:58 UTC
I have ported several java-based ports that fetch during their build phase, and not only is this a bad idea, but it also make the build a moving target. So much, that it isn't a predictable and repeatable process.

The best way to tackle this is to build locally, and then package up the repository that is populated by maven. Then point ant, or whatever compiler is used, to include your local build repository for sources.

A good example of this is archivers/snappy-java

Please let me know if you need any assistance, and I can see what I can help with.

-jgh
Comment 4 John Marino freebsd_committer freebsd_triage 2015-05-17 15:42:37 UTC
No updated submission has been received more than 5 months after the initial review, so I'm closing the PR.