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

Bug 380850

Summary: Server Icon is Eclipse 4.2
Product: [ECD] Orion Reporter: Ian Skerrett <ian.skerrett>
Component: RelengAssignee: Anthony Hunter <ahunter.eclipse>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: ahunter.eclipse, gheorghe, ken_walker, mike.milinkovich, torkildr
Version: 0.5   
Target Milestone: 6.0 M2   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Bug Depends on: 379297    
Bug Blocks:    

Description Ian Skerrett CLA 2012-05-28 18:38:31 EDT
In the 0.5 M2 download the icon for the server process is the Eclipse 4.2 icon.  It should be the Orion icon.
Comment 1 John Arthorne CLA 2012-05-30 15:53:12 EDT
We actually have a good icon, but there was a bug in the p2 branding code that prevents it working for a large 256x256 icon. We will need to move up to a new builder with this fix.
Comment 2 Ken Walker CLA 2012-06-19 17:09:53 EDT
Did we get a new builder for this?
Comment 3 John Arthorne CLA 2012-06-19 18:00:02 EDT
No we are still using the eclipse 3.7 builder. At this point we'll just move up after we both ship.
Comment 4 John Arthorne CLA 2012-11-23 09:25:30 EST
I upgraded to 3.8 base builder and unfortunately the branding still did not apply.

http://git.eclipse.org/c/orion/org.eclipse.orion.server.git/commit/?id=fffb35d93527321d6382ece348bdc800fbec29d9

I have checked the orion.ico file and the icons look good. I tried manually invoking the branding iron and it applied the orion icons correctly to the executable, so I need to debug the build to figure out what is going on. It is possibly a problem of the orion.ico not being found at build time.
Comment 5 John Arthorne CLA 2012-11-23 10:50:58 EST
Found this in the build log:

generate.p2.metadata:
[eclipse.brand.p2.artifacts] Error - 7 icon(s) not replaced in /opt/public/eclipse/e4/orion/I201211222230/tmp/p2.branding/org.eclipse.equino
x.executable_root.win32.win32.x86/launcher.exe using /shared/eclipse/e4/orion/I201211222230/../support/org.eclipse.orion.releng/builder/html
/orion.ico

This message comes from IconExe line 79. It means the executable was found and contained 7 images, but there were no matching replacement images found. I confirmed the orion.ico referenced in that path above was present and correct though. Again, running IconExe on my Windows box with the same inputs ran correctly with no errors. Bogdan and I are both stumped at the moment.
Comment 6 Mike Milinkovich CLA 2013-02-20 19:45:27 EST
Any chance this is going to get fixed for Orion 2.0 ?
Comment 7 John Arthorne CLA 2013-02-22 12:55:39 EST
(In reply to comment #6)
> Any chance this is going to get fixed for Orion 2.0 ?

Nope.
Comment 8 Torkild Resheim CLA 2014-05-26 10:42:57 EDT
I'm seeing this same thing when building our Eclipse 4.4 based product using PDE build. The builder we're using is from Eclipse 4.4M7.
Comment 9 Mike Milinkovich CLA 2014-05-26 10:54:39 EDT
(In reply to Torkild U. Resheim from comment #8)
> I'm seeing this same thing when building our Eclipse 4.4 based product using
> PDE build. The builder we're using is from Eclipse 4.4M7.

Torkild,

This bug is related to the fact that the Orion server process uses the Eclipse icon. 

If your bug is that you are picking up an old Eclipse icon for your project's build, you should file a separate bug.

John,

While I have you, any chance of fixing this in 6.0?
Comment 10 Torkild Resheim CLA 2014-05-26 11:34:29 EDT
Mike,

Sorry for hijacking the bug. I realize it appears to be specific to Orion. However I see exactly the same symptoms. The log output is the same and the icons in the *.exe file are not replaced. They remain those of Eclipse (4.4 in this case). I also tested by building IconExe and running it on my .exe and *.ico which produced a correct result.
Comment 11 John Arthorne CLA 2014-06-16 16:35:13 EDT
This has been fixed. I have 6.0 M2 Windows build and I see the iron logo in the exe file.