search

cern-fts / davix

High-performance file management over WebDAV / HTTP
GNU Lesser General Public License v2.1
83 stars 34 forks source link

Update CMake FindPackage Python to allow Python3 on macOS #92

Closed Axel-Naumann closed 2 years ago

Axel-Naumann commented 2 years ago

Current macOS does not come with python 2 anymore. CMake should be updated to use the new FindPackage Python, similar to what's done here: https://github.com/dpiparo/vdt/pull/12/files - that will happily find Python3.

This blocks building ROOT on current macOS systems, version 12.3.

mpatrascoiu commented 2 years ago

Hello Axel,

Thanks for bringing this up. Commit 96fbb96 should fix it. It will be released in davix-0.8.1.

Cheers, Mihai

Axel-Naumann commented 2 years ago

Excellent, thank you, Mihai! Do you have an ETA for 0.8.1?

mpatrascoiu commented 2 years ago

I knew this question would show up.

I want to also merge #83 , but that change is also tied to gfal2 v2.21.0 upcoming release, which we don't expect until a month or so.

Axel-Naumann commented 2 years ago

Thanks. The current Davix release does not build anymore on macOS 12; this PR unblocks that. We need to release a patch release of ROOT 6.26 to address this (among other things). I'd very much prefer to use a Davix tag, rather than using the previous Davix release plus this issue's commit. If it's reasonably simple to create a release (e.g. for ROOT it's quite involved...) could I propose to have 0.8.1 released "now" and the gfal PR in a subsequent release - also given its dependence on another library and its schedule? FYI @gganis

mpatrascoiu commented 2 years ago

Hello Axel,

Thanks for letting me know on the ROOT plans. Sure, there is a way to move forward and I think we can provide davix 0.8.1 within this week.

Cheers, Mihai

Axel-Naumann commented 2 years ago

Wow that would be fantastic - thank you so much!

mpatrascoiu commented 2 years ago

Hello Axel,

Davix 0.8.1 has been released. Please find the RPMs in the official DMC repository.

Cheers, Mihai