Skip to content
Snippets Groups Projects

AnalysisBase: Add dummy flake8 configuration

Merged Frank Winklmeier requested to merge fwinkl/athena:anabase_flake8 into master
2 files
+ 12
0
Compare changes
  • Side-by-side
  • Inline
Files
2
  • 8181c17d
    `flake8` is not available in AnalysisBase. While our cmake configuration
    does handle this correctly, it fails once developers start using special
    flake8 options, e.g.:
    ```
    ... POST_BUILD_CMD ${ATLAS_FLAKE8} --enable-extensions ATL902
    ```
    Even if `ATLAS_FLAKE8` is not defined it will now try to run a command
    of name `--enable-extensions`, which of course fails. An easy workaround
    is to `set( ATLAS_FLAKE8 true )` where `true` here is the shell builtin,
    which returns success and swallows any command line args given to it.
@@ -4,6 +4,12 @@
# @CMAKE_PROJECT_NAME@.
#
# Set up the project's flake8 usage.
set( ATLAS_FLAKE8 @ATLAS_FLAKE8@
CACHE STRING "Default flake8 command" )
set( ATLAS_PYTHON_CHECKER @ATLAS_PYTHON_CHECKER@
CACHE STRING "Python checker command to run during Python module compilation" )
# Figure out whether to use QUIET in the find_package call(s).
set( _quietFlag )
if( @CMAKE_PROJECT_NAME@_FIND_QUIETLY )
Loading