-
-
Notifications
You must be signed in to change notification settings - Fork 708
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
Outdated docs for hacking on sandstorm shell #3047
Comments
ok nice... the steps to get that _icons.scss:
|
Problem is much bigger than that. Before running make or any other steps from Option 4: Installing from Source, do these (all commands are for Ubuntu 16.04):
Add |
Right now i cannot make
update: here is relevant bug from llvm bugzilla. it seems closed. update 2: looks it can be a CDN problem. another run and |
I made Debian 9 VirtualBox Appliance where Wekan and Sandstorm compile successfully: There is Wekan-Sandstorm.ova. |
Ok, now the sandstorm is not compiling. Not that I think it's really necessary. I just thought that maybe, if you want to hack on shell, you need to get it make-able first. I will try to continue.
update update 2 i've updated gcc to 7.2.0:
now we have different error. |
When I try to run
update the error is meaningless. It should be smth with this part of
see that |
@xet7 i downloaded
File size is 4398810112 bytes, sha1 hash ba1f92b2577a32bce7ee4a42b696a0a7f7191599 made with command
|
What a nice catch-22.
Do you see that? |
I think i will give up for today. It's already evening in moscow. Short summary of the day:
|
Your VirtualBox version 5.1.20 r114629 (Qt5.6.2) for Mac OS X is really old. That VM has been done yesterday with newest VirtualBox 5.2.6, you can You can also check SHA256SUMS.txt that your download is not corrupted: For me downloading that VM, importing to VirtualBox 5.2.6 and starting it works fine. Current install Sandstorm from source instructions are at: My previous issue about compiling sandstorm is #3036 , dependencies are currently these. On Debian 9 clang-6.0 is at Debian package repos. Currently I have these capnproto packages installed, you may need some of those maybe, if those are not installed with with that sudo apt line.
Also because Debian did not have sudo installed, I installed it:
Then continue installing:
For that VirtualBox VM, I did install Sandstorm normally, without compiling it from source, from https://sandstorm.io/install . I did install it in dev mode. I did also compile sandstorm from source on that VM. To prevent Sandstorm from auto-updating, dev channel needs to be commented out at /opt/sandstorm/sandstorm.conf . Then install compiled sandstorm version, more info at: npm for Debian is possible to install from here: @kentonv uses Debian for developing Sandstorm. |
Thnx @xet7 , upgrading VirtualBox helped. My bad. |
Anyway, the problem with hacking the shell is the same:
Full log
|
After i made |
Hi @dmitry-sher, I'm also facing |
@urakagi I gave up and used pre-built VM. Read first comment: #3047 (comment) . BTW, what OS are you using? I have seen a difference between my initial VM and xet7's one: I had Ubuntu 16.03 with gcc 7.2.0. xet7's: Debian 9 with gcc 7.3.0 |
I'm using Ubuntu 16.04. I was able to build up hmm. |
Hi all, I eliminated the Clang dependency because it trips people up. The build now downloads Clang binaries directly from the Chromium project, so you don't have to install it from apt.llvm.org or whatever. See commit 4463c3f. I've verified that with this change, the build works correctly in a fresh Debian Stretch VM. |
Very nice, thank you. Separately, I have a list of Sandstorm dependencies other than Clang for Fedora 27. Do you think it's worth adding that to docs/install.md underneath the Debian/Ubuntu section? |
@Michael-S It's probably going to be useful to someone, probably wouldn't hurt to have it there, and it may end up helping people with other distros too. |
@Michael-S Yes, that would be great. |
Okay, done with #3060 |
I think I can close this, as fixes were made, and the PR that Michael made after this was also merged. |
UPDATE
Easiest way to hack on sandstorm shell is as follows:
ssh -p 4222 wekan@localhost
. You can add your keysssh-copy-id -p 4222 wekan@localhost
/opt/sandstorm/sandstorm.conf
in VM. setBIND_IP=0.0.0.0
. comment out#UPDATE_CHANNEL=dev
. restart sandstormsudo service sandstorm restart
. Now you can access sandstorm from host OS on port 6080sudo usermod -a -G sandstorm wekan
. logout and login back for changes to take place.~/repos/sandstorm
andmake update
sandstorm dev-shell
from the repo root.Below are my previous attempts to do it on Ubuntu 16.04.3. You can safely skip it.
Are not working anymore. Current command is
sandstorm dev-shell
in project root.If you run it, you would see a multitude of problems.
1a. Preparations to run hacking.
sandstorm dev-shell
. From the latest master commit 760faa2As you can see, the file is missing in the sources:
https://github.com/sandstorm-io/sandstorm/tree/master/shell/client/styles.
It's generated. See how to generate it below.
You would miss capnp.node and capnp.js. You can find them in
/opt/sandstorm/latest/node_modules
. Copy everything from that folder toshell/node_modules
.There are some missing
capnp
files, likepayments.capnp
. You can find all current capnp files in project insrc/sandstorm/
. Copy them toshell/node_modules/sandstorm
.The text was updated successfully, but these errors were encountered: