794772 - Port |Bug 713802 - Build with GIO support (and drop GnomeVFS)|

Status: RESOLVED FIXED

(SeaMonkey :: Build Config, defect)

Description

[Jens Hatlak (:InvisibleSmiley)]

Bug 713802 just landed on mozilla-inbound (m-c will follow soon). We need to port this (esp. the gnomevfs removal) to stay in sync.

Comment 1

[Philip Chee]

Don't we have a dependency on GnomeVFS in our GNOME shell service?

http://mxr.mozilla.org/comm-central/search?string=nsIGnomeVFSService.h

perhaps we don't actually use it.

Comment 2

[Philip Chee]

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1348801551.1348801708.16537.gz#err0
checking MOZ_PANGO_LIBS... -L/lib -lpangoft2-1.0 -lpangocairo-1.0 -lpango-1.0 -lcairo -lgobject-2.0 -lgmodule-2.0 -ldl -lglib-2.0
checking for gtk+-2.0 >= 2.14... configure: error: * * * Could not find gtk+-2.0 > 2.14. Required for build with gio.
configure: error: /builds/slave/comm-cen-trunk-lnx-dbg/build/mozilla/configure failed for mozilla

Comment 3

[Jens Hatlak (:InvisibleSmiley)]

(In reply to Philip Chee from comment #2)
> http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1348801551.1348801708.
> 16537.gz#err0
> checking MOZ_PANGO_LIBS... -L/lib -lpangoft2-1.0 -lpangocairo-1.0
> -lpango-1.0 -lcairo -lgobject-2.0 -lgmodule-2.0 -ldl -lglib-2.0
> checking for gtk+-2.0 >= 2.14... configure: error: * * * Could not find
> gtk+-2.0 > 2.14. Required for build with gio.

Hmm, I thought bug 794285 restored the ability to build with GTK 2.10? Was the build you referred to started before the checkin for bug 794285?

Comment 4

[Philip Chee]

> Hmm, I thought bug 794285 restored the ability to build with GTK 2.10? Was the build
> you referred to started before the checkin for bug 794285?

See TB patch:
http://hg.mozilla.org/comm-central/rev/43533642df86
"Temporarily force-enable gnomevfs and disable gio until our Linux builders are upgraded (bug 794378) due to bug 713802 landing in mozilla-central. r=bustage-fix for CLOSED TREE"

Comment 5

[Jens Hatlak (:InvisibleSmiley)]

A second changeset (modifying configure.in) was pushed for the base bug:
http://hg.mozilla.org/mozilla-central/rev/157f3efdee37
http://hg.mozilla.org/mozilla-central/rev/e520ade90abf

Also note dependent bug 795144:
http://hg.mozilla.org/mozilla-central/rev/358ade692d25
http://hg.mozilla.org/mozilla-central/rev/1949b59f3d12

Maybe we should do all of that here.

Please note that I'll be offline until Thursday, so I cannot work on this meanwhile (and not attend the status meeting on Tuesday either).

Comment 6

[Edmund Wong (:ewong)]

Attachment: Port patch (v1)

Comment 7

[Ian Neal]

As well those mentioned in comment 5 there is also the configure.in changes from the following:
bug 805202 - http://hg.mozilla.org/mozilla-central/diff/a4e9d21b8ca8/configure.in
bug 467168 - http://hg.mozilla.org/mozilla-central/diff/78ed4ddb5fbd/configure.in
bug 402892 - http://hg.mozilla.org/mozilla-central/diff/2572312e17df/configure.in
Plus a few more if you do some digging.

Comment 8

[Ian Neal]

Comment on attachment 704864 [details] [diff] [review]
Port patch (v1)

It is a start but more porting is needed I think.

Comment 9

[Edmund Wong (:ewong)]

Gone back to square one, so this bug will have to have someone else to give it love.  I might retake this bug when I feel I have the brain cells to match the required skill.  Until then, this is available for anyone to take.

Comment 10

[Edmund Wong (:ewong)]

Taking this up again :)

Comment 11

[Edmund Wong (:ewong)]

Given the work from bug 846540, does this mean that we just need to
remove the --enable-gnomevfs from the mozconfigs and push the patch given
in this bug? (https://bugzilla.mozilla.org/attachment.cgi?id=704864) ?

As I understand it, all the GIO stuff is in mozilla/'s configure.in and if
I understand bug 846540, this means suite/ just needs the aforementioned patch?

Comment 12

[Ian Neal]

Comment on attachment 704864 [details] [diff] [review]
Port patch (v1)

>-MOZ_EXTENSIONS_DEFAULT=" venkman inspector irc gnomevfs"
>+MOZ_EXTENSIONS_DEFAULT=" venkman inspector irc gio"

Obviously without the venkman now.
r=me

Comment 13

[Edmund Wong (:ewong)]

(In reply to Ian Neal from comment #12)
> Comment on attachment 704864 [details] [diff] [review]
> Port patch (v1)
> 
> >-MOZ_EXTENSIONS_DEFAULT=" venkman inspector irc gnomevfs"
> >+MOZ_EXTENSIONS_DEFAULT=" venkman inspector irc gio"
> 
> Obviously without the venkman now.
> r=me

I'll update the patch.

Comment 14

[Edmund Wong (:ewong)]

Attachment: port patch (v2)

Comment 15

[Edmund Wong (:ewong)]

Pushed to comm-central:
https://hg.mozilla.org/comm-central/rev/7b92fe9c3408