Bug 119853 - Generate AJDoc wizard doesn't work on the Mac
Summary: Generate AJDoc wizard doesn't work on the Mac
Status: RESOLVED FIXED
Alias: None
Product: AJDT
Classification: Tools
Component: UI (show other bugs)
Version: 1.2.1   Edit
Hardware: Macintosh Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: 1.5.3   Edit
Assignee: Andrew Eisenberg CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-12-08 08:28 EST by Sian January CLA
Modified: 2009-05-15 00:10 EDT (History)
1 user (show)

See Also:


Attachments
This patch fixes the hard coding of paths so that ajdoc can work on macs. (9.21 KB, patch)
2008-05-29 14:45 EDT, Andrew Eisenberg CLA
andrew.eisenberg: iplog+
Details | Diff
mylyn/context/zip (29.62 KB, application/octet-stream)
2008-05-29 14:45 EDT, Andrew Eisenberg CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Sian January CLA 2005-12-08 08:28:56 EST
The Generate AJDoc wizard asks the user to provide the location of tools.jar, however tools.jar doesn't exist as part of the Java installation on the Mac so the user cannot proceed with the wizard.
Comment 1 Helen Beeken CLA 2005-12-09 08:11:05 EST
After a brief look....

The underlying code in ajdoc (on aspectj's side) looks for com.sun.tools.javadoc.Main which, on windows and linux is contained within the tools.jar. What the AJdocWizard does in AJDT is ask for the location of the tools.jar and then adds it to the list of vmargs. What might be the fix is that within the AJdocWizard, instead of asking for the location of tools.jar, to instead on the MAC, ask for the location of this class - or the directory which it is in. Possibly, another alternative as Sian pointed out, is to ask in the wizard for where java is installed and then under the covers look for the relevant place.
Comment 2 Andrew Eisenberg CLA 2008-05-29 14:09:30 EDT
On macs, there is no tools.jar.  Instead, the desired class is in classes.jar.  Need to point to that instead.

The correct location should be something like:
/System/Library/Frameworks/JavaVM.framework/Versions/Classes/classes.jar

Need to change the Wizard to reflect this.

Also, there is a bit of hard coding of paths in the AJDoc implementation that doesn't work on macs.  Need to change this as well.
Comment 3 Andrew Eisenberg CLA 2008-05-29 14:45:15 EDT
Created attachment 102708 [details]
This patch fixes the hard coding of paths so that ajdoc can work on macs.

Also, I changed some of the UI messages.
Comment 4 Andrew Eisenberg CLA 2008-05-29 14:45:19 EDT
Created attachment 102709 [details]
mylyn/context/zip
Comment 5 Andrew Clement CLA 2008-05-29 19:23:12 EDT
patches committed.
iplog