Fix a few ccache cache miss (#2573)

* Fix ccache cache miss

* Do not define _VERSION_ in python bindings
This commit is contained in:
Cheng
2025-09-09 07:41:05 +09:00
committed by GitHub
parent e5a33f2223
commit a44b27f5f8
5 changed files with 7 additions and 9 deletions

View File

@@ -1,10 +1,8 @@
// Copyright © 2025 Apple Inc.
#include <string>
namespace mlx::core {
std::string version() {
const char* version() {
return MLX_VERSION;
}

View File

@@ -15,6 +15,6 @@ namespace mlx::core {
*
* For dev builds, the version will include the suffix ".devYYYYMMDD+hash"
*/
std::string version();
const char* version();
} // namespace mlx::core

View File

@@ -52,7 +52,6 @@ set_target_properties(
${MLX_PYTHON_BINDINGS_OUTPUT_DIRECTORY})
target_link_libraries(core PRIVATE mlx)
target_compile_definitions(core PRIVATE _VERSION_=${MLX_VERSION})
if(BUILD_SHARED_LIBS)
if(${CMAKE_SYSTEM_NAME} MATCHES "Darwin")

View File

@@ -2,9 +2,9 @@
#include <nanobind/nanobind.h>
#define STRINGIFY(x) #x
#define TOSTRING(x) STRINGIFY(x)
#include "mlx/version.h"
namespace mx = mlx::core;
namespace nb = nanobind;
void init_mlx_func(nb::module_&);
@@ -48,5 +48,5 @@ NB_MODULE(core, m) {
init_distributed(m);
init_export(m);
m.attr("__version__") = TOSTRING(_VERSION_);
m.attr("__version__") = mx::version();
}

View File

@@ -121,7 +121,8 @@ class CMakeBuild(build_ext):
build_args += [f"-j{os.cpu_count()}"]
# Avoid cache miss when building from temporary dirs.
os.environ["CCACHE_BASEDIR"] = os.path.abspath(self.build_temp)
os.environ["CCACHE_BASEDIR"] = os.path.realpath(self.build_temp)
os.environ["CCACHE_NOHASHDIR"] = "true"
subprocess.run(
["cmake", ext.sourcedir, *cmake_args], cwd=build_temp, check=True