Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

afterPack callback is not called at the same time for windows & mac. #1689

Closed
DxCx opened this issue Jun 17, 2017 · 4 comments
Closed

afterPack callback is not called at the same time for windows & mac. #1689

DxCx opened this issue Jun 17, 2017 · 4 comments
Labels

Comments

@DxCx
Copy link

DxCx commented Jun 17, 2017

  • Version: 19.4.2
  • Target: mac(dmg+zip) + windows(nsis)

Hey,
I'm using the afterPack option in order to do some modifications in the package before we are doing the actual shipping.
but it looks like the same callback is called on a different time in build pipe on windows and mac.
code signing on windows happens before afterPack and on mac after afterPack.

on windows it is called at postInitApp (https://github.com/electron-userland/electron-builder/blob/master/packages/electron-builder/src/winPackager.ts#L271)
on mac it is called right after doPack (https://github.com/electron-userland/electron-builder/blob/master/packages/electron-builder/src/macPackager.ts#L111)
please note that doPack is the one that actually calls postInitApp (https://github.com/electron-userland/electron-builder/blob/master/packages/electron-builder/src/platformPackager.ts#L194)
and then afterPack at the end of it.

IMO, sign should not be inside packing stage but right after it (just like mac)
anyway it looks wired it's not the same.

@DxCx
Copy link
Author

DxCx commented Jun 29, 2017

hey @develar any suggestions about this one?

@develar develar added the bug label Jun 29, 2017
develar added a commit to develar/electron-builder that referenced this issue Jun 29, 2017
@develar
Copy link
Member

develar commented Jun 29, 2017

Fixed. After CI run, will be pushed to upstream and released (~3 hours).

@DxCx
Copy link
Author

DxCx commented Jun 29, 2017

neat! thank you @develar !

@DxCx
Copy link
Author

DxCx commented Jun 29, 2017

Alright @develar i was able to verify it is working as intended on version 19.11.1.
thanks alot! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants