search

julianbuettner / swapvec

A Rust Vector which swaps to disk based on given parameters
MIT License
44 stars 4 forks source link

Fix conditional OS imports #5

Closed Almeida-a closed 1 year ago

Almeida-a commented 1 year ago

AsRawFd was triggering an error when compiling the project in a linux environment.

This seems to fix #2 .

Almeida-a commented 1 year ago

Important note: I have not tested this in windows/wsl.

julianbuettner commented 1 year ago

Thank you!

rmg-x commented 1 year ago

@julianbuettner @Almeida-a The build is breaking on Windows again.

The additional import for non-Unix targets is the cause and is unnecessary since we set file descriptors to None for those targets regardless. Also I'm not sure there is even a mechanism for displaying file descriptors on those platforms anyway.

I can't open a PR right now to fix it.