Comments (20)
We are able to build the app again. For details see fec8024
Is there anything left we need to clean up or can we close this issue @xet7 ?
from wordpress-sandstorm.
I am super excited that progress has been made here!
from wordpress-sandstorm.
From @kentonv on February 11, 2018 6:18
It appears as if Sandstorm isn't installed inside the Vagrant image, which is weird. Maybe there's some more initialization that needs to be done by vagrant-spk? TBH I have never used vagrant-spk so I don't really know. Maybe @dwrensha (who created the Wordpress package) has an idea.
from wordpress-sandstorm.
From @dwrensha on February 11, 2018 14:3
Yeah I probably forgot to include the Sandstorm capnp files in the vagrant-spk build: dwrensha/wordpress-sandstorm@17d3971
EDIT: hm. Those sandstorm files really should be there. When I do vagrant-spk up
and vagrant-spk ssh
, I see them in /opt/sandstorm/
from wordpress-sandstorm.
From @dwrensha on February 11, 2018 14:31
When I try to build https://github.com/dwrensha/wordpress-sandstorm, it looks like I get farther than you did, but I still hit an error:
(cd sandstorm && make CXX=g++)
make[1]: Entering directory '/opt/app/sandstorm'
mkdir -p tmp
capnp compile --src-prefix=/opt/sandstorm/latest/usr/include -oc++:tmp /opt/sandstorm/latest/usr/include/sandstorm/*.capnp
*** Uncaught exception ***
kj/filesystem-disk-unix.c++:469: failed: mmap: Invalid argument
stack: 0x405b42 0x43a40e 0x43af2c 0x7f7d19cb38c5 0x7f7d19cb52c6
c++: plugin failed: exit code 1
Makefile:21: recipe for target 'tmp/genfiles' failed
I conjecture that this has to do with the fact that the /vagrant
directory is synced between the host and guest.
from wordpress-sandstorm.
From @kentonv on February 11, 2018 21:33
Ugh, apparently shared folders don't implement writable mmap(). I'm surprised they can get away with that. I guess maybe the capnp C++ output plugin needs to go back to using write(). Sad.
from wordpress-sandstorm.
This is the PR they used to fix their Vagrant 1.9.6 downloading the sandstorm version of the box: github.com/sandstorm-io/vagrant-spk/pull/209
Got that from @ocdtrekkie . I did change previously to probably broken box version.
from wordpress-sandstorm.
From @JamborJan on March 6, 2018 19:38
Hi guys,
I have updated my fork of wordpress for Sandstorm (see https://github.com/JamborJan/wordpress-sandstorm/commit/cb93151f8b0901b6db1eb85aec7dbc28e230a4ef). Now I'm running into a different error:
(cd sandstorm && make CXX=g++)
make[1]: Entering directory '/opt/app/sandstorm'
generating capnp files...
/opt/sandstorm/latest/usr/include/sandstorm/payments.capnp:21:14-37: error: Import failed: /sandstorm/util.capnp
/opt/sandstorm/latest/usr/include/sandstorm/payments.capnp:28:29-42: error: Not defined: LocalizedText
/opt/sandstorm/latest/usr/include/sandstorm/payments.capnp:65:25-38: error: Not defined: LocalizedText
/opt/sandstorm/latest/usr/include/sandstorm/payments.capnp:73:17-30: error: Not defined: LocalizedText
/opt/sandstorm/latest/usr/include/sandstorm/payments.capnp:22:14-43: error: Import failed: /sandstorm/supervisor.capnp
/opt/sandstorm/latest/usr/include/sandstorm/payments.capnp:87:58-74: error: Not defined: SystemPersistent
/opt/sandstorm/latest/usr/include/sandstorm/payments.capnp:88:62-78: error: Not defined: SystemPersistent
/opt/sandstorm/latest/usr/include/sandstorm/payments.capnp:89:46-62: error: Not defined: SystemPersistent
Makefile:21: recipe for target 'tmp/genfiles' failed
make[1]: *** [tmp/genfiles] Error 1
make[1]: Leaving directory '/opt/app/sandstorm'
Makefile:26: recipe for target 'sandstorm/bin/getPublicId' failed
make: *** [sandstorm/bin/getPublicId] Error 2
I was not able yet to dig a bit deeper into these error messages but will do so as soon as possible. I wanted to leave a note here just in case someone of you can easily help or solve this.
PS: @xet7 as soon as we have figured out how we deal with the app key I will contribute to your repo. Until then I use my fork for testing reasons. Otherwise I would not be able to build packages and test them.
Thanks
from wordpress-sandstorm.
From @ocdtrekkie on March 6, 2018 19:44
@JamborJan payments.capnp is a weird thing to show up here. I know that @kentonv just merged that back into Sandstorm proper, so it's possible something's missing there, and it's never been an issue with building before for that reason: sandstorm-io/sandstorm#3042
from wordpress-sandstorm.
No problem, a moment ago I did fetch and merge changes from your repo to this repo, and also add some more modifications for php5 to php7. I will look at app key when bulding wp-sandstorm works, and there is something worth releasing.
from wordpress-sandstorm.
From @JamborJan on March 7, 2018 8:3
You also took my sandstorm-pkgdef.capnp. I can send you my private app key (over another secure channel) so that you are able tu build packages. If this is okay for you, I would be able to delete my fork and use yours instead.
from wordpress-sandstorm.
Yes it's OK. You can install desktop or mobile client of https://about.riot.im so we can enable end-to-end encryption, my Riot nickname is xet7wekan
from wordpress-sandstorm.
Hey @ocdtrekkie, did I get your comment right and the payments.capnp will be analysed or is there something I need to do because I did something wrong when I tried to build the Wordpress app?
from wordpress-sandstorm.
I am just suspecting it's a new problem cause by the fact that payments.capnp is new to the Sandstorm repo. Kenton should be around this weekend and can probably explain why it's showing up as an error.
from wordpress-sandstorm.
It looks like there's a bug in the makefile:
capnp compile --src-prefix=/opt/sandstorm/latest/usr/include -oc++:tmp /opt/sandstorm/latest/usr/include/sandstorm/*.capnp
This line is missing an argument:
-I/opt/sandstorm/latest/usr/include
It fails on payments.capnp because that file contains absolute imports rather than relative, which is vestigial since it used to live in Blackrock rather than Sandstorm. I will change it to use relative imports in today's release, but it may also make sense to fix the Wordpress app's Makefile.
from wordpress-sandstorm.
With latest changes in git I have this progress, after installing vagrant:
git clone [email protected]:wpsandstorm/wordpress-sandstorm.git
cd wordpress-sandstorm
./build-spk-in-vagrant.sh
Then inside VirtualBox VM window, where is username: vagrant password: vagrant
sudo apt install task-lxde-desktop
startx
Then on Firefox ESR http://local.sandstorm.io:6080
from wordpress-sandstorm.
Related sandstorm-io/sandstorm#3054
from wordpress-sandstorm.
I guess it's about the new PHP7 version according to the error log. I tried to run and enter a grain to figure out what is wrong but I ran into another issue (see sandstorm-io/vagrant-spk#213).
I've realised that there are some php config files in the repo. There is one line which seems to be wrong for PHP7, but it seems these config files are not used at all.
@xet7 I guess you added this symlink to solve the issue, which didn't work out. Right?
...** SANDSTORM SUPERVISOR: Starting up grain. Sandbox type: userns
[12-Mar-2018 06:48:59] NOTICE: [pool www] 'user' directive is ignored when FPM is not running as root
[12-Mar-2018 06:48:59] NOTICE: [pool www] 'group' directive is ignored when FPM is not running as root
[12-Mar-2018 06:48:59] ERROR: unable to bind listening socket for address '/run/php/php7.0-fpm.sock': No such file or directory (2)
[12-Mar-2018 06:48:59] ERROR: FPM initialization failed
started php-fpm. status code: 70
started nginx. status code: 0
from wordpress-sandstorm.
Yes, it did not work, it caused error about symlink to symlink, or something like that.
from wordpress-sandstorm.
I removed that symlink from repo.
from wordpress-sandstorm.
Related Issues (20)
- 404s are not nice formatted HOT 1
- Pages which are not referenced in menus are missing (404s)
- Installing certain Plugins causes Error 500 HOT 4
- Metadata updates HOT 2
- Build Process HOT 10
- Screenshots should be updated HOT 1
- Sometimes the app will leave the sandstorm frame HOT 3
- New Gutenberg editor not working in Sandstorm HOT 2
- Add Finnish FI translation HOT 2
- loss of new posts HOT 4
- Media loading doesn't work in AOSP HOT 4
- 'Restore this revision' button is inactive at the revisions comparisment screen.
- 'bulk actions' doesn't work
- licensing server is unable to connect with my wp site HOT 5
- Why is this so out of date? HOT 1
- RSS Feed doesn't contain valid links
- WordPress testing official SQLite Support. Useful? HOT 2
- Release 17 tends to hang on powerbox-http-proxy setup HOT 3
- cURL error 60: SSL certificate problem: self signed certificate in certificate chain HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from wordpress-sandstorm.