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

Bug 466921

Summary: Branding icons eclipse16.png is missing
Product: [Eclipse Project] PDE Reporter: Vikas Chandra <Vikas.Chandra>
Component: UIAssignee: Lars Vogel <Lars.Vogel>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: Lars.Vogel, markus.kell.r, Vikas.Chandra
Version: 4.5Flags: Vikas.Chandra: review+
Target Milestone: 4.5 RC1   
Hardware: PC   
OS: Windows 7   
See Also: https://git.eclipse.org/r/47632
https://git.eclipse.org/c/pde/eclipse.pde.ui.git/commit/?id=e0466aff57d13ae5c10c4fba6330919863f85df2
Whiteboard:
Attachments:
Description Flags
See the icon of branding
none
this icon should be restored at /org.eclipse.pde.ui/icons/obj16/ none

Description Vikas Chandra CLA 2015-05-09 14:53:10 EDT
Created attachment 253358 [details]
See the icon of branding

On eclipse-SDK-I20150507-2000-win32-x86_64

See image attached
Comment 1 Vikas Chandra CLA 2015-05-09 15:38:05 EDT
Created attachment 253359 [details]
this icon should be restored at /org.eclipse.pde.ui/icons/obj16/
Comment 2 Vikas Chandra CLA 2015-05-09 15:41:56 EDT
Lars, can you please look at this and any other icons that may be missing in the same way as this one.
Comment 4 Lars Vogel CLA 2015-05-11 07:48:21 EDT
.

*** This bug has been marked as a duplicate of bug 466840 ***
Comment 5 Vikas Chandra CLA 2015-05-11 09:11:20 EDT
I cant see branding icons after pulling the latest code.

That's probably because org.eclipse.pde.ui/icons/obj16/eclipse16.png


Lars, can you please re-check?
Comment 6 Markus Keller CLA 2015-05-11 09:19:17 EDT
Not a dup & not fixed.

This is the other eclipse16.png that was found missing in bug 433714 comment 73. It is referenced in code in PDEPluginImages#DESC_PRODUCT_BRANDING.
Comment 7 Lars Vogel CLA 2015-05-11 09:22:11 EDT
(In reply to Vikas Chandra from comment #5)
> I cant see branding icons after pulling the latest code.
> 
> That's probably because org.eclipse.pde.ui/icons/obj16/eclipse16.png

Missed that, as this is not part of the plugin.xml. Sorry, again. Fixing that soon.
Comment 8 Eclipse Genie CLA 2015-05-11 09:24:19 EDT
New Gerrit change created: https://git.eclipse.org/r/47632
Comment 9 Lars Vogel CLA 2015-05-11 09:24:45 EDT
https://git.eclipse.org/r/47632
Comment 10 Lars Vogel CLA 2015-05-11 09:26:21 EDT
With this change, as icons in the product editor are present AFAICS. Please +1 so that this can be included in the build.
Comment 12 Lars Vogel CLA 2015-05-11 09:40:01 EDT
Thanks Vikas for the review flag.
Comment 13 Vikas Chandra CLA 2015-05-11 14:10:34 EDT
I will verify this in tomorrow's build. I will also try to find anything else in and around this area during verification.
Comment 14 Lars Vogel CLA 2015-05-12 04:08:19 EDT
Verified in 4.5.0.I20150511-2130