Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 24770: /trunk/packaging/ /trunk/pack
From: Gerald Combs <[email protected]>
Date: Fri, 04 Apr 2008 12:05:07 -0700
[ I'm replying to -dev instead of Stephen directly since this warrants a more
detailed and public explanation. ]

We have indeed decided to drop GTK1 support*. The main reasons are:

  - GTK1 has been unsupported by the GTK+ team for a long time now. The fact
    that we've been able to use it up to this point (that is, GTK1 + GLib2)
    has been due to random chance. That went away with recent GLib2 releases.

  - Having to support GTK1 has been holding us back in a lot of areas, and
    has added a lot of maintenance overhead. Dropping GTK1 means we can
    modernize the column configuration, coloring rules dialog, and lots of
    other things.

  - Now that 1.0 is out, if someone needs GTK1 (e.g. Windows users forced to use
    8-bit color), they have a supported version to fall back on.

  - The GTK team is working on GTK+3. Our #ifdef situation is bad enough with
    two versions of GTK. Three will just add to the pain.

8-bit color seems to be the only sticking point, in that it is possible that we
could come out with Wireshark 1.2 or 2.0 before the issue is fixed in GTK2. I
think this is vastly outweighed by the advantages of dropping GTK1.

*This is a nice way of saying "no one in either session objected, and Gerald
decided on behalf of everyone who wasn't there. Wasn't that nice of him?"

Stephen Fisher wrote:
> I didn't get to attend the whole session on Monday morning so I missed 
> the discussion of removing GTK1.  Has it been decided to completely 
> remove GTK1 from Wireshark?
> On Fri, Apr 04, 2008 at 06:18:25PM +0000, [email protected] wrote:
>> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=24770
>> User: gerald
>> Date: 2008/04/04 06:18 PM
>> Log:
>>  Start the process of removing GTK1.
>> Directory: /trunk/packaging/nsis/
>>   Changes    Path              Action
>>   +0 -22     Makefile.nmake    Modified
>>   +2 -95     wireshark.nsi     Modified
>> Directory: /trunk/packaging/portableapps/win32/
>>   Changes    Path            Action
>>   +1 -8      makenmake.pl    Modified
>> Directory: /trunk/packaging/u3/win32/
>>   Changes    Path            Action
>>   +1 -8      makenmake.pl    Modified
>> _______________________________________________
>> Wireshark-commits mailing list
>> [email protected]
>> http://www.wireshark.org/mailman/listinfo/wireshark-commits