Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 338418 - Launch configuration's Environment tab variables are not passed to the gdb process itself
Summary: Launch configuration's Environment tab variables are not passed to the gdb pr...
Status: CLOSED WONTFIX
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug-cdi-gdb (show other bugs)
Version: 8.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact: Elena Laskavaia CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-28 09:45 EST by Gaetano Santoro CLA
Modified: 2016-04-02 16:18 EDT (History)
3 users (show)

See Also:


Attachments
proposed patch (3.22 KB, patch)
2011-02-28 09:46 EST, Gaetano Santoro CLA
no flags Details | Diff
proposed patch (3.26 KB, patch)
2011-02-28 11:22 EST, Gaetano Santoro CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gaetano Santoro CLA 2011-02-28 09:45:34 EST
Build Identifier: I20110127-2034

environment variables defined in launch configurations environment tab are not visible by the gdb process. As consequence of this, command/executable launched inside .gdbinit script or other gdb script cannot use the above defined environment variable
this appears only in debug, while works in run mode

Reproducible: Always

Steps to Reproduce:
1. create a simple C hello world project
2. in launch configuration environment tab add a variable, for eg. ZHELLO = CIAO
3. start debug
4. with external tool like sysinternals project explorer check that the environment of gdb process don't have the ZHELLO variable defined
Comment 1 Gaetano Santoro CLA 2011-02-28 09:46:56 EST
Created attachment 189947 [details]
proposed patch

attached my proposed patch
Comment 2 Gaetano Santoro CLA 2011-02-28 11:22:31 EST
Created attachment 189963 [details]
proposed patch

moved code to AbstractCLaunchDelegate so it is available to general class that extend it
Comment 3 Marc Khouzam CLA 2016-04-02 16:18:59 EDT
CDI has been removed so there is no more need for this fix.