Michael Keck comments

Results 343 comments of


                                            Michael Keck

Add logging level for conan-center hook

What is the current status of this?

[conan centre] conandata.yml extended format for patches

This sounds really good 👍

Prefer tools.is_apple_os instead of settings.os check

I think the hooks have no way to post recommendations yet and I think that there are valid cases to only check of macOS but not iOS and the rest....

Integrate spdx check into conan-center hook

FYI, I have started working on it a few days ago, but because of holidays and everything it will rest now for a few days before I continue

Integrate spdx check into conan-center hook

As part of the work I will also collect a list of allowed exceptions based on the CCI recipes (because not every used license out there has a SPDX entry)...

Please include package name "cern-root" in allow list for #KB-H050: "DEFAULT SHARED OPTION VALUE"

I would rather see a `shared` attribute with the default and only possible value `True` instead 🤔

Please include package name "cern-root" in allow list for #KB-H050: "DEFAULT SHARED OPTION VALUE"

If this is possible it might be a good hook improvement, until then I would rather have it more explicit in the recipe + I don't think it is a...

[conan center] add clarification why CMake 3.1 is required

> There is an issue in Conan asking about the general cmake version used by companies, all people responded 3+. I remember the issue, but can't find it right now....

[conan center] add clarification why CMake 3.1 is required

Yes, thanks!

[conan-center] The regular os.rename should be replaced by tools.rename

That is another case where I wish that hooks were capable of outputting warnings, which the CCI bot is then posting directly in its build result comment to make them...