Skip to content

Fix for OSX spawn test#22788

Merged
quinnj merged 1 commit into
masterfrom
jq/osxspawn
Jul 14, 2017
Merged

Fix for OSX spawn test#22788
quinnj merged 1 commit into
masterfrom
jq/osxspawn

Conversation

@quinnj
Copy link
Copy Markdown
Member

@quinnj quinnj commented Jul 13, 2017

Ran into this several times while running the tests about a million times so far for #22441. I don't think this is related to #17626, but one could always hope, right?

@ararslan ararslan added system:mac Affects only macOS test This change adds or pertains to unit tests labels Jul 13, 2017
@iblislin
Copy link
Copy Markdown
Member

iblislin commented Jul 13, 2017

Not sure it's related or not, I found a way to make success(`true`) insane.

  1. borrow a MBA from my classmate :p
  2. test.jl
using Base.Test

for i in 1:100000  # just a large number
  println(i)
  @test success(`true`)
end

yes, we need @test to trigger crashing/freezing. no idea why single success works fine.
3. julia test.jl
4. hold your Crtl+t, keep sending SIGINFO
5. sometimes you got segmentation fault; sometimes the program freezed, only kill -9 can stop it.

@vtjnash
Copy link
Copy Markdown
Member

vtjnash commented Jul 13, 2017

@iblis17 looks unrelated. I ran your code in my debugger and found the issue is that Apple's libunwind does not conform to its specification (its unw_init_local allocates memory, but is documented to not allocate memory)

@iblislin
Copy link
Copy Markdown
Member

@vtjnash :-o file it as another issue?

@vtjnash
Copy link
Copy Markdown
Member

vtjnash commented Jul 13, 2017

Should be a radar, but I'm not very active about getting those submitted.

@quinnj quinnj merged commit f8ff488 into master Jul 14, 2017
@quinnj quinnj deleted the jq/osxspawn branch July 14, 2017 03:00
jeffwong pushed a commit to jeffwong/julia that referenced this pull request Jul 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

system:mac Affects only macOS test This change adds or pertains to unit tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants