Bug 28531

Summary:	Classpath Entry: Output folder can not be set to project
Product:	[Eclipse Project] JDT	Reporter:	Martin Aeschlimann <martinae>
Component:	Core	Assignee:	Jerome Lanneluc <jerome_lanneluc>
Status:	VERIFIED FIXED	QA Contact:
Severity:	normal
Priority:	P3
Version:	2.1
Target Milestone:	2.1 M5
Hardware:	PC
OS:	Windows 2000
Whiteboard:

Description Martin Aeschlimann

2002-12-17 11:35:18 EST

20021216
1. create new project
2. source folder on project
3. set the source folder's output folder also to the project
src-entry:
 /proj1
   output folder: /proj1
default output location:
 /proj/bin
4. ok
5. reopen the build paths dialog: source folder's output location is null

setRawClasspath contains the output folder path.
A getRawClasspath just following still contains it, but when reopening the 
wizard, the getRawClasspath has a 'null' value for the output folder.

Comment 1 Jerome Lanneluc

2002-12-17 13:17:59 EST

Martin, isn't it a dup of bug 28463?

Comment 2 Philipe Mulet

2002-12-17 14:43:00 EST

.classpath file looks fine. Sounds like a dup of UI bug.

Comment 3 Martin Aeschlimann

2002-12-18 03:41:50 EST

I fixed bug 28463, but then I saw this problem.
the created .classfile is
<?xml version="1.0" encoding="UTF-8"?>
<classpath>
    <classpathentry excluding="src/" kind="src" output="" path=""/>
    <classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER"/>
    <classpathentry kind="output" path="bin"/>
</classpath>

Shouldn't the output be "proj", or is it also a project relative path? If it 
is, I guess the problem is to know if "" means 'project' or 'no output folder 
set'

Comment 4 Philipe Mulet

2002-12-18 04:15:41 EST

It is a project relative path. 
Null should be answered when no output is used.

Need to double check. Not critical.

Comment 5 Jerome Lanneluc

2003-01-10 05:24:29 EST

When reading the .classpath file, we used to consider that the "output" 
attribute if empty was equivalent to no attribute.

Fixed ClasspathEntry.elementDecode to check if the attribute exists instead.
Added ClasspathTests.testReadEmptyCustomOutput().

Comment 6 David Audel

2003-02-07 11:01:43 EST

Verified.