This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 58752 - Calibration script doesn't support JDK5/6 VM
Summary: Calibration script doesn't support JDK5/6 VM
Status: CLOSED FIXED
Alias: None
Product: profiler
Classification: Unclassified
Component: Base (show other bugs)
Version: 4.x
Hardware: All All
: P2 blocker (vote)
Assignee: Jiri Sedlacek
URL:
Keywords:
Depends on:
Blocks: 58753
  Show dependency tree
 
Reported: 2005-05-11 05:53 UTC by mishadmitriev
Modified: 2006-07-24 14:31 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description mishadmitriev 2005-05-11 05:53:22 UTC
Users of our remote profiling stuff are confused when they try to use it with
JDK5/6, since in that case the profiler server code requires machinedata.jdk15
file, but we ship only one calibration script per hardware platform, that has
hardcoded call to the customized 1.4.2 JFluid VM. Furthermore, this VM won't be
available by default in M7.

Need to rewrite the calibration scripts, perhaps to make them require the path
to the VM the user intends to use for profiling as a parameter. Calibration data
will be saved per VM version anyway (into ~/.jfluid/machinedata or
~/.jfluid/machinedata.jdk15)
Comment 1 ygor_sk 2005-06-06 13:09:05 UTC
M6v3 calibration scripts are a great improvement but:
- profiler-server-m6v3-solsparc-15.zip contains an error in calibrate.sh script,
the name of the lib directory should be solaris-sparc and not linux, there might
be more of these tiny errors
- scripts for linux and solaris might end up with NoClassDefFoundError for
ProfilerCalibrator class, since all lines contain #10 character at the end,
which confuses the java command
Comment 2 iformanek 2005-06-06 14:11:22 UTC
Thanks for reporting the troubles, I fixed those for M7.
Comment 3 ehucka 2006-07-24 14:31:39 UTC
old issue, closed