Bug 32097 - Regression - attached source not found when in a subdirecory of archive
Summary: Regression - attached source not found when in a subdirecory of archive
Status: CLOSED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 2.1   Edit
Hardware: PC Windows 2000
: P3 normal (vote)
Target Milestone: 2.1 RC1   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-02-18 01:55 EST by Ron Baldwin CLA
Modified: 2003-02-20 01:38 EST (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ron Baldwin CLA 2003-02-18 01:55:33 EST 
When using "Open Declaration", source files in an attached source archive are
not found when the source tree is in a subdirectory of the archive.

For example, when using the downloaded zip for log4j as the source attachment
(jakarta.apache.org/log4j/docs/download.html), the source is located at
"/jakarta-log4j-1.2.7/src/java" inside the zip file.  This worked when the
source root could be specified.
Comment 1 Philipe Mulet CLA 2003-02-18 06:20:53 EST 
The root path inference code is being revisited at the moment (a better version 
is released today for integration so as to support multiple root paths).

Olivier - can you please verify this one issue with our new inference code?
Comment 2 Olivier Thomann CLA 2003-02-18 09:03:46 EST 
I will investigate.
Comment 3 Olivier Thomann CLA 2003-02-18 09:19:51 EST 
I tried the following settings and it works great with the latest changes.
I create a java project and I added the external library log4j-1.2.7.jar on the
classpath. I attached the source to the directory in which I unzipped the zip
file (for me D:\jakarta-log4j-1.2.7) and I could get the source for the .class
file inside log4j-1.2.7.jar without a problem.
Then I attached the source to D:\jakarta-log4j-1.2.7\src and it worked as well.
So this is fixed with today's integration build.
Please reopen if it doesn't work for you.
Comment 4 Olivier Thomann CLA 2003-02-18 09:26:39 EST 
Closed as the fix was made for another bug.
Comment 5 Ron Baldwin CLA 2003-02-20 01:38:59 EST 
Verified works in I20030218.