Skip to content

Commit

Permalink
Revert f432f54.
Browse files Browse the repository at this point in the history
The issue I was seeing turned out to be caused by juliaup.
  • Loading branch information
maleadt committed Dec 14, 2022
1 parent fbfe062 commit f235682
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
2 changes: 1 addition & 1 deletion Project.toml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
name = "BugReporting"
uuid = "bcf9a6e7-4020-453c-b88e-690564246bb8"
authors = ["Keno Fischer <[email protected]>", "Tim Besard <[email protected]>"]
version = "0.3.1"
version = "0.3.2"

[deps]
Downloads = "f43a241f-c20a-4ad4-852c-f6b1247861c6"
Expand Down
10 changes: 6 additions & 4 deletions src/BugReporting.jl
Original file line number Diff line number Diff line change
Expand Up @@ -422,10 +422,12 @@ function handle_child_error(p::Base.Process)

# If the child instead signalled, we recreate the same signal in ourselves
# by first disabling Julia's signal handling and then killing ourselves.
ccall(:sigaction, Cint, (Cint, Ptr{Cvoid}, Ptr{Cvoid}), p.termsignal, C_NULL, C_NULL)
ccall(:kill, Cint, (Cint, Cint), getpid(), p.termsignal)
# XXX: why doesn't raise() work?
#ccall(:raise, Cint, (Cint,), p.termsignal)
# NOTE: signal is deprecated in favor of sigaction,
# but the sigaction struct is nontrivial and platform-dependent.
ccall(:signal, Ptr{Cvoid}, (Cint, Ptr{Cvoid}), p.termsignal, C_NULL)
ccall(:raise, Cint, (Cint,), p.termsignal)
# NOTE: when switching to sigaction, we somehow need to use kill instead of raise
#ccall(:kill, Cint, (Cint, Cint), getpid(), p.termsignal)
end
end

Expand Down

0 comments on commit f235682

Please sign in to comment.