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

Add to wiki: Creating Sandstorm .spk from Wekan source #823

Closed
xet7 opened this issue Feb 8, 2017 · 3 comments
Closed

Add to wiki: Creating Sandstorm .spk from Wekan source #823

xet7 opened this issue Feb 8, 2017 · 3 comments

Comments

@xet7
Copy link
Member

xet7 commented Feb 8, 2017

Add to wiki, how to create wekan.spk from source, so it can be installed to Sandstorm instance and tested there.

I have not looked are there instructions somewhere already, so if there is, this can be closed with link to existing documentation.

From @ocdtrekkie

This should be relatively trivial to document. You pretty much just run spk pack in the source folder. (Though it might be meteor-spk pack actually, which would need to be installed, as well as Sandstorm itself.)

The important part is that a third party trying to build it will not have the required key to publish an app with the Wekan app ID. They would need to change it, which is a process documented here: https://docs.sandstorm.io/en/latest/developing/publishing-apps/#double-check-your-app-id

And they could not upgrade grains from official Wekan to a Wekan version they built themselves.

From @dwrensha

@ocdtrekkie Unfortunately it's not quite so simple.

Due to sandstorm-io/meteor-spk#25, a locally-modified version of meteor-spk v0.2.1 needs to be used (updated schemas need to be dropped in at meteor-spk.deps/node_modules/sandstorm/).

Due to the issue described here, I've been maintaining an upstream patch in Wekan that I apply before releasing.

@xet7
Copy link
Member Author

xet7 commented Mar 6, 2017

Moved to here from #790

From @ocdtrekkie

I think @dwrensha pretty much detailed everything in #823 but it may just be best for him to PR that being disabled when he packages the new version. It sounds like a lot of this will be simpler after Wekan moves to a newer Meteor version.

Ok thanks! I'll try building .spk using these instructions.

@ocdtrekkie
Copy link
Contributor

This issue can probably be closed now.

@xet7
Copy link
Member Author

xet7 commented Sep 1, 2017

@xet7 xet7 closed this as completed Sep 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants