You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I am trying to use your template for a project of mine.
Unfortunately, I am not sure I understand the definition of the package maintainer ( myproject_PACKAGING_MAINTAINER_MODE)
I the myproject_PACKAGING_MAINTAINER_MODE considered the build for production and the other is build in development ?
Why is WARNING_AS_ERRORS OFF by default ?
Why is ENAME_SANITIZER_** set within the if<->else ? Could this not just be defined in front of the if ?
Why is myproject_ENABLE_IPO be disabled on Package Maintainer mode ? Would't it make more sende to disable it in NON-PACKAGE_MAINTAINER mode ?
I would be really happy if someone could untangle my confusion a bit :-)
The text was updated successfully, but these errors were encountered:
db-tech
changed the title
What is the package maintainer
What is the package maintainer ?
Aug 23, 2023
From my understanding the packaging maintainer mode should only be used by the developers of this package. I.e. we only want to enable additional checks for the developers.
I believe that there is a bug in this line: ProjectOptions.cmake
The condition should be NOT PROJECT_IS_TOP_LEVEL OR NOT myproject_PACKAGING_MAINTAINER_MODE. (Notice the second NOT.)
Hi,
I am trying to use your template for a project of mine.
Unfortunately, I am not sure I understand the definition of the package maintainer ( myproject_PACKAGING_MAINTAINER_MODE)
I the myproject_PACKAGING_MAINTAINER_MODE considered the build for production and the other is build in development ?
Why is WARNING_AS_ERRORS OFF by default ?
Why is ENAME_SANITIZER_** set within the if<->else ? Could this not just be defined in front of the if ?
Why is myproject_ENABLE_IPO be disabled on Package Maintainer mode ? Would't it make more sende to disable it in NON-PACKAGE_MAINTAINER mode ?
I would be really happy if someone could untangle my confusion a bit :-)
The text was updated successfully, but these errors were encountered: