Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 321905

Summary: IMAGE_FILTER set incorrectly if project is external
Product: z_Archived Reporter: Severin Gehwolf <sgehwolf>
Component: LinuxToolsAssignee: OProfile Inbox <linux.oprofile-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: kksebasti, sgehwolf
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Linux   
Whiteboard:
Attachments:
Description Flags
Regression-test for this bug. akurtakov: iplog+

Description Severin Gehwolf CLA 2010-08-05 13:32:32 EDT 
Steps to reproduce:

1. Create new project
2. Uncheck "Use default location"
3. Compile, try to profile
4. Eclipse shows "No profiling data is on the system"

When looking at /root/.oprofile/daemonrc

$ grep IMAGE_FILTER /root/.oprofile/daemonrc
IMAGE_FILTER=/path/to/current/worspace/project_name/binary

But since it's an external project this path does not exist.

I remember that Eclipse's external project support is fairly limited. Not sure if this is the problem.
Comment 1 Roland Grunberg CLA 2010-08-06 11:32:10 EDT 
Should be fixed in the revision referenced by BZ # 296228 .
Comment 2 Severin Gehwolf CLA 2010-08-10 11:16:19 EDT 
Created attachment 176246 [details]
Regression-test for this bug.

This bug seems to apply to the R0_5_0 tag. Later versions and branches/0.5.0 have been fixed. I wrote a test which fails when run on tags/R0_5_0 and succeeds on branches/0.5.0 which includes the fix.
Comment 3 Severin Gehwolf CLA 2010-08-10 14:27:16 EDT 
(In reply to comment #2)
> Created an attachment (id=176246) [details]
> Regression-test for this bug.
> 
> This bug seems to apply to the R0_5_0 tag. Later versions and branches/0.5.0
> have been fixed. I wrote a test which fails when run on tags/R0_5_0 and
> succeeds on branches/0.5.0 which includes the fix.

With this patch applied launch.tests should all pass in trunk :-)
Comment 4 Roland Grunberg CLA 2010-08-10 15:45:07 EDT 
Revision 25347 and 25348. Thanks for the patch :) Confirmed that test passes on trunk.
Comment 5 Roland Grunberg CLA 2010-08-27 15:43:53 EDT 
Fixed.