After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 655711 - summarize depscan output by package
summarize depscan output by package
Status: RESOLVED FIXED
Product: gtk-doc
Classification: Platform
Component: general
1.17
Other Linux
: Normal enhancement
: 1.18
Assigned To: gtk-doc maintainers
gtk-doc maintainers
Depends on:
Blocks:
 
 
Reported: 2011-08-01 12:00 UTC by Yeti
Modified: 2011-08-03 12:31 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
proposed patch (2.94 KB, patch)
2011-08-01 12:00 UTC, Yeti 		committed Details | Review

Description Yeti 2011-08-01 12:00:52 UTC 
Created attachment 192978 [details] [review]
proposed patch

The current gtkdoc-depscan output is useful for figuring out where exactly the symbols are used.  It is not so useful for just seeing what packages and versions are required and why.

An implementation is attached.  For instance, running it on current evince produces:

gdk-2.24 required for
    1 gdk_x11_set_sm_client_id
gdk-pixbuf-2.6 required for
    4 gdk_pixbuf_rotate_simple
    2 gdk_pixbuf_format_is_disabled
gio-2.26 required for
    16 g_settings_get_boolean
    9 g_dbus_method_invocation_return_value
    6 g_settings_set_boolean
    4 g_dbus_proxy_call
    3 g_dbus_node_info_new_for_xml
    3 g_dbus_connection_emit_signal
    3 g_dbus_connection_register_object
    3 g_settings_new
    3 GDBusNodeInfo
    3 GDBusInterfaceVTable
    2 g_dbus_connection_call
    2 G_DBUS_ERROR
    2 g_dbus_method_invocation_return_error_literal
    2 g_dbus_connection_call_finish
    2 g_dbus_proxy_call_finish
    2 g_dbus_proxy_new_for_bus
    2 g_settings_get_int
    2 g_dbus_connection_unregister_object
    2 g_settings_get_string
    2 g_dbus_proxy_new_for_bus_finish
    2 g_settings_get_enum
    1 g_settings_set
    1 g_dbus_node_info_unref
    1 g_dbus_node_info_ref
    1 g_settings_set_string
    1 g_dbus_proxy_get_name_owner
    1 g_dbus_connection_signal_subscribe
    1 g_bus_unown_name
    1 GDBusSignalCallback
    1 g_dbus_connection_flush_sync
    1 g_settings_delay
    1 g_settings_get
    1 g_bus_own_name
    1 g_dbus_connection_signal_unsubscribe
    1 g_dbus_connection_call_sync
    1 g_bus_watch_name_on_connection
    1 g_settings_set_int
    1 g_bus_unwatch_name
    1 g_bus_get_sync
glib-2.28 required for
    1 g_list_free_full
gobject-2.26 required for
    3 g_value_get_variant
gtk-2.24 required for
    15 gtk_combo_box_text_append_text
    3 gtk_combo_box_text_new
    1 gtk_combo_box_get_has_entry
Comment 1 Stefan Sauer (gstreamer, gtkdoc dev) 2011-08-01 13:32:02 UTC 
Review of attachment 192978 [details] [review]:

Yes, great idea.

::: gtkdoc-depscan.in
@@ +381,3 @@
+            print '%s required for' % since
+            for x in summary[since]:
+                print '    %u %s' % (x[1], x[0])

what about a %4d for nicer alignment?
Comment 2 Yeti 2011-08-01 17:07:38 UTC 
(In reply to comment #1)
> what about a %4d for nicer alignment?

If you think so.  I can see reasons both for aligning it and for keeping it close to the symbol.
Comment 3 Stefan Sauer (gstreamer, gtkdoc dev) 2011-08-03 12:31:50 UTC 
Yeti, please set your username in gitconfig. Fixed it for the attached patch already.