Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Add mac installer files #2995

Merged
merged 8 commits into from Nov 2, 2016
Merged

Add mac installer files #2995

merged 8 commits into from Nov 2, 2016

Conversation

gavofyork
Copy link
Contributor

@gavofyork gavofyork commented Oct 30, 2016

Would be nice to allow the GUI to configure the wake-up parameters for the backend, too. e.g. to hibernate it (syncing off), to move to dark (syncing on only when UI open), passive (syncing turned on regularly) or active (syncing on). See #3002 for this.

@General-Beck It would also be good to have this building from within the CI - will need to use packagesbuild as described in the Packages docs.

@gavofyork gavofyork added A0-pleasereview 🤓 Pull request needs code review. M0-build 🏗 Building and build system. labels Oct 30, 2016
@coveralls
Copy link

Coverage Status

Coverage increased (+0.003%) to 86.037% when pulling 613cd81 on mac-installer into 6da02e0 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 86.047% when pulling 613cd81 on mac-installer into 6da02e0 on master.

<true/>
<key>StandardErrorPath</key>
<string>/var/log/parity.log</string>
</dict>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we also add --warp and --sync to homebrew plist?
https://github.com/ethcore/homebrew-ethcore/blob/master/parity.rb#L33

<key>GID</key>
<integer>80</integer>
<key>PATH</key>
<string>/Users/gav/Core/parity/mac/Parity Ethereum.webloc</string>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems to be machine specific. A few other places like this below

@coveralls
Copy link

Coverage Status

Coverage increased (+0.003%) to 86.037% when pulling f5f72f9 on mac-installer into 6da02e0 on master.

@gavofyork gavofyork changed the title Add mac installer files. Add mac installer files Oct 30, 2016
@coveralls
Copy link

Coverage Status

Coverage increased (+0.003%) to 86.037% when pulling 785a2f6 on mac-installer into 6da02e0 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.034% when pulling 785a2f6 on mac-installer into 6da02e0 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.034% when pulling 785a2f6 on mac-installer into 6da02e0 on master.

@gavofyork gavofyork mentioned this pull request Oct 31, 2016
@gavofyork
Copy link
Contributor Author

@General-Beck would also be good to populate the mac/Parity Ethereum.app/Contents/Info.plist file immediately prior to building the package. The version can some from parity --version.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 9b32f91 on mac-installer into * on master*.

@gavofyork
Copy link
Contributor Author

gitlab build issue fixed in #3014

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 86.024% when pulling 0afb83c on mac-installer into 8599a11 on master.

@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Nov 2, 2016
@gavofyork gavofyork merged commit 9b1f67b into master Nov 2, 2016
@gavofyork gavofyork deleted the mac-installer branch November 2, 2016 16:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M0-build 🏗 Building and build system.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants