Skip to content

Conversation

yosuke-furukawa
Copy link
Member

related #1675

@brendanashworth brendanashworth added test Issues and PRs related to the tests. dgram Issues and PRs related to the dgram subsystem / UDP. labels May 12, 2015
@yosuke-furukawa
Copy link
Member Author

jenkins ci: https://jenkins-iojs.nodesource.com/job/iojs+any-pr+multi/663/

I don't have a permission to run jenkins ci. How do I get the permission??

@bnoordhuis
Copy link
Member

@rvagg Can you give @yosuke-furukawa access?

@yosuke-furukawa When you have an account, go here, fill in the 'user' and 'branch' fields ('yosuke-furukawa' and 'hotfix/test-dgram-send-callback-recursive' in this case), leave the other fields at their defaults, click 'Build' and go get a cup of coffee - you'll have plenty of time to finish it.

@yosuke-furukawa yosuke-furukawa changed the title test: loosen infinite detection condition test: loosen infinite loop detect condition May 12, 2015
@Fishrock123
Copy link
Contributor

@Fishrock123
Copy link
Contributor

CI seems happier. The windows failures are unrelated afaik, I'll try to root them out today.

Fishrock123 pushed a commit that referenced this pull request May 14, 2015
Fixes: #1675
PR-URL: #1681
Reviewed-By: Jeremiah Senkpiel <[email protected]>
@Fishrock123
Copy link
Contributor

Landed in 4e2f999 thanks!

Fishrock123 pushed a commit to Fishrock123/node that referenced this pull request May 19, 2015
Fixes: nodejs#1675
PR-URL: nodejs#1681
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dgram Issues and PRs related to the dgram subsystem / UDP. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants