James

Results 2692 comments of James

It depends a bit on the feedback and stabilization of alpha and beta. We are just into alpha1 and expect to do a couple of alphas more, then move to...

https://github.com/conan-io/conan-vs-extension/issues/201

The platform toolset is already defined in the provided .props file by ``MSBuildToolchain``, isn't it working? We have some tests that should cover this.

This is what I mean, there are tests that have the PlatformToolset defined, like https://github.com/conan-io/conan/blob/8aeea2230fd6375ce1da15f2380097bed9098377/conans/test/functional/toolchains/microsoft/test_msbuild.py#L98 in the project file, and apparently that value is correctly overriden by the ``conan_toolchain.props`` file,...

Yes, it is necessary to inject the .props files, for third-party vcxproj, I guess it is necessary to apply a patch > Nop, same issues with vs_layout(). Yes, layout is...

I did a try in https://github.com/conan-io/conan/pull/12046/commits/baa6b8fd71bb429ca978dfb26a8aed12b705eb13, it seemed to work, but we decided to simplify first while rolling the changes in ``test_package``

This feature does not address the case of some recipes (``protobuf`` and similar), that need to be built both as regular "host" requires and as "build" tool_requires. So probably not...

Hi @Nekto89 Have you tried operating directly on the ``options``, something like: ```python def package_id(self): del self.info.options["boost"].without_python self.info.requires["boost"].package_id = None ``` (not sure if del works on that, if not...

Yes, it would be equivalent, but if you don't want options to affect, you need to drop it, because otherwise it is going to affect your package_id, even if you...

This doesn't seem a Conan issue, but a CMake issue, as ``-O3`` is the CMake default for ``Release``. Maybe this should be addressed directly in your ``CMakeLists.txt``, and not injected...