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

Upgrade sleep to 2.0 to fix yosemite compilation problem #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jimmys
Copy link

@jimmys jimmys commented Jul 27, 2015

I've upgraded to sleep 2.0 to fix the problem I was having in #15

I also noted that other users have problems with the older version of sleep on Yosemite:

erikdubbelboer/node-sleep#26

@yegor256
Copy link
Owner

@rultor try to merge

@rultor
Copy link
Collaborator

rultor commented Jul 27, 2015

@rultor try to merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor
Copy link
Collaborator

rultor commented Jul 27, 2015

@rultor try to merge

@jimmys @yegor256 Oops, I failed. You can see the full log here (spent 12s)

+ set -e
+ set -o pipefail
++ dirname /tmp/rultor-QK8Z/run.sh
+ cd /tmp/rultor-QK8Z
+ echo 25434
+ echo '1.56.2 4ed1abf'
1.56.2 4ed1abf
+ date
Mon Jul 27 17:47:47 UTC 2015
+ uptime
 17:47:47 up 6 days, 16:28,  0 users,  load average: 0.87, 3.12, 5.88
+ [email protected]:jimmys/dynamo-archive.git
+ fork_branch=upgrade-sleep-to-2.0
+ [email protected]:yegor256/dynamo-archive.git
+ head_branch=master
+ pull_id=16
+ pull_title='Upgrade sleep to 2.0 to fix yosemite compilation problem'
+ vars=('[email protected]:jimmys/dynamo-archive.git' '--env=fork_branch=upgrade-sleep-to-2.0' '[email protected]:yegor256/dynamo-archive.git' '--env=head_branch=master' '--env=pull_id=16' '--env=pull_title=Upgrade sleep to 2.0 to fix yosemite compilation problem')
+ image=yegor256/rultor
+ directory=
+ scripts=()
+ squash=false
+ ff=default
+ rebase=false
+ container=yegor256_dynamo-archive_16
+ as_root=false
+ mkdir -p /home/rultor/.ssh
+ echo -e 'Host github.com\n\tStrictHostKeyChecking no\n'
+ chmod 600 /home/rultor/.ssh/config
+ git clone [email protected]:yegor256/dynamo-archive.git repo
Cloning into 'repo'...
+ cd repo
+ git config user.email [email protected]
+ git config user.name rultor
+ '[' -z '' ']'
+ '[' -e pom.xml ']'
+ '[' -e Gemfile ']'
+ '[' -e build.xml ']'
+ [[ -e build.sbt ]]
+ [[ -e project/Build.scala ]]
+ echo 'I can'\''t guess your build automation tool, see http://doc.rultor.com/basics.html'
I can't guess your build automation tool, see http://doc.rultor.com/basics.html
+ exit -1

@davvd
Copy link
Collaborator

davvd commented Jul 28, 2015

@jimmys Thanks, let me find someone who can review this pull request

@davvd
Copy link
Collaborator

davvd commented Jul 28, 2015

@yegor256 review this pls (it's a rather short pull request)

@yegor256
Copy link
Owner

@rultor try to merge

@rultor
Copy link
Collaborator

rultor commented Oct 15, 2015

@rultor try to merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor
Copy link
Collaborator

rultor commented Oct 15, 2015

@rultor try to merge

@jimmys @yegor256 Oops, I failed. You can see the full log here (spent 1min)

npm http GET https://registry.npmjs.org/semver
npm http GET https://registry.npmjs.org/prr
npm http GET https://registry.npmjs.org/bl
npm http GET https://registry.npmjs.org/deferred-leveldown
npm http GET https://registry.npmjs.org/readable-stream/-/readable-stream-1.0.33.tgz
npm http GET https://registry.npmjs.org/xtend/-/xtend-3.0.0.tgz
npm http GET https://registry.npmjs.org/level-hooks/-/level-hooks-4.5.0.tgz
npm http 200 https://registry.npmjs.org/errno
npm http GET https://registry.npmjs.org/errno/-/errno-0.1.4.tgz
npm http 200 https://registry.npmjs.org/prr
npm http GET https://registry.npmjs.org/prr/-/prr-0.0.0.tgz
npm http 200 https://registry.npmjs.org/semver
npm http 200 https://registry.npmjs.org/bl
npm http 200 https://registry.npmjs.org/level-hooks/-/level-hooks-4.5.0.tgz
npm http 200 https://registry.npmjs.org/xtend/-/xtend-3.0.0.tgz
npm http 200 https://registry.npmjs.org/readable-stream/-/readable-stream-1.0.33.tgz
npm http 200 https://registry.npmjs.org/deferred-leveldown
npm http GET https://registry.npmjs.org/semver/-/semver-2.3.2.tgz
npm http GET https://registry.npmjs.org/bl/-/bl-0.8.2.tgz
npm http 200 https://registry.npmjs.org/errno/-/errno-0.1.4.tgz
npm http GET https://registry.npmjs.org/deferred-leveldown/-/deferred-leveldown-0.2.0.tgz
npm http 200 https://registry.npmjs.org/prr/-/prr-0.0.0.tgz
npm http 200 https://registry.npmjs.org/bl/-/bl-0.8.2.tgz
npm http 200 https://registry.npmjs.org/semver/-/semver-2.3.2.tgz
npm http 200 https://registry.npmjs.org/deferred-leveldown/-/deferred-leveldown-0.2.0.tgz
npm http GET https://registry.npmjs.org/clone
npm http GET https://registry.npmjs.org/is-object
npm WARN deprecated [email protected]: Please update to the latest object-keys
npm http GET https://registry.npmjs.org/object-keys/-/object-keys-0.2.0.tgz
npm http 200 https://registry.npmjs.org/clone
npm http 200 https://registry.npmjs.org/object-keys/-/object-keys-0.2.0.tgz
npm http 200 https://registry.npmjs.org/is-object
npm http GET https://registry.npmjs.org/clone/-/clone-0.1.19.tgz
npm http GET https://registry.npmjs.org/is-object/-/is-object-0.1.2.tgz
npm http 200 https://registry.npmjs.org/clone/-/clone-0.1.19.tgz
npm http 200 https://registry.npmjs.org/is-object/-/is-object-0.1.2.tgz
npm http GET https://registry.npmjs.org/base64-js/0.0.2
npm http GET https://registry.npmjs.org/to-utf8/0.0.1
npm http GET https://registry.npmjs.org/is
npm http GET https://registry.npmjs.org/foreach
npm http GET https://registry.npmjs.org/indexof
npm http 200 https://registry.npmjs.org/to-utf8/0.0.1
npm http 200 https://registry.npmjs.org/indexof
npm http 200 https://registry.npmjs.org/base64-js/0.0.2
npm http GET https://registry.npmjs.org/indexof/-/indexof-0.0.1.tgz
npm http GET https://registry.npmjs.org/to-utf8/-/to-utf8-0.0.1.tgz
npm http GET https://registry.npmjs.org/base64-js/-/base64-js-0.0.2.tgz
npm http 200 https://registry.npmjs.org/is
npm http 200 https://registry.npmjs.org/foreach
npm http 200 https://registry.npmjs.org/indexof/-/indexof-0.0.1.tgz
npm http 200 https://registry.npmjs.org/to-utf8/-/to-utf8-0.0.1.tgz
npm http GET https://registry.npmjs.org/is/-/is-0.2.7.tgz
npm http GET https://registry.npmjs.org/foreach/-/foreach-2.0.5.tgz
npm http 200 https://registry.npmjs.org/base64-js/-/base64-js-0.0.2.tgz
npm http GET https://registry.npmjs.org/inherits
npm http 200 https://registry.npmjs.org/is/-/is-0.2.7.tgz
npm http 200 https://registry.npmjs.org/foreach/-/foreach-2.0.5.tgz
npm http 304 https://registry.npmjs.org/inherits
npm ERR! not ok code 0

@yegor256
Copy link
Owner

@jimmys there is something wrong with sleep, it just can't be built :( any ideas?

@jimmys
Copy link
Author

jimmys commented Oct 16, 2015

Looks like the rultor auto merge tool doesn't understand this project?

@yegor256
Copy link
Owner

@jimmys that was a few months ago and it's fixed. The latest log is here: http://www.rultor.com/t/4637-148540699

@jimmys
Copy link
Author

jimmys commented Oct 23, 2015

I moved on from this software due to the installation issues, so I won't be able to help any more.

zakj added a commit to zakj/dynamo-archive that referenced this pull request Nov 4, 2015
This replaces the patch in yegor256#16, and works for me on El Capitan with nodejs 4.x+.
@zakj zakj mentioned this pull request Nov 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants