On 02/28/2014 02:13 PM, Brian C. Lane wrote:
pylint-1.1.0 appears to have 'better' introspection, and generates more E1101 errors on various Gtk and NM items. Filter them out.
These changes do not cause any problems with pylint-1.0.0
I'd like three more in this list related to pylint trying to run a lot more of the gobject-introspection based code. I'm getting a bunch of "Unable to init server" messages when running on a VM without X, so that means Jenkins might, too, and I'm getting a two-line hash table assert message in the network spoke while pylint tries to make sense of NMClient.ActiveConnection.
^Unable to init server$ gi/module.py:[0-9]*: Warning: g_hash_table_insert_internal: assertion 'hash_table != NULL' failed$ ^ g_type = info.get_g_type()$
(two spaces in the start of the g_type line). I'd also like to add GI_TYPELIB_PATH and LD_LIBRARY_PATH to testenv.sh so it can find the locally-built copy of AnacondaWidgets, but that can be a separate patch.
These patches look fine, and with the extra false positives and a patched astroid the pylint errors with 1.1 are down to the ones in accordion that are probably worth a closer look.
tests/pylint/pylint-false-positives | 73 +++++++++++++++++++++++++++++++++++++ 1 file changed, 73 insertions(+)
diff --git a/tests/pylint/pylint-false-positives b/tests/pylint/pylint-false-positives index e06680a..7a11f4b 100644 --- a/tests/pylint/pylint-false-positives +++ b/tests/pylint/pylint-false-positives @@ -7,5 +7,78 @@ ^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_property' member$ ^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_padding' member$ ^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'show_all' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'show' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'get_object' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_translation_domain' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'add_objects_from_file' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'add_from_file' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'add_window' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_decorated' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_default_response' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'run' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'destroy' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'load_from_path' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'load_from_data' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'attach' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_label_widget' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_markup' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_halign' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_margin_left' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_margin_right' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_margin_bottom' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'props' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_row_spacing' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_column_spacing' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'get_children' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_has_tooltip' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'append_text' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_active' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_hexpand' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'message' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'get_from_server' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_variants' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'activate' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_layouts' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_options' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_sort_column_id' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_expand' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_use_underline' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_mnemonic_widget' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_mode' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_value' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_min_value' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_max_value' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'add_offset_value' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_chosen' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Instance of '.*' has no 'set_title' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'get_default_root_window' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'pixbuf_get_from_window' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'init_check' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'main_level' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'main' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'source_remove' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'spawn_close_pid' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'spawn_async_with_pipes' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'dbus_address_get_for_bus_sync' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'x11_get_default_xdisplay' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'bus_get_sync' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'G_MAXUINT' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'get_locale_direction' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'parse_markup' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'init' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Module '.*' has no 'bind' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Class '.*' has no 'get_default' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Class '.*' has no 'add_provider_for_screen' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Class '.*' has no 'new_with_default_fallbacks' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Class '.*' has no 'new_for_address_sync' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Class '.*' has no 'new_for_bus_sync' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Class '.*' has no 'new_from_file' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Class '.*' has no 'new_for_xml' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Class '.*' has no 'new' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Class '.*' has no 'get_instance' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Class '.*' has no 'dialog_new' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Class '.*' has no 'dialog_set_layout' member$ +^E1101:[ 0-9]*,[0-9]*:.*: Class '.*' has no 'set_default_direction' member$ ^E0611:[ 0-9]*,[0-9]*: No name '_isys' in module 'pyanaconda'$ ^E0611:[ 0-9]*,[0-9]*:.*: No name '_isys' in module 'pyanaconda'$ +^E0712:[ 0-9]*,[0-9]*:.*: Catching an exception which doesn't inherit from BaseException: GError$