Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 512489 - Add ability to sign dmg file to eclipse-dmg-packager plugin
Summary: Add ability to sign dmg file to eclipse-dmg-packager plugin
Status: RESOLVED FIXED
Alias: None
Product: CBI
Classification: Technology
Component: maven-plugins (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: CBI Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 461673 (view as bug list)
Depends on:
Blocks: 461670
  Show dependency tree
 
Reported: 2017-02-21 06:22 EST by Sravan Kumar Lakkimsetti CLA
Modified: 2017-04-01 12:41 EDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sravan Kumar Lakkimsetti CLA 2017-02-21 06:22:26 EST
We need to add ability to sign dmg file during the dmg file creation using eclipse-dmg-packager plugin.

See https://bugs.eclipse.org/bugs/show_bug.cgi?id=512019#c21 for proposed solution.

Going forward we will not be able to ship dmg files without signing as mentioned in https://developer.apple.com/library/content/releasenotes/MacOSX/WhatsNewInOSX/Articles/OSXv10.html#//apple_ref/doc/uid/TP40017145-SW1

So it better to enable signing by default.
Comment 1 Eclipse Genie CLA 2017-03-03 02:11:38 EST
New Gerrit change created: https://git.eclipse.org/r/92217
Comment 2 Sravan Kumar Lakkimsetti CLA 2017-03-17 02:00:53 EDT
Can we have this feature for M7? We need to deliver mac distribution as a signed dmg for 4.7

Thanks
Comment 3 Mikaël Barbero CLA 2017-03-17 04:22:52 EDT
Yes, it's on the top of my todo list after DevoxxUS next week.
Comment 4 Eclipse Genie CLA 2017-03-30 11:00:14 EDT
New Gerrit change created: https://git.eclipse.org/r/94154
Comment 5 Mikaël Barbero CLA 2017-03-30 11:02:10 EDT
I've pushed a review with the necessary changes. I will deploy a snapshot version tomorrow and see how it goes from there. I did not want to add a dependency between the jarsigner and dmgpackager services, so I've re-implemented a signer in dmgpackager (hence the new review instead of re-using your scaffolding code)
Comment 7 Mikaël Barbero CLA 2017-03-31 08:04:19 EDT
The integration tests of the merged commit gives me a valid signature for the sample dmg (https://hudson.eclipse.org/cbi/view/signing-packaging/job/webservices-gerrit/ws/webservice/packaging/dmg/target/integration-tests/org.eclipse.rt.osgistarterkit.product-macosx.cocoa.x86_64.dmg)

$ codesign --verify --verbose org.eclipse.rt.osgistarterkit.product-macosx.cocoa.x86_64-2.dmg 
org.eclipse.rt.osgistarterkit.product-macosx.cocoa.x86_64-2.dmg: valid on disk
org.eclipse.rt.osgistarterkit.product-macosx.cocoa.x86_64-2.dmg: satisfies its Designated Requirement

I will deploy a new version of the webservice now.
Comment 8 Mikaël Barbero CLA 2017-03-31 08:51:25 EDT
A new version 1.1.4-SNAPSHOT of the eclipse-dmg-packager maven plugin has been deployed. It adds the "sign" parameter to the packager. The value is false by default, hence you need to specify it in your pom. This default value is kept is avoid breaking previous behavior.

The new version of the web service is now deployed. I've updated your test job (https://hudson.eclipse.org/releng/view/Releng/job/ATestForSravan/) with the <sign>true</sign> parameter. Both files are properly signed (https://hudson.eclipse.org/releng/view/Releng/job/ATestForSravan/ws/):

$ codesign --verify --verbose eclipse-platform-I20170330-2000-macosx-cocoa-x86_64.dmg && codesign --verify --verbose eclipse-SDK-I20170330-2000-macosx-cocoa-x86_64.dmg
eclipse-platform-I20170330-2000-macosx-cocoa-x86_64.dmg: valid on disk
eclipse-platform-I20170330-2000-macosx-cocoa-x86_64.dmg: satisfies its Designated Requirement
eclipse-SDK-I20170330-2000-macosx-cocoa-x86_64.dmg: valid on disk
eclipse-SDK-I20170330-2000-macosx-cocoa-x86_64.dmg: satisfies its Designated Requirement

Feel free to reopen if needed.
Comment 9 Mikaël Barbero CLA 2017-03-31 13:37:12 EDT
*** Bug 461673 has been marked as a duplicate of this bug. ***