CoffeaTeam / fsspec-xrootd

An XRootD implementation for fsspec
https://coffeateam.github.io/fsspec-xrootd/
BSD 3-Clause "New" or "Revised" License
3 stars 5 forks source link

Improve error if port is not available for XRootD #53

Closed chrisburr closed 8 months ago

chrisburr commented 8 months ago

While working on #52 I ran into https://github.com/xrootd/xrootd/pull/2208 which caused the xrootd server to be left open after the first test run. This resulted in some very weird errors.

In another project I've used this function to give a clearer error message if the port is already taken. Feel free to close this PR if you'd prefer not to add this.

nsmith- commented 8 months ago

Looks like @lobis was going to have the test choose a port in https://github.com/CoffeaTeam/fsspec-xrootd/pull/37/files#diff-e52e4ddd58b7ef887ab03c04116e676f6280b824ab7469d5d3080e5cba4f2128