Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 357728

Summary:	EGL Compiler can NOT handler illegal directory name
Product:	z_Archived	Reporter:	Yu Hao <yuhaodl>
Component:	EDT	Assignee:	Tony Chen <chenzhh>
Status:	CLOSED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	chenzhh, yuhaodl
Version:	unspecified
Target Milestone:	---
Hardware:	PC
OS:	Windows XP
Whiteboard:

Description Yu Hao

2011-09-15 01:56:30 EDT

Build Identifier: 

If you set the Generation directory to an illegal name, the EGL compiler setting dialog did NOT notice any warning message, then the generating will work abnormal

Reproducible: Always

Steps to Reproduce:
1.Create a project
2.Set the EGL Compiler, set the generation direction to '~/generatedJava'
3.Apply and close the windows
4.There will create two new folders one is '~/generatedJava' and another is '~'
5.Re-open properties->EGL Compiler and click 'Restore Defaults' button, nothing happened

Comment 1 Tony Chen

2011-09-18 22:32:33 EDT

This is work as designed.

~ is a valid char for folder name

~/generatedJava means you want to create two nested folders ~ (as parent) and generatedJava (as childern), and make the children folder the target for generation. The parent folder(~) is listed in project explorer as a folder resource. The generation target folder (~/generatedJava) is set as a JavaSource folder, so it is listed as JavaSource folder, you can see a label decorator on it. 

'Restore Defaults' means if you have typed in some text, and want to discard it, you press 'Restore Defaults' to get the values which was there before you type. It is not to restore to the workspace preference.

Comment 2 Tony Chen

2011-09-18 22:34:46 EDT

resolve as invalid. 

Both parent and target folders being listed might look weird here. What I'm actually thinking is that a user should never specify a sub folder as the generation output folder.

Comment 3 Yu Hao

2011-09-21 01:44:06 EDT

The invalid folder name which include "? " *" can not be handler. I can set the generator folder to ""?*" and apply it. No warning/error message display. And the illegal folder was not created. So the generator setting failed.

Comment 4 Tony Chen

2011-10-08 23:08:09 EDT

Fixed. Now it is checking for invalid resource name of the platform.

Comment 5 Tony Chen

2011-10-08 23:08:24 EDT

fixed

Comment 6 Yu Hao

2011-10-12 22:31:20 EDT

Verified in 201110120952