Bug 138007 - system_path macro should support common extensions in Windows
Summary: system_path macro should support common extensions in Windows
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Ant (show other bugs)
Version: 3.1.2   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: 3.3 M6   Edit
Assignee: Michael Rennie CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
: 95083 (view as bug list)
Depends on:
Blocks:
 
Reported: 2006-04-21 13:17 EDT by Dobes Vandermeer CLA
Modified: 2007-02-26 10:43 EST (History)
3 users (show)

See Also:

Attachments
Patch proposal (1.96 KB, patch)
2007-02-20 07:13 EST, Wieant 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 Dobes Vandermeer CLA 2006-04-21 13:17:12 EDT 
When I create a project, edit it's properties, choose "Builders", click "New...", and then select "Program", and enter as the location "${system_path:python.exe}", this works in Windows but not in Linux.  If I enter "${system_path:python}" it works in linux but not windows.  This means I cannot use a program builder in a cross-platform project!

The "system_path" macro should use the PATHEXT environment variable in Windows to figure out what command to really run, and behave as it currently does on other platforms.
Comment 1 Jerome Lanneluc CLA 2006-04-23 04:37:32 EDT 
Moving to Platform Ant
Comment 2 Darin Swanson CLA 2006-04-23 12:35:20 EDT 
*** Bug 95083 has been marked as a duplicate of this bug. ***
Comment 3 Darin Swanson CLA 2006-04-26 15:10:13 EDT 
Deferred to post 3.2
Comment 4 Darin Swanson CLA 2006-10-23 22:34:20 EDT 
reopening
Comment 5 Darin Swanson CLA 2007-01-10 12:50:37 EST 
not planned for 3.3
Comment 6 Wieant CLA 2007-02-20 07:13:47 EST 
Created attachment 59362 [details]
Patch proposal
Comment 7 Darin Swanson CLA 2007-02-20 10:24:28 EST 
Thank you for the patch.
I will review and investigate for 3.3
Comment 8 Darin Swanson CLA 2007-02-23 10:25:51 EST 
Reviewed and released patch.
Changes to SystemPathResolver.
Comment 9 Darin Swanson CLA 2007-02-23 10:26:19 EST 
Please verify Michael.
Comment 10 Michael Rennie CLA 2007-02-26 10:23:35 EST 
verified on Windows and Linux
Comment 11 Darin Swanson CLA 2007-02-26 10:43:52 EST 
Thanks Michael. Build notes updated.