From: Pratyush Yadav <me@yadavpratyush.com>
To: Junio C Hamano <gitster@pobox.com>
Cc: Eric Sunshine <sunshine@sunshineco.com>,
Git List <git@vger.kernel.org>,
Jonathan Gilbert <JonathanG@iqmetrix.com>
Subject: Re: [PATCH v1 0/2] git-gui: reduce Tcl version requirement from 8.6 to 8.5
Date: Tue, 17 Mar 2020 18:19:02 +0530 [thread overview]
Message-ID: <20200317124902.fitwgrrm6jtv24ec@yadavpratyush.com> (raw)
In-Reply-To: <xmqqwo7k8fnk.fsf@gitster.c.googlers.com>
On 16/03/20 08:48AM, Junio C Hamano wrote:
> Eric Sunshine <sunshine@sunshineco.com> writes:
>
> > On Sat, Mar 14, 2020 at 9:47 PM Pratyush Yadav <me@yadavpratyush.com> wrote:
> >> Some MacOS distributions ship with Tcl 8.5. This means we can't use
> >> TclOO. So, use our homegrown class.tcl instead.
> >
> > It should be mentioned that this patch series fixes a regression in
> > Git v2.25 in which git-gui could not even be launched on Mac OS. The
> > problem was reported here[1] a couple months ago.
> >
> > I performed some rudimentary testing of this patch series on Mac OS,
> > and it appears to be working as expected; it certainly fixes the
> > problem of git-gui not launching on Mac OS. (I did notice a
> > misbehavior related to the original patch which caused git-gui to be
> > unusable on Mac OS in v2.25, but I suspect that misbehavior is not
> > related to or caused by this patch series, thus shouldn't prevent its
> > acceptance.)
>
> I was actually hesitant to see this kind of change for the first
> time this late in the cycle
*sigh* I intended to finish the change much sooner, but one thing after
another, and I kept putting it on the backburner :-(
> (the code may work with old Tcl/Tk but do we know it does with newer
> ones?)
It is actually the other way around. My system has Tcl 8.6 installed,
and it works well with it. The problem is checking if it works with 8.5.
My distro's package manager doesn't show Tcl 8.5 as an option (only
8.6), so I have to try and build 8.5 from source to test it. It is more
work than I can afford right now. So I am relying on Eric and other
MacOS users' reports.
But IIUC, unless there are any hidden gotchas in Tcl 8.5, there is a
fairly little chance that this patch breaks anything significant that
wasn't already broken before. The patch reverts from using a new
8.6-specific feature to pure, backward-compatible Tcl that should work
with older versions too. No, a larger concern for me is whether I missed
something somewhere that breaks the feature for _all_ versions of
git-gui.
> I'll pull git-gui updates when Pratyush tells me to, which would
> happen before the final (scheduled on 22nd). I'll trust git-gui
> maintainer's decision to include these changes in it, or to cook
> longer to wait for the 2.27 cycle. Comments like yours that help
> Pratyush make the right judgment is greatly appreciated.
Honestly, I'd like to cook it a bit longer but the reality is that very
few people, if any, actually track and test my tree. Most people
actually discover bugs when the changes hit a new Git release.
I cooked the original series that bumped the version requirement for
quite some time because I suspected some platforms might still be on
older versions. But I got absolutely no feedback/complaints, so I went
ahead and sent you a pull request.
The change hit my 'master' on December 6. Git v2.25.0 was tagged on
January 13th. MacOS breakage was reported on January 15th.
So, I have a choice between waiting for the 2.27 cycle keeping git-gui
broken on MacOS for another few months in hopes that someone comes in
and discovers a bug, or have it merged in v2.26 fixing git-gui for MacOS
but risk (though it shouldn't be too high) introducing a bug in _all_
platforms. Neither choice is ideal, but I'm leaning towards the latter.
Having _most_ things working is better than having nothing working at
all.
--
Regards,
Pratyush Yadav
next prev parent reply other threads:[~2020-03-17 12:49 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-14 22:41 [PATCH v1 0/2] git-gui: reduce Tcl version requirement from 8.6 to 8.5 Pratyush Yadav
2020-03-14 22:41 ` [PATCH v1 1/2] " Pratyush Yadav
2020-03-14 22:41 ` [PATCH v1 2/2] git-gui: create a new namespace for chord script evaluation Pratyush Yadav
2020-03-15 18:54 ` [PATCH v1 0/2] git-gui: reduce Tcl version requirement from 8.6 to 8.5 Eric Sunshine
2020-03-16 15:48 ` Junio C Hamano
2020-03-17 12:49 ` Pratyush Yadav [this message]
2020-03-19 15:25 ` Eric Sunshine
2020-03-17 13:29 ` [PATCH v2 " Pratyush Yadav
2020-03-17 13:29 ` [PATCH v2 1/2] " Pratyush Yadav
2020-03-17 13:29 ` [PATCH v2 2/2] git-gui: create a new namespace for chord script evaluation Pratyush Yadav
2020-03-19 15:22 ` [PATCH v2 0/2] git-gui: reduce Tcl version requirement from 8.6 to 8.5 Eric Sunshine
2020-03-19 16:05 ` Pratyush Yadav
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200317124902.fitwgrrm6jtv24ec@yadavpratyush.com \
--to=me@yadavpratyush.com \
--cc=JonathanG@iqmetrix.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=sunshine@sunshineco.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).