Skip to content

Changed react/socket to 0.8#49

Closed
skaffille wants to merge 3 commits intofriends-of-reactphp:masterfrom
skaffille:skaffille/react-socket-0.8
Closed

Changed react/socket to 0.8#49
skaffille wants to merge 3 commits intofriends-of-reactphp:masterfrom
skaffille:skaffille/react-socket-0.8

Conversation

@skaffille
Copy link
Copy Markdown

Updated dependencies to other libraries to allow usage of react/socket 0.8.11
Changed streams and usage of timer.
Adapted unit tests.

@WyriHaximus WyriHaximus requested review from WyriHaximus, clue and jsor June 7, 2018 19:07
@skaffille skaffille force-pushed the skaffille/react-socket-0.8 branch from 255ca57 to 7bb1482 Compare June 30, 2018 09:43
@skaffille
Copy link
Copy Markdown
Author

I will fix the PHP 5.6 issues as soon as possible.
Regarding the activemq issues ... there are some installation problems during the build for all PHP versions

Reading package lists... Done
Building dependency tree       
Reading state information... Done
E: Unable to locate package tree
cp: cannot create regular file ‘/etc/activemq/instances-available/main/’: No such file or directory
ln: failed to create symbolic link ‘/etc/activemq/instances-enabled/main’: No such file or directory
activemq: unrecognized service

@skaffille skaffille force-pushed the skaffille/react-socket-0.8 branch 2 times, most recently from 98833f6 to 1444852 Compare July 2, 2018 16:02
Changed streams and usage of timer.
Adapted unit tests.
@skaffille skaffille force-pushed the skaffille/react-socket-0.8 branch from 1444852 to 19400d0 Compare July 2, 2018 16:16
@skaffille skaffille changed the title Changed react/socket to 0.8.11 Changed react/socket to 0.8 Jul 2, 2018
@skaffille
Copy link
Copy Markdown
Author

Build works for php5.6 now, as well.
activemq still failing...

@brettmc
Copy link
Copy Markdown

brettmc commented Jul 10, 2018

@skaffille I came across those failures with #48 and made some changes to .travis.yml which fixed them. I think if you pull in the second and third of my commits from PR 48, it'll work for you.

brettmc added 2 commits July 11, 2018 18:41
tree dependency is not found in trusty - two options to fix: 'apt update' before installing tree, or don't install it. I chose the latter because it does not look like it is being used
I was able to somewhat replicate the activemq install issue with the ubuntu:trusty docker image, and an 'apt update' resolved it
@skaffille
Copy link
Copy Markdown
Author

@brettmc Thank you, very much. Now the build completes successfully!

adrianspacely pushed a commit to adrianspacely/stomp that referenced this pull request Feb 11, 2019
@skaffille skaffille closed this Oct 18, 2019
@skaffille skaffille mentioned this pull request Oct 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants