Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 323588 - External tools (Ant) launch config looses project(s) to be built
Summary: External tools (Ant) launch config looses project(s) to be built
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Ant (show other bugs)
Version: 3.6   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.7 M2   Edit
Assignee: Michael Rennie CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-25 06:36 EDT by Dani Megert CLA
Modified: 2010-08-31 06:02 EDT (History)
0 users

See Also:

Attachments
fix (1.48 KB, patch)
2010-08-25 10:20 EDT, Michael Rennie CLA 		no flags Details | Diff
View All 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 Dani Megert CLA 2010-08-25 06:36:16 EDT 
3.6.

0. create Java project 'P'
1. add an Ant script (or use attached f.xml)
2. select it, Run As > Ant Build...
3. on the 'Build' tab select 'Specific project' and choose 'P'
4. click 'Apply'
5. click 'Close'
6. select it, Run As > Ant Build...
==> dialog shows error: [Build]: No projects specified
Comment 1 Michael Rennie CLA 2010-08-25 10:20:35 EDT 
Created attachment 177426 [details]
fix

The problem was that we had some bad logic while trying to compute the project scope from the configuration. 

Specifically we were checking that the project name recovered from the configuration was longer than one char, when it should be checking if it is longer than zero chars
Comment 2 Michael Rennie CLA 2010-08-25 10:25:30 EDT 
applied fix to HEAD, and added a note to the smoke tests to test against resources of varying name lengths.
Comment 3 Dani Megert CLA 2010-08-26 02:22:09 EDT 
Bundle version update is missing.
Comment 4 Michael Rennie CLA 2010-08-26 10:17:51 EDT 
Done. Good catch Dani.
Comment 5 Dani Megert CLA 2010-08-26 10:22:54 EDT 
Verified in N20100825-2000 and HEAD.
Comment 6 Dani Megert CLA 2010-08-31 06:02:13 EDT 
>Verified in N20100825-2000 and HEAD.
Bundle version was not correct but corrected by Darin by now.