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 745022 - Drop deprecated API
Drop deprecated API
Status: RESOLVED FIXED
Product: gnome-dictionary
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-dictionary-maint
gnome-dictionary-maint
Depends on:
Blocks:
 
 
Reported: 2015-02-23 15:12 UTC by Emmanuele Bassi (:ebassi)
Modified: 2015-02-23 15:35 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
build: Do not use INCLUDES (1.11 KB, patch)
2015-02-23 15:13 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
build: Do not use INCLUDES (1.34 KB, patch)
2015-02-23 15:13 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
build: Modernize autotools usage (8.92 KB, patch)
2015-02-23 15:13 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
gdict: Fix introspection annotations (2.16 KB, patch)
2015-02-23 15:13 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
gdict: Remove GTK_STOCK_* usage (6.07 KB, patch)
2015-02-23 15:14 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
app: Remove use of deprecated stock items (5.57 KB, patch)
2015-02-23 15:14 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
window: Do not use deprecate composite-child API (1.86 KB, patch)
2015-02-23 15:14 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
gdict: Silence a compiler warning (806 bytes, patch)
2015-02-23 15:14 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
gdict: Drop deprecated API from DatabaseChooser (4.49 KB, patch)
2015-02-23 15:15 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
gdict: Drop deprecated API from SourceChooser (5.50 KB, patch)
2015-02-23 15:15 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
gdict: Drop deprecated API from Speller (4.94 KB, patch)
2015-02-23 15:15 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
gdict: Drop deprecated API from StrategyChooser (5.16 KB, patch)
2015-02-23 15:15 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
gdict: Drop deprecated API from DefBox (10.21 KB, patch)
2015-02-23 15:16 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
app: Drop all deprecated API (4.74 KB, patch)
2015-02-23 15:16 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
Bump the libgdict soname (1.07 KB, patch)
2015-02-23 15:16 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
docs: Fix up the documentation build (1.54 KB, patch)
2015-02-23 15:16 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review
Bump up the GTK+ requirement (666 bytes, patch)
2015-02-23 15:19 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review

Description Emmanuele Bassi (:ebassi) 2015-02-23 15:12:45 UTC
Ensure that the build does not suck too badly.

While technically the changes inside libgdict amount to an ABI break, there are mitigating factors:

 • nothing is subclassing these types, as far as we know
 • GtkVBox has the same size of GtkBox
 • GtkVBox has no API, except the type function and a constructor
Comment 1 Emmanuele Bassi (:ebassi) 2015-02-23 15:13:00 UTC
Created attachment 297655 [details] [review]
build: Do not use INCLUDES

It's deprecated, and we can replace it with a target specific CPPFLAGS
stanza.
Comment 2 Emmanuele Bassi (:ebassi) 2015-02-23 15:13:15 UTC
Created attachment 297656 [details] [review]
build: Do not use INCLUDES

It's deprecated. We should be using AM_CPPFLAGS, in this case, since we
share the same pre-processor flags with the introspection scanner.
Comment 3 Emmanuele Bassi (:ebassi) 2015-02-23 15:13:30 UTC
Created attachment 297657 [details] [review]
build: Modernize autotools usage

While not really bleeding edge, at least we have something adequate for
this century.
Comment 4 Emmanuele Bassi (:ebassi) 2015-02-23 15:13:45 UTC
Created attachment 297658 [details] [review]
gdict: Fix introspection annotations

This brings down the number of g-ir-scanner warnings to zero.
Comment 5 Emmanuele Bassi (:ebassi) 2015-02-23 15:14:00 UTC
Created attachment 297659 [details] [review]
gdict: Remove GTK_STOCK_* usage

We're still allowing it from users of the API, but internally we don't
use images from stock ids.
Comment 6 Emmanuele Bassi (:ebassi) 2015-02-23 15:14:15 UTC
Created attachment 297660 [details] [review]
app: Remove use of deprecated stock items

We want proper labels, and we want to drop deprecation warnings.
Comment 7 Emmanuele Bassi (:ebassi) 2015-02-23 15:14:30 UTC
Created attachment 297661 [details] [review]
window: Do not use deprecate composite-child API

It's useless anyway.
Comment 8 Emmanuele Bassi (:ebassi) 2015-02-23 15:14:45 UTC
Created attachment 297662 [details] [review]
gdict: Silence a compiler warning

NULL is not FALSE.
Comment 9 Emmanuele Bassi (:ebassi) 2015-02-23 15:15:00 UTC
Created attachment 297663 [details] [review]
gdict: Drop deprecated API from DatabaseChooser

Remove all the deprecation warnings.

*** WARNING ***

This is really an API change, since GtkBox is not a GtkVBox; on the
other hand:

 • nothing is really subclassing this type
 • GtkVBox does not have any API except a constructor
Comment 10 Emmanuele Bassi (:ebassi) 2015-02-23 15:15:16 UTC
Created attachment 297664 [details] [review]
gdict: Drop deprecated API from SourceChooser

Remove all the deprecation warnings.

*** WARNING ***

This is really an API change, since GtkBox is not a GtkVBox; on the
other hand:

 • nothing is really subclassing this type
 • GtkVBox does not have any API except a constructor
Comment 11 Emmanuele Bassi (:ebassi) 2015-02-23 15:15:32 UTC
Created attachment 297666 [details] [review]
gdict: Drop deprecated API from Speller

Remove all the deprecation warnings.

*** WARNING ***

This is really an API change, since GtkBox is not a GtkVBox; on the
other hand:

 • nothing is really subclassing this type
 • GtkVBox does not have any API except a constructor
Comment 12 Emmanuele Bassi (:ebassi) 2015-02-23 15:15:48 UTC
Created attachment 297667 [details] [review]
gdict: Drop deprecated API from StrategyChooser

Remove all the deprecation warnings.

*** WARNING ***

This is really an API change, since GtkBox is not a GtkVBox; on the
other hand:

 • nothing is really subclassing this type
 • GtkVBox does not have any API except a constructor
Comment 13 Emmanuele Bassi (:ebassi) 2015-02-23 15:16:04 UTC
Created attachment 297668 [details] [review]
gdict: Drop deprecated API from DefBox

Remove all the deprecation warnings.

*** WARNING ***

This is really an API change, since GtkBox is not a GtkVBox; on the
other hand:

 • nothing is really subclassing this type
 • GtkVBox does not have any API except a constructor
Comment 14 Emmanuele Bassi (:ebassi) 2015-02-23 15:16:19 UTC
Created attachment 297669 [details] [review]
app: Drop all deprecated API
Comment 15 Emmanuele Bassi (:ebassi) 2015-02-23 15:16:36 UTC
Created attachment 297670 [details] [review]
Bump the libgdict soname

We changed parts of the ABI.
Comment 16 Emmanuele Bassi (:ebassi) 2015-02-23 15:16:51 UTC
Created attachment 297671 [details] [review]
docs: Fix up the documentation build
Comment 17 Emmanuele Bassi (:ebassi) 2015-02-23 15:19:50 UTC
Created attachment 297676 [details] [review]
Bump up the GTK+ requirement
Comment 18 Emmanuele Bassi (:ebassi) 2015-02-23 15:33:12 UTC
Attachment 297655 [details] pushed as 04094b0 - build: Do not use INCLUDES
Attachment 297656 [details] pushed as 04094b0 - build: Do not use INCLUDES
Attachment 297657 [details] pushed as e5eb618 - build: Modernize autotools usage
Attachment 297658 [details] pushed as d864ca5 - gdict: Fix introspection annotations
Attachment 297659 [details] pushed as 66e8e50 - gdict: Remove GTK_STOCK_* usage
Attachment 297660 [details] pushed as 496b111 - app: Remove use of deprecated stock items
Attachment 297661 [details] pushed as f353608 - window: Do not use deprecate composite-child API
Attachment 297662 [details] pushed as 466408b - gdict: Silence a compiler warning
Attachment 297663 [details] pushed as 2ab2182 - gdict: Drop deprecated API from DatabaseChooser
Attachment 297664 [details] pushed as d078eaf - gdict: Drop deprecated API from SourceChooser
Attachment 297666 [details] pushed as f9c484d - gdict: Drop deprecated API from Speller
Attachment 297667 [details] pushed as 2cd2500 - gdict: Drop deprecated API from StrategyChooser
Attachment 297668 [details] pushed as 533b6d6 - gdict: Drop deprecated API from DefBox
Attachment 297669 [details] pushed as c60d8c8 - app: Drop all deprecated API
Attachment 297670 [details] pushed as fa69aa0 - Bump the libgdict soname
Attachment 297671 [details] pushed as 69eae8d - docs: Fix up the documentation build
Attachment 297676 [details] pushed as 0cbca4c - Bump up the GTK+ requirement