[patch] fix gecko-embed pkgconfig

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[patch] fix gecko-embed pkgconfig

Robert Staudinger
Hi,

the attached patch does away with gecko-embed's hardcoded .pc file and
adds an autogen'd one instead. So it becomes usable from a c
application.

The included sample can be built with
gcc `pkg-config --cflags --libs gtk+-2.0 gecko-embed` -lxul -o
gecko-embed-test gecko-embed-test.c
(there seems to be a bug in xulrunner, -lxul is not mentioned in any
of the installed .pc files)

Ok to commit?

- Rob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gecko-embed-pc.diff
Type: text/x-patch
Size: 1995 bytes
Desc: not available
Url : http://mailman.laptop.org/pipermail/sugar/attachments/20060726/60ca5135/gecko-embed-pc.bin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gecko-embed-test.c
Type: text/x-csrc
Size: 722 bytes
Desc: not available
Url : http://mailman.laptop.org/pipermail/sugar/attachments/20060726/60ca5135/gecko-embed-test.bin
Reply | Threaded
Open this post in threaded view
|

[patch] fix gecko-embed pkgconfig

Marco Pesenti Gritti
Robert Staudinger wrote:

> Hi,
>
> the attached patch does away with gecko-embed's hardcoded .pc file and
> adds an autogen'd one instead. So it becomes usable from a c
> application.
>
> The included sample can be built with
> gcc `pkg-config --cflags --libs gtk+-2.0 gecko-embed` -lxul -o
> gecko-embed-test gecko-embed-test.c
> (there seems to be a bug in xulrunner, -lxul is not mentioned in any
> of the installed .pc files)
>
> Ok to commit?
Yeah, please check it in. Thanks!!

Marco
Reply | Threaded
Open this post in threaded view
|

[patch] fix gecko-embed pkgconfig

Robert Staudinger
On 7/27/06, Marco Pesenti Gritti <[hidden email]> wrote:
> Robert Staudinger wrote:

[...]

> > (there seems to be a bug in xulrunner, -lxul is not mentioned in any
> > of the installed .pc files)
> >
> > Ok to commit?
> Yeah, please check it in. Thanks!!

Checked in with a fix for the problem mentioned above and ChangeLog
entry (there was no ChangeLog yet).

- Rob