From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED.blaine.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: Increase minimum required GTK versions? Date: Wed, 10 Apr 2019 18:57:47 +0300 Message-ID: <83sgup6ef8.fsf@gnu.org> References: <87h8b9tz84.fsf@gmail.com> <83h8b98wg3.fsf@gnu.org> <87d0lxty7f.fsf@gmail.com> <537036b7-549c-0aab-cc06-70c26c683ac7@cs.ucla.edu> <87y34kiloy.fsf@gmail.com> <87o95f141i.fsf@gmail.com> <838swj6nuk.fsf@gnu.org> <87h8b61rid.fsf@gmail.com> Injection-Info: blaine.gmane.org; posting-host="blaine.gmane.org:195.159.176.226"; logging-data="210850"; mail-complaints-to="usenet@blaine.gmane.org" Cc: eggert@cs.ucla.edu, monnier@iro.umontreal.ca, emacs-devel@gnu.org To: Alex Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Wed Apr 10 17:58:45 2019 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([209.51.188.17]) by blaine.gmane.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1hEFcb-000sjd-1v for ged-emacs-devel@m.gmane.org; Wed, 10 Apr 2019 17:58:45 +0200 Original-Received: from localhost ([127.0.0.1]:33742 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hEFca-0000FR-13 for ged-emacs-devel@m.gmane.org; Wed, 10 Apr 2019 11:58:44 -0400 Original-Received: from eggs.gnu.org ([209.51.188.92]:38355) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hEFbj-0000D9-Bj for emacs-devel@gnu.org; Wed, 10 Apr 2019 11:57:52 -0400 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]:36516) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hEFbi-0006Yd-A6; Wed, 10 Apr 2019 11:57:50 -0400 Original-Received: from [176.228.60.248] (port=3414 helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1hEFbg-0005Q7-9R; Wed, 10 Apr 2019 11:57:49 -0400 In-reply-to: <87h8b61rid.fsf@gmail.com> (message from Alex on Tue, 09 Apr 2019 21:14:02 -0600) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.org gmane.emacs.devel:235220 Archived-At: > From: Alex > Cc: eggert@cs.ucla.edu, monnier@iro.umontreal.ca, emacs-devel@gnu.org > Date: Tue, 09 Apr 2019 21:14:02 -0600 > > > To allow proper review, please make the log message more detailed. In > > particular, stuff you remove that is not under an explicit GTK version > > condition should have a minimal explanation for why it is TRT to > > remove it. (The rationale doesn't have to be in the log message, > > although it would be nice to have it there; it could be just in the > > email that accompanies the patch.) > > Here's a brief overview of the changes outside of explicit checks: Thanks, I think most of this text should be in the commit log. It's not too long for that, and provides useful information. > * Old file selection dialog > This is only optionally used in GTK 2, but the old dialog is long > deprecated in GTK 2 (removed in GTK 3), so I figured that this is a good > time to remove it and simplify the existing code. I think this should be left alone. It's a separate issue anyway, and should be submitted separately. It would be good to try to establish whether this feature is turned on by someone, but given that it's only for GTK2, I tend to think we should simply let it die a natural death together with GTK2. > > Please also tell with which GTK versions did you try building with to > > verify the correctness of the changes. > > I tested with GTK 2.24.32 and GTK 3.24.7. I didn't see a need to test > with an older GTK 3 version since the GTK 3 part of the change involves > simplifying version checks 3 <= version <= 3.10 (and > USE_NEW_GTK_FONT_CHOOSER) to HAVE_GTK3 checks. I think it would be good to see if the changes compile with 3.10 nonetheless. Can someone please try that? > > Not sure we discussed 3.10 as the minimum GTK3 version, we only talked > > about 2.x AFAIR. What is the situation with GTK3 versions in the > > current distributions of GNU/Linux? > > My messages briefly discussed the minimum of 3.10 and how Debian Jessie > (oldstable -- soon to be oldoldstable), and Ubuntu 14.04LTS have GTK 3.10. What about others, like RH? > RHEL doesn't even support GTK 3 until RHEL 7, which has 3.22. AFAIK, Red Hat is not limited to RHEL, and then there are derivatives, like CentOS. (I'm not saying those others come with older GTK3, I'm just saying we should check.) > So I think bumping to 3.10 doesn't hurt, especially since there's still > GTK 2 support. It sounds to me a bit drastic to tell people to downgrade to GTK2 because we don't support old enough GTK3. Thanks.