search

pevers / gdcm-rs

GDCM bindings for Rust
7 stars 6 forks source link

Windows support #13

Closed ykszk closed 8 months ago

ykszk commented 8 months ago

Adding windows support with the following changes:

pevers commented 8 months ago

Thanks for your work! This looks good to me. On a side note, maybe it would be good to transfer the repository to @Enet4 because I'm very busy at the moment with little time at my hands to maintain.

What do you think @Enet4 ?

Enet4 commented 8 months ago

@pevers I can contribute to the maintenance of gdcm-rs, whether through repository transfer or just by being added as a collaborator and crate owner. My primary focus will continue to be on DICOM-rs and giving it first-class support for various transfer syntaxes without depending on GDCM, but for as long as there is value in this integration, it is in my best interest to keep it working. :)

ykszk commented 8 months ago

Thank you for the reviews! I updated build flags for cleaner builds.

pevers commented 8 months ago

@pevers I can contribute to the maintenance of gdcm-rs, whether through repository transfer or just by being added as a collaborator and crate owner. My primary focus will continue to be on DICOM-rs and giving it first-class support for various transfer syntaxes without depending on GDCM, but for as long as there is value in this integration, it is in my best interest to keep it working. :)

Yes understandable! I'll add you as a collaborator and crate owner.