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

Migrate to MacOS Catalina #1585

Merged
merged 8 commits into from
Nov 15, 2019
Merged

Migrate to MacOS Catalina #1585

merged 8 commits into from
Nov 15, 2019

Conversation

barmac
Copy link
Collaborator

@barmac barmac commented Nov 13, 2019

Closes #1529
Closes #1572
Closes #1584

@bpmn-io-tasks bpmn-io-tasks bot added the in progress Currently worked on label Nov 13, 2019
@barmac

This comment has been minimized.

@barmac barmac force-pushed the 1572-catalina-fix branch 5 times, most recently from f9a6a15 to 3a50148 Compare November 14, 2019 15:24
@barmac
Copy link
Collaborator Author

barmac commented Nov 14, 2019

The actual publishing jobs are available here: https://travis-ci.org/camunda/camunda-modeler/builds/611909768

Tested on both MacOS Mojave (10.14.6) and MacOS Catalina (10.15.1).

Notarized build is already available as nightly. Non-release builds will not be notarized.

@barmac barmac requested a review from nikku November 14, 2019 15:29
@barmac barmac marked this pull request as ready for review November 14, 2019 15:31
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed in progress Currently worked on labels Nov 14, 2019
@pinussilvestrus
Copy link
Contributor

npm run build and starting the App work absolutely fine on my Mac! (Version 10.15.1)

As of `electron-builder@20.38.2`, the built zip files has been incorrect.
On the other hand, older versions of `electron-builder` are unable
to build application on MacOS 10.15 Catalina.

Due to above, this commit removes `zip` from MacOS targets and adds
a script that adds the built application to a correct zip archive.

Related electron-userland/electron-builder#3534

Closes #1529
@nikku
Copy link
Member

nikku commented Nov 15, 2019

Works on Arch Linux, too.

@nikku
Copy link
Member

nikku commented Nov 15, 2019

@philippfromme Can you quick check if the distribution still works on Windows, too?

@philippfromme
Copy link
Contributor

Works on Windows 👏

@nikku nikku merged commit b4704f4 into develop Nov 15, 2019
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Nov 15, 2019
@delete-merged-branch delete-merged-branch bot deleted the 1572-catalina-fix branch November 15, 2019 09:51
@nikku
Copy link
Member

nikku commented Nov 15, 2019

Good work @barmac 🚲.

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

Successfully merging this pull request may close these issues.

Migrate to MacOS Catalina Mac OSX v3.4.1 can not be started Can't build on MacOS 10.15 Catalina
4 participants