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 689952 - Add introspection support for Windows MSVC builds
Add introspection support for Windows MSVC builds
Status: RESOLVED OBSOLETE
Product: atk
Classification: Platform
Component: build
unspecified
Other Windows
: Normal normal
: ---
Assigned To: Fan, Chun-wei
ATK maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2012-12-10 07:12 UTC by Fan, Chun-wei
Modified: 2013-02-20 10:51 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Update atk.symbols for the missed symbols (1.15 KB, patch)
2012-12-10 07:12 UTC, Fan, Chun-wei
committed Details | Review
Batch to build the introspection files for ATK (5.29 KB, patch)
2012-12-10 07:16 UTC, Fan, Chun-wei
committed Details | Review
Create a filelist to create the ATK .gir/.typelib files on Windows (3.85 KB, patch)
2013-01-18 10:49 UTC, Fan, Chun-wei
none Details | Review

Description Fan, Chun-wei 2012-12-10 07:12:22 UTC
Created attachment 231113 [details] [review]
Update atk.symbols for the missed symbols

Hi,

I have come up with some patches to enable the build of introspection support on Windows, without the need of a bash-style shell.  First up, is to update atk.symbols to make up for the items that need to be exported so that the dumper program can be properly linked.
Comment 1 Fan, Chun-wei 2012-12-10 07:16:45 UTC
Created attachment 231114 [details] [review]
Batch to build the introspection files for ATK

Next is the Windows batch (.bat) file to build and "install" the Atk-1.0.gir and Atk-1.0.typelib introspection files with an MSVC build of ATK, which does not require the use of a MSYS/Cygwin/BASH style shell on Windows.

A file list needs to be created for this to work, so I will work on this and post the updates to the autotools files for this part in the next few days.

With blessings, thank you!
Comment 2 Alejandro Piñeiro Iglesias (IRC: infapi00) 2012-12-10 09:50:36 UTC
Review of attachment 231113 [details] [review]:

Looks good to me
Comment 3 Alejandro Piñeiro Iglesias (IRC: infapi00) 2012-12-10 09:53:49 UTC
Review of attachment 231114 [details] [review]:

Well, I don't have any experience with Windows MSVC builds and I don't have any way to test it. Anyway, What I see seems to make sense, you are really experienced with this and this will not affect the most common GNOME platform (Linux) so I'm ok with this. If you think that it is ok, go on with the change.
Comment 4 Fan, Chun-wei 2012-12-13 09:06:30 UTC
Review of attachment 231113 [details] [review]:

Pushed as 5235e68e.  Sorry I forgot to include the bug number in the commit :|
Comment 5 Fan, Chun-wei 2012-12-13 09:07:47 UTC
Review of attachment 231114 [details] [review]:

Pushed as 59d01864.  Sorry I forgot to include the bug number in the commit message for this as well :|

With blessings, thank you!
Comment 6 Alejandro Piñeiro Iglesias (IRC: infapi00) 2012-12-17 18:31:24 UTC
(In reply to comment #5)
> Review of attachment 231114 [details] [review]:
> 
> Pushed as 59d01864.  Sorry I forgot to include the bug number in the commit
> message for this as well :|
> 
> With blessings, thank you!

Taking into account that the patches were pushed I'm closing the bug. Those commits will be included on the 2.7.3 release.

Thanks.
Comment 7 Fan, Chun-wei 2013-01-18 10:49:06 UTC
Created attachment 233740 [details] [review]
Create a filelist to create the ATK .gir/.typelib files on Windows 

Hi,

Sorry, this patch came much later than I would have liked.  This adds an autotools
file which is included by atk/Makefile.am to create the atk_list filelist file to be consumed by the gengir.bat, in order to generate the .gir/.typelib files for ATK for Visual C++ builds.

This also dists the gengir.bat and the generated atk_list in the tarballs.

With blessings, thank you!
Comment 8 Fan, Chun-wei 2013-02-20 10:51:37 UTC
Hi,

Bug is superseded by https://bugzilla.gnome.org/show_bug.cgi?id=694255.

Thank you, with blessings!