Bug 229414 - Does not allow folder root in package naming structure - FORCES nesting!
Summary: Does not allow folder root in package naming structure - FORCES nesting!
Status: VERIFIED WORKSFORME
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.3.2   Edit
Hardware: PC Windows 2000
: P3 normal (vote)
Target Milestone: 3.6 M5   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords: needinfo
Depends on:
Blocks:
 
Reported: 2008-04-29 13:49 EDT by Ken Silveman CLA
Modified: 2010-01-25 02:18 EST (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ken Silveman CLA 2008-04-29 13:49:18 EDT
Build ID: M20080221-1800

Steps To Reproduce:
Create some classes in c:\dev\webmind\core    then Open a NEW project using existing code.  Specify code folder as "webmind".  ALL builds FAIL because Eclipse is UNFLEXIBLE and will not allow your classes to have webmind in the package name as "package webmind.core"  (see more info.)

More information:
If you specify c:\dev as the package - that is a stupid solution, because then you have to exclude all the other folders you may have in dev that you dont want in the project - and yet they still show up in the package explorer - that is a second bug!!!! If you exclude it, you shouldn't have to see it anymore!!!!  

But still bogus, You SHOULD BE ABLE TO INLCUDE THE FOLDER ROOT (i.e. webmind)  in the package naming structure - and eclipse should KNOW how to deal with this!!!!!!  I have wasted weeks on this problem, over and over again.  You can't get a simple clean project explorer view the way you want it.  Linking the source gives the SAME PROBLEM - 

A forced folder nesting is a BOGUS solution as well.  Why have you not addressed this ages ago???  It is so simple to have a check box for "package naming includes root".

If I want to have a c:\com folder and all my classes under com have com.xxx in the package name - you can't win with eclipse!  Why?  Becasue c:\ has tons of other files and folders that you dont want in the project.  Forcing a nesting is not a good solution.  

Forcing the programmer to not have com in the package name - when it is a good programming practice (for many) to include the parent directory name of the actual location of the class in the package name is also not a good solution.  some times you may include two-up in the package naming structure.  Like "package sun.com.xxx"   Eclipse should be able to deal with this easily - with one checkbox only!  Peace. Ken 917 741 3377 kensilverman@rcn.com
Comment 1 Ken Silveman CLA 2008-04-29 13:58:43 EDT
This:

" More information:
If you specify c:\dev as the package "

should be changed to:

" More information:
If you specify c:\dev as the project source root instead of c:\dev\webmind "



As a side note, when adding a project in 3.32 with existing source, and you check "allow source folders as root"  you should not still be forced to specify a default output folder - like .../bin    because you may want ONLY the source folders to be the output folders.  Yet, with this forced default output folder,  "allow source as output" checkbox seems to be ignored - dumping all output in this default output folder that was FORCED to be specified right under the checkbox for "allow source as output"!   Highly counter-intuitive.
Comment 2 Jerome Lanneluc CLA 2008-08-25 11:36:50 EDT
(In reply to comment #0)
> Build ID: M20080221-1800
> 
> Steps To Reproduce:
> Create some classes in c:\dev\webmind\core
Could you please give an example of the classes you put in this directory? What is the package statement?
>    then Open a NEW project using
> existing code.  
What is the path of the project?

> Specify code folder as "webmind". 
What is a code folder? Do you mean a source folder?

> ALL builds FAIL because
> Eclipse is UNFLEXIBLE and will not allow your classes to have webmind in the
> package name as "package webmind.core"  (see more info.)
No need to shout :-) 

> More information:
> If you specify c:\dev as the package - that is a stupid solution, because 
> then you have to exclude all the other folders you may have in dev that you 
> dont want in the project 
What other folders? I'm not seeing other folder from the description you gave.

> - and yet they still show up in the package explorer 
> - that is a second bug!!!! 
Please enter a separate bug report if this is a second bug.

Skipping the rest of this bug report. Please see http://www.chiark.greenend.org.uk/%7Esgtatham/bugs.html
Comment 3 Olivier Thomann CLA 2010-01-05 10:02:40 EST
Closing as WORKSFORME.
Required information was never provided.
Reopen once you provided it.
Comment 4 Ayushman Jain CLA 2010-01-25 02:11:02 EST
Verified for 3.6M5.
Comment 5 Srikanth Sankaran CLA 2010-01-25 02:18:03 EST
Verified.