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

Bug 565511

Summary: [Chromium] swt-chromium-win32*.dll present at wrong folder.
Product: [Eclipse Project] Platform Reporter: Niraj Modi <niraj.modi>
Component: SWTAssignee: Niraj Modi <niraj.modi>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: guillez
Version: 4.17   
Target Milestone: 4.17 M3   
Hardware: PC   
OS: Windows 10   
Whiteboard: m2
Bug Depends on:    
Bug Blocks: 549585    

Description Niraj Modi CLA 2020-07-24 03:33:24 EDT
swt-chromium-win32*.dll present at wrong folder, not expected to be present under directory below directory:
bundles/org.eclipse.swt.win32.win32.x86_64/
Comment 1 Niraj Modi CLA 2020-07-24 03:34:15 EDT
I removed this extra dll via below patch:
https://git.eclipse.org/r/c/platform/eclipse.platform.swt.binaries/+/166695
Comment 2 Niraj Modi CLA 2020-07-24 03:38:00 EDT
@Guillermo,
As per our understanding, the swt-chromium-win32*.dll is not expected to be under:
bundles/org.eclipse.swt.win32.win32.x86_64/

Let us know if you feel otherwise. Thanks!
Comment 3 Guillermo Zunino CLA 2020-07-24 10:37:55 EDT
(In reply to Niraj Modi from comment #2)
> @Guillermo,
> As per our understanding, the swt-chromium-win32*.dll is not expected to be
> under:
> bundles/org.eclipse.swt.win32.win32.x86_64/
> 
> Let us know if you feel otherwise. Thanks!

That's correct. No chromium binaries should be required in swt bundles. Only in chromium fragments.
Comment 4 Niraj Modi CLA 2020-07-24 11:30:53 EDT
(In reply to Guillermo Zunino from comment #3)
> (In reply to Niraj Modi from comment #2)
> That's correct. No chromium binaries should be required in swt bundles. Only
> in chromium fragments.

Thanks for the confirmation, resolving now.
Comment 5 Niraj Modi CLA 2020-08-19 05:25:38 EDT
Verified in sources.