Using cmake extension modules alongside other extension modules

[Xiretza]

Unless I'm misinterpreting something, using BuildExtension as cmdclass['build_ext'] in setup.py makes any non-cmake extension modules completely ineffective, because BuildExtension.run() simply ignores them:

cmake-build-extension/src/cmake_build_extension/build_extension.py

Line 82 in 2c3d822

cmake_extensions = [e for e in self.extensions if isinstance(e, CMakeExtension)]

Is my understanding correct? How would I use this project if I also have e.g. cython extension modules?

[diegoferigo]

Hi @Xiretza, thanks for raising this point. I'm not really sure to be honest, I'm not expert in setuptools hooks and I naively assumed that each build extension should process only its own stuff. I imagine that also other build extensions implement their own BuildExtension.run method, and I assume that they also filter their instance but I might be wrong.

What's the behavior you're experiencing? Are you sure that the line you linked prevents the execution of other build extensions? Can you check on your example by adding some print statements in all run methods of the extensions you're using?

[Xiretza]

This project overrides the global build_ext:

cmake-build-extension/example/setup.py

Line 106 in 2c3d822

build_ext=cmake_build_extension.BuildExtension,

As far as I can tell there can only be one build_ext, and if it filters out all modules it isn't specifically designed for, that's just not gonna work.

[diegoferigo]

Ok now I see what you mean, do you have any suggestion on how to work around this limitation?

[Xiretza]

Not really. BuildExtension.run() should probably call super().run() at some point, but that will of course try to build and install the CMakeExtensions as well.

[kdewald]

@diegoferigo you could in theory skip most of the preprocessing that is currently being done in the run function and check instance types inside build_extension and forwarding to the correct build_ext object.

[yishai1999]

Any news on this? I just encountered this problem