Skip to content

Fix constructing TCPSocket from RawFD#43617

Merged
JeffBezanson merged 1 commit intoJuliaLang:masterfrom
wentasah:fix-tcpsocket-from-rawfd
Jan 5, 2022
Merged

Fix constructing TCPSocket from RawFD#43617
JeffBezanson merged 1 commit intoJuliaLang:masterfrom
wentasah:fix-tcpsocket-from-rawfd

Conversation

@wentasah
Copy link
Copy Markdown
Contributor

Without this, an attempt to call this function ends with:

ERROR: LoadError: UndefVarError: pipe not defined
Stacktrace:
 [1] TCPSocket(fd::RawFD)
   @ Sockets /nix/store/w0xq12x4gb3ng0pdh1r9ir4z14pmr3pp-julia-bin-1.7.0/share/julia/stdlib/v1.7/Sockets/src/Sockets.jl:100
 [2] top-level scope

Also add a test for this function. It designed only for Linux, because
I don't know whether it would work on other platforms.

Without this, an attempt to call this function ends with:

ERROR: LoadError: UndefVarError: pipe not defined
Stacktrace:
 [1] TCPSocket(fd::RawFD)
   @ Sockets /nix/store/w0xq12x4gb3ng0pdh1r9ir4z14pmr3pp-julia-bin-1.7.0/share/julia/stdlib/v1.7/Sockets/src/Sockets.jl:100
 [2] top-level scope

Also add a test for this function. It designed only for Linux, because
I don't know whether it would work on other platforms.
@JeffBezanson JeffBezanson merged commit f958438 into JuliaLang:master Jan 5, 2022
@JeffBezanson JeffBezanson added bugfix This change fixes an existing bug io Involving the I/O subsystem: libuv, read, write, etc. labels Jan 5, 2022
@wentasah wentasah deleted the fix-tcpsocket-from-rawfd branch January 27, 2022 09:06
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Feb 22, 2022
Without this, an attempt to call this function ends with:

ERROR: LoadError: UndefVarError: pipe not defined
Stacktrace:
 [1] TCPSocket(fd::RawFD)
   @ Sockets /nix/store/w0xq12x4gb3ng0pdh1r9ir4z14pmr3pp-julia-bin-1.7.0/share/julia/stdlib/v1.7/Sockets/src/Sockets.jl:100
 [2] top-level scope

Also add a test for this function. It designed only for Linux, because
I don't know whether it would work on other platforms.
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Mar 8, 2022
Without this, an attempt to call this function ends with:

ERROR: LoadError: UndefVarError: pipe not defined
Stacktrace:
 [1] TCPSocket(fd::RawFD)
   @ Sockets /nix/store/w0xq12x4gb3ng0pdh1r9ir4z14pmr3pp-julia-bin-1.7.0/share/julia/stdlib/v1.7/Sockets/src/Sockets.jl:100
 [2] top-level scope

Also add a test for this function. It designed only for Linux, because
I don't know whether it would work on other platforms.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix This change fixes an existing bug io Involving the I/O subsystem: libuv, read, write, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants