{"action":"created","issue":{"url":"https://api.github.com/repos/pywbem/pywbem/issues/7","labels_url":"https://api.github.com/repos/pywbem/pywbem/issues/7/labels{/name}","comments_url":"https://api.github.com/repos/pywbem/pywbem/issues/7/comments","events_url":"https://api.github.com/repos/pywbem/pywbem/issues/7/events","html_url":"https://github.com/pywbem/pywbem/issues/7","id":127107243,"number":7,"title":"Pywbem makefile/pylint does not honor files marked to ignore","user":{"login":"KSchopmeyer","id":2340817,"avatar_url":"https://avatars.githubusercontent.com/u/2340817?v=3","gravatar_id":"","url":"https://api.github.com/users/KSchopmeyer","html_url":"https://github.com/KSchopmeyer","followers_url":"https://api.github.com/users/KSchopmeyer/followers","following_url":"https://api.github.com/users/KSchopmeyer/following{/other_user}","gists_url":"https://api.github.com/users/KSchopmeyer/gists{/gist_id}","starred_url":"https://api.github.com/users/KSchopmeyer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/KSchopmeyer/subscriptions","organizations_url":"https://api.github.com/users/KSchopmeyer/orgs","repos_url":"https://api.github.com/users/KSchopmeyer/repos","events_url":"https://api.github.com/users/KSchopmeyer/events{/privacy}","received_events_url":"https://api.github.com/users/KSchopmeyer/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-01-17T16:50:38Z","updated_at":"2016-01-17T18:24:02Z","closed_at":null,"body":"The makefile check target uses pylint to run lint tests on the python code and puts the results in a log file.\r\n\r\nThe statement to execute this lint defines two files to be ignored with the --ignore option. However,\r\nexecuting make check causes pylint to lint at least the mofparsetab file to be checked which is an\r\nissue in part because almost not of the file will pass the lint tests so it generates an enormous number\r\nof lines in the log file.  Since this is a file generated by the yacc parser itself, there should be not\r\nreason to run lint against it.  The syntax of the pylint options seems correct but both of the\r\nfiles defined get processed.  Adding the # pylint: skip-file at the head of the file does cause the\r\nfile to be to be ignored in check (A message in the log indicates that pylint looked at the file and ignored it).  Since this also happens when pylint is manually executed (ex. pylint --ignore=mofparsetab.py mof*.py  it would appear that this might be an issue in pylint self.  We tried\r\n\r\npylint --rcfile=../pylint.rc --ignore=moflextab.py mof*.py\r\npylint --rcfile=../pylint.rc --ignore=mof*.py mof*.py\r\npylint --rcfile=../pylint.rc --ignore=moflextab mof*.py\r\n\r\nNone of which appear to cause the file to be ignored in the check.\r\n\r\nNote that we do not want to include these files in the pylint check since they are generated files and\r\nwe do not want to try fixing lint problems in them.\r\npylint.log: $(pylint_rc_file) setup.py os_setup.py $(package_name)/*.py testsuite/*.py\r\n\t-sh -c \"PYTHONPATH=. pylint --rcfile=$(pylint_rc_file) --ignore=moflextab.py,mofparsetab.py --output-format=text setup.py os_setup.py $(package_name) testsuite/*.py |tee pylint.log\"\r\n"},"comment":{"url":"https://api.github.com/repos/pywbem/pywbem/issues/comments/172360952","html_url":"https://github.com/pywbem/pywbem/issues/7#issuecomment-172360952","issue_url":"https://api.github.com/repos/pywbem/pywbem/issues/7","id":172360952,"user":{"login":"KSchopmeyer","id":2340817,"avatar_url":"https://avatars.githubusercontent.com/u/2340817?v=3","gravatar_id":"","url":"https://api.github.com/users/KSchopmeyer","html_url":"https://github.com/KSchopmeyer","followers_url":"https://api.github.com/users/KSchopmeyer/followers","following_url":"https://api.github.com/users/KSchopmeyer/following{/other_user}","gists_url":"https://api.github.com/users/KSchopmeyer/gists{/gist_id}","starred_url":"https://api.github.com/users/KSchopmeyer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/KSchopmeyer/subscriptions","organizations_url":"https://api.github.com/users/KSchopmeyer/orgs","repos_url":"https://api.github.com/users/KSchopmeyer/repos","events_url":"https://api.github.com/users/KSchopmeyer/events{/privacy}","received_events_url":"https://api.github.com/users/KSchopmeyer/received_events","type":"User","site_admin":false},"created_at":"2016-01-17T18:24:02Z","updated_at":"2016-01-17T18:24:02Z","body":"It turns out that apparently, while the command line option does not honor the file list, the same option in the configuration file does so that the statement:\r\n\r\nignore moflextab.py,mofparsetab.py\r\n\r\nin the file pylint.rc does cause those files to be ignored by pylint."}}