Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 299977 - environment variables from client are merged with that of the server when building
Summary: environment variables from client are merged with that of the server when bui...
Status: RESOLVED FIXED
Alias: None
Product: PTP
Classification: Tools
Component: RDT (show other bugs)
Version: 2.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 2.1.4   Edit
Assignee: Chris Recoskie CLA
QA Contact: Chris Recoskie CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-18 14:13 EST by Mike Kucera CLA
Modified: 2011-05-14 06:42 EDT (History)
2 users (show)

See Also:


Attachments
patch (12.52 KB, patch)
2010-01-18 14:13 EST, Mike Kucera CLA
no flags Details | Diff
fixed some issues (2.19 KB, patch)
2010-01-19 13:45 EST, Peter Wang CLA
g.watson: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Kucera CLA 2010-01-18 14:13:08 EST
The CDT variable manager is taking the variables from our RemoteConfigurationEnvironmentSupplier and appending them to the local environment.  There is a radio button that allows you to specify to replace the native environment (which corresponds to the local environment), or append to it, and by default it appends.  We can't really change that behaviour, BUT, if we create a new version of that page without any option to append or replace, and just have it replace by default, that should fix things.
Comment 1 Mike Kucera CLA 2010-01-18 14:13:49 EST
Created attachment 156427 [details]
patch
Comment 2 Mike Kucera CLA 2010-01-18 15:11:50 EST
Fixed in 2.1, 3.0 and head
Comment 3 Peter Wang CLA 2010-01-19 13:45:52 EST
Created attachment 156544 [details]
fixed some issues

Fixed spelling mistake in code, made labels more detailed.
Comment 4 Vivian Kong CLA 2010-01-19 14:18:25 EST
(In reply to comment #3)
> Created an attachment (id=156544) [details]
> fixed some issues
> 
> Fixed spelling mistake in code, made labels more detailed.

Fixed in ptp_2_0, ptp_3_0 and HEAD. Thanks Peter
Comment 5 Peter Wang CLA 2010-03-23 18:26:31 EDT
Removed (Author request)