[GitHub] [thrift] tokcum commented on pull request #2539: THRIFT-3877: add support for Unix Domain Sockets in lib/rs

[GitBox <gi...@apache.org>]

tokcum commented on pull request #2539:
URL: https://github.com/apache/thrift/pull/2539#issuecomment-1059836481


   I reviewed the CI results. The build failures are not related to this PR, I think. Meanwhile I also cross compiled lib/rs to Windows with the following cargo targets:
   
   - i686-pc-windows-gnu
   - i686-pc-windows-msvc
   - x86_64-pc-windows-gnu
   - x86_64-pc-windows-msvc
   
   This releaved that I should have also tagged the use statement for Path.
   
   ```
   warning: unused import: `std::path::Path`
     --> src/server/threaded.rs:21:5
      |
   21 | use std::path::Path;
      |     ^^^^^^^^^^^^^^^
      |
      = note: `#[warn(unused_imports)]` on by default
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@thrift.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org