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 612480 - Improvements to GObject Introspection support
Improvements to GObject Introspection support
Status: RESOLVED FIXED
Product: libgda
Classification: Other
Component: Client library
4.1.x
Other Linux
: Normal normal
: ---
Assigned To: malerba
gnome-db Maintainers
Depends on: 612022
Blocks:
 
 
Reported: 2010-03-10 20:29 UTC by Daniel Espinosa
Modified: 2011-04-18 19:13 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Improvements to GI support on GDA (5.57 KB, patch)
2010-03-11 01:47 UTC, Daniel Espinosa
none Details | Review

Description Daniel Espinosa 2010-03-10 20:29:12 UTC
Exists some typo on libgda/Makefile.am, witch made not to include all *.c
files, this will exclude any GOI annotations and some details are hiden like
implemented interfaces and so on.

But in order to fix this GOI must fix bug 612022.

Some missing typedef for GError codes must be added or fixed its enum's name.
This could be done easy on the way, including add annotations on *.c files for
improve GOI support.
Comment 1 Daniel Espinosa 2010-03-10 20:33:18 UTC
Missed GError codes has been pushed to master.

Now sources files to be scanned must be selected and include as external c code (not to be scanned) in order to allow correct typelibs generation.
Comment 2 Daniel Espinosa 2010-03-11 01:47:39 UTC
Created attachment 155816 [details] [review]
Improvements to GI support on GDA

This patch include the C files required to generate GIR file, keeps out the ones used for internal porpoises.
Comment 3 Murray Cumming 2010-06-14 11:35:34 UTC
Is this still relevant?

Maybe Daniel should just go ahead?
Comment 4 malerba 2010-06-14 13:12:02 UTC
The GObject introspection building has been re-written and is now IMO correct, si this patch won't apply anymore.
Daniel can you check I haven't forgotten anything?
Comment 5 Daniel Espinosa 2011-04-18 19:13:25 UTC
Confirmed. Closing bug report.