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

Change package entry for electron-builder. #165

Merged
merged 1 commit into from
Jun 12, 2016

Conversation

razzeee
Copy link
Contributor

@razzeee razzeee commented Jun 11, 2016

As #138 (comment) has been fixed, we can reference the head version again.

@yuya-oc
Copy link
Contributor

yuya-oc commented Jun 12, 2016

Probably electron-builder has breaking changes for arguments. Please check.

@razzeee razzeee force-pushed the electron-builder branch 3 times, most recently from 3ad2386 to fc40305 Compare June 12, 2016 09:22
@razzeee
Copy link
Contributor Author

razzeee commented Jun 12, 2016

https://github.com/electron-userland/electron-builder/releases/tag/v4.0.0 seems to be the (first) reason.
Not (yet) sure what exactly breaks it.

@razzeee razzeee force-pushed the electron-builder branch from fc40305 to 02c2fc1 Compare June 12, 2016 09:35
@razzeee
Copy link
Contributor Author

razzeee commented Jun 12, 2016

It's down to https://github.com/mattermost/desktop/blob/master/package.json#L26

build --platform linux --arch all -d deb

-d seems to now be -t but for some reason platform and arch also don't work

@razzeee razzeee force-pushed the electron-builder branch from 02c2fc1 to 87da29d Compare June 12, 2016 11:53
@razzeee
Copy link
Contributor Author

razzeee commented Jun 12, 2016

Circle CI is green again

@yuya-oc
Copy link
Contributor

yuya-oc commented Jun 12, 2016

Thanks!

@yuya-oc yuya-oc merged commit 1c6ea92 into mattermost:master Jun 12, 2016
@razzeee razzeee deleted the electron-builder branch June 12, 2016 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants