Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Support for unsigned builds of the MacOS app #1803

Merged
merged 3 commits into from
Jan 30, 2019

Conversation

zone117x
Copy link
Contributor

A few new build scripts for building the native MacOS app without having the $100 Apple developer certificate or needing access to the private PBC cert.

Useful for CI automated building & Blockstack.app artifact publishing as well. See commit 96c5c82 for the travisCI build config, which we can PR after this one, if wanted.

@kantai
Copy link
Collaborator

kantai commented Jan 17, 2019

I'll have to defer to @yknl on this review, as I've never done macOS builds.

@zone117x zone117x self-assigned this Jan 17, 2019
@zone117x
Copy link
Contributor Author

To help reviewers - here is a working TravisCI build config that uses a typical/stock MacOS environment:

package.json Outdated Show resolved Hide resolved
Copy link
Collaborator

@hstove hstove left a comment

Choose a reason for hiding this comment

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

Hey, it didn't work for me before, but I tried again and it worked! Very cool.

@hstove hstove mentioned this pull request Jan 24, 2019
* Add error messaging for when the browser server script processes terminate from error
@zone117x
Copy link
Contributor Author

@hstove I made a considerable change to this PR with the removal of nexe. I think you should revoke the PR approval and re-review

Copy link
Collaborator

@hstove hstove left a comment

Choose a reason for hiding this comment

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

Good updates. I built the app successfully.

I'm not sure how to test the CORS proxy that is built. Any help there?

@yknl
Copy link
Collaborator

yknl commented Jan 28, 2019

The production build is now working. I tested the CORS proxy with curl:

curl localhost:1337/www.blockstack.org

@kantai kantai removed their request for review January 28, 2019 18:25
@yknl
Copy link
Collaborator

yknl commented Jan 28, 2019

If I remember correctly, the CORS proxy works a bit differently in the windows build. We'll have to test that out as well.

@hstove
Copy link
Collaborator

hstove commented Jan 30, 2019

If I remember correctly, the CORS proxy works a bit differently in the windows build. We'll have to test that out as well.

Doesn't this not touch the Windows build?

@zone117x zone117x merged commit b3dd8a9 into develop Jan 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants