Generate vapi from gir instead of gi.

Generate vapi from gir instead of gi.


BUG=none
TEST=manually on Fedora 16

[Peng, 2012-01-31 04:39:09]

On 2012/01/31 04:29:03, Daiki Ueno wrote:

https://github.com/phuang/ibus/tree/devel

i am developing ibus 1.5 in my personal branch. it uses vala. could you please
testyour changewith my branc？

[Daiki Ueno, 2012-01-31 05:02:42]

On 2012/01/31 04:39:09, Peng wrote:
> On 2012/01/31 04:29:03, Daiki Ueno wrote:
> 
> https://github.com/phuang/ibus/tree/devel
> 
> i am developing ibus 1.5 in my personal branch. it uses vala. could you please
> testyour changewith my branc？

Sure, and I actually did that for testing :)
Here are relevant patches rebased against your devel branch:

http://ueno.fedorapeople.org/ibus-vapi-gir/

- 0001-Minor-fixes-when-generating-GIR.patch
Same as https://codereview.appspot.com/5600052/
- 0002-Generate-vapi-from-gir-instead-of-gi.patch
Same as this patchset
- 0003-Additional-fixes.patch
Some fixes for latest valac

I did:
$ git clone git://github.com/phuang/ibus.git ibus-phuang
$ git checkout -b devel origin/devel
$ patch ... # apply all 3 patches
$ CFLAGS="-Wall -g -O0" ./autogen.sh --prefix=/usr --libdir=/usr/lib64
--enable-gtk3 --disable-silent-rules
$ make

[Peng, 2012-01-31 15:25:02]

On 2012/01/31 05:02:42, Daiki Ueno wrote:
> On 2012/01/31 04:39:09, Peng wrote:
> > On 2012/01/31 04:29:03, Daiki Ueno wrote:
> > 
> > https://github.com/phuang/ibus/tree/devel
> > 
> > i am developing ibus 1.5 in my personal branch. it uses vala. could you
please
> > testyour changewith my branc？
> 
> Sure, and I actually did that for testing :)
> Here are relevant patches rebased against your devel branch:
> 
> http://ueno.fedorapeople.org/ibus-vapi-gir/
> 
> - 0001-Minor-fixes-when-generating-GIR.patch
> Same as https://codereview.appspot.com/5600052/
> - 0002-Generate-vapi-from-gir-instead-of-gi.patch
> Same as this patchset
> - 0003-Additional-fixes.patch
> Some fixes for latest valac
> 
> I did:
> $ git clone git://github.com/phuang/ibus.git ibus-phuang
> $ git checkout -b devel origin/devel
> $ patch ... # apply all 3 patches
> $ CFLAGS="-Wall -g -O0" ./autogen.sh --prefix=/usr --libdir=/usr/lib64
> --enable-gtk3 --disable-silent-rules
> $ make

Thanks. LGTM.

[Peng, 2012-01-31 15:33:40]

On 2012/01/31 15:25:02, Peng wrote:
> On 2012/01/31 05:02:42, Daiki Ueno wrote:
> > On 2012/01/31 04:39:09, Peng wrote:
> > > On 2012/01/31 04:29:03, Daiki Ueno wrote:
> > > 
> > > https://github.com/phuang/ibus/tree/devel
> > > 
> > > i am developing ibus 1.5 in my personal branch. it uses vala. could you
> please
> > > testyour changewith my branc？
> > 
> > Sure, and I actually did that for testing :)
> > Here are relevant patches rebased against your devel branch:
> > 
> > http://ueno.fedorapeople.org/ibus-vapi-gir/
> > 
> > - 0001-Minor-fixes-when-generating-GIR.patch
> > Same as https://codereview.appspot.com/5600052/
> > - 0002-Generate-vapi-from-gir-instead-of-gi.patch
> > Same as this patchset
> > - 0003-Additional-fixes.patch
> > Some fixes for latest valac
> > 
> > I did:
> > $ git clone git://github.com/phuang/ibus.git ibus-phuang
> > $ git checkout -b devel origin/devel
> > $ patch ... # apply all 3 patches
> > $ CFLAGS="-Wall -g -O0" ./autogen.sh --prefix=/usr --libdir=/usr/lib64
> > --enable-gtk3 --disable-silent-rules
> > $ make
> 
> Thanks. LGTM.

One question. Is it better to rename IBus-1.0* to ibus-1.0* ? Do you know what
the standard file names for them?

[Peng, 2012-01-31 15:37:43]

http://codereview.appspot.com/5599052/diff/14/bindings/vala/Makefile.am
File bindings/vala/Makefile.am (right):

http://codereview.appspot.com/5599052/diff/14/bindings/vala/Makefile.am#newco...
bindings/vala/Makefile.am:29: # don't rebuild vapi every time gir is updated
Don't rebuild vapi every time, if gir is updated.

[Daiki Ueno, 2012-02-01 00:57:43]

On 2012/01/31 15:33:40, Peng wrote:
> On 2012/01/31 15:25:02, Peng wrote:
> > On 2012/01/31 05:02:42, Daiki Ueno wrote:
> > > On 2012/01/31 04:39:09, Peng wrote:
> > > > On 2012/01/31 04:29:03, Daiki Ueno wrote:
> > > > 
> > > > https://github.com/phuang/ibus/tree/devel
> > > > 
> > > > i am developing ibus 1.5 in my personal branch. it uses vala. could you
> > please
> > > > testyour changewith my branc？
> > > 
> > > Sure, and I actually did that for testing :)
> > > Here are relevant patches rebased against your devel branch:
> > > 
> > > http://ueno.fedorapeople.org/ibus-vapi-gir/
> > > 
> > > - 0001-Minor-fixes-when-generating-GIR.patch
> > > Same as https://codereview.appspot.com/5600052/
> > > - 0002-Generate-vapi-from-gir-instead-of-gi.patch
> > > Same as this patchset
> > > - 0003-Additional-fixes.patch
> > > Some fixes for latest valac
> > > 
> > > I did:
> > > $ git clone git://github.com/phuang/ibus.git ibus-phuang
> > > $ git checkout -b devel origin/devel
> > > $ patch ... # apply all 3 patches
> > > $ CFLAGS="-Wall -g -O0" ./autogen.sh --prefix=/usr --libdir=/usr/lib64
> > > --enable-gtk3 --disable-silent-rules
> > > $ make
> > 
> > Thanks. LGTM.
> 
> One question. Is it better to rename IBus-1.0* to ibus-1.0* ? Do you know what
> the standard file names for them?

The file name seems to be used to determine NAMESPACE and VERSION:
https://live.gnome.org/GObjectIntrospection/AutotoolsIntegration
(see the last bullet list)

Though they are explicitly set in the GIR, I think it would be safe to keep
using IBus-1.0*.

[Daiki Ueno, 2012-02-01 00:58:52]

https://codereview.appspot.com/5599052/diff/14/bindings/vala/Makefile.am
File bindings/vala/Makefile.am (right):

https://codereview.appspot.com/5599052/diff/14/bindings/vala/Makefile.am#newc...
bindings/vala/Makefile.am:29: # don't rebuild vapi every time gir is updated
On 2012/01/31 15:37:44, Peng wrote:
> Don't rebuild vapi every time, if gir is updated.

Done.