183389 - [gtk2] porting xpinstall to gtk2

Status: RESOLVED FIXED

(SeaMonkey :: Installer, defect)

Description

[Robin Lu]

xpinstall need port to gtk2

Comment 1

[Robin Lu]

Attachment: patch

Comment 2

[Remi Cohen-Scali]

*** Bug 191828 has been marked as a duplicate of this bug. ***

Comment 3

[Samir Gehani]

Comment on attachment 108147 [details] [diff] [review]
patch

Could you explain why you have forked the code when so much of it looks the
same?  Also could you please attach diffs between the *GTK2.{h,cpp} classes and
the original classes.  

What is the plan to move to GTK2?  Are we going to tranistion mozilla to switch
over entirely any time soon?  If so, maybe just using macros (if only needed in
few places) would be sufficient for now and we could remove them and supprt for
GTK 1.2 once we make the transition.

As it stands it is hard to recognize what this patch has changed.

Comment 4

[Robin Lu]

I did so just because if some day we want to totally seperate the two install
wizards, it might be easier. It's just my concern. If you think it's better to
use macro to seperate them, I can provide a new patch. 

I do not know mozilla.org's plan for GTK2. I guess it might not be a complete
switch and mozilla for gtk1.2 and gtk2.0 would coexist for a while. However,
there are a lot of people interested in moz+gtk2 and want it could be released soon.

Comment 5

[Robin Lu]

Attachment: patch

This patch ueses macro instead of create new files.

Comment 6

[Andrew Schultz]

Robin: I think Samir (sgehani) is very gone.

==> Installer

Comment 7

[Alexander Opitz]

Robin: Does the patch work with the existing trunk? (I ask, cause the patch is
very old :-))

Comment 8

[Robin Lu]

I think the patch mostly works for the trunk since there's not many changes in
this part of code. I will update the patch if needed.

Comment 9

[Andrew Schultz]

Comment on attachment 114394 [details] [diff] [review]
patch

Robin: I broke this patch in a few places with "silent" mode and removal of the
completion dialog.  I'm trying out the patch now.

So far, I've noticed the gtk2 installer complaining:
(mozilla-installer-bin:16149): Gtk-WARNING **: gtkwidget.c:3697: widget not
within a GtkWindow

from
http://lxr.mozilla.org/seamonkey/source/xpinstall/wizard/unix/src2/nsXInstaller
.cpp#315

    GTK_WIDGET_SET_FLAGS(gCtx->next, GTK_CAN_DEFAULT);
    gtk_widget_grab_default(gCtx->next);

the code should go farther down (after gCtx->next has been attached).  I put it
right above the gtk_widget_show calls, and it works fine (gtk1 and gtk2).

Also, the text area for the README and License looks different in the gtk2
version.  The gtk1 version has a border and the gtk2 does not.	Is there any
way to add that back in for the gtk2 version?

If you attach a new patch, remember to not add any new tabs (the files already
have some tabs randomly dispersed).

Comment 10

[Andrew Schultz]

I forgot to mention, there's also 'error: invalid conversion from `const gchar*'
to `char*' here:
http://lxr.mozilla.org/mozilla/source/xpinstall/wizard/unix/src2/nsSetupTypeDlg.cpp#662

   char *selDir = gtk_file_selection_get_filename(
                     GTK_FILE_SELECTION(aFileSel));

Comment 11

[Robin Lu]

> the code should go farther down (after gCtx->next has been attached).  I put it
> right above the gtk_widget_show calls, and it works fine (gtk1 and gtk2).
I did the same thing as you in my working patch


> I forgot to mention, there's also 'error: invalid conversion from `const gchar*'
> to `char*' here:
I have changed the "char*" to "const char*"

Comment 12

[Andrew Schultz]

bryner just checked in a gtk2 installer for firefox/thunderbird...
it's in mozilla/toolkit/mozapps/unix/wizard

Comment 13

[Frédéric Buclin]

How to get this for Mozilla Seamonkey?
I still use the GTK1 mozilla-installer-bin from the official release and put it
in /xpinstall/wizard/unix/src2/ before using "perl deliver.pl" in
/xpinstall/packager/unix/. But this is not very smart!

Comment 14

[Andrew Schultz]

Attachment: updated patch

this is Robin's patch, merged to trunk and just removing the use of packer
instead of #ifdef'ing it out.

Comment 15

[Daniel Veditz [:dveditz]]

Comment on attachment 160207 [details] [diff] [review]
updated patch

r=dveditz

Comment 16

[Andrew Schultz]

me

Comment 17

[Andrew Schultz]

fixed