Re: [PATCH] Remove dependency on IO::String from Git.pm test

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Johannes Schindelin
Date: Wednesday, June 18, 2008 - 12:35 pm

Hi,

On Wed, 18 Jun 2008, Jakub Narebski wrote:


So?  Why do you want to break the _test_ on those machines where you need 
them most?  We _know_ that a release from Junio works fine on Linux.


I have to point out that the platforms I was speaking of are not know to 
make upgrading as easy as Linux.  And some of them _do_ come with pretty 
old perl.  And yes, I had this exact issue (remember when I worked on 
removing Git's dependency on Python?  That was it.  Not enough quota.  
Uncooperative admin.  Desperate need for a sensible SCM).

In any case, I have to reiterate my point: breaking 
backwards-compatibility for _no_ good reason is wrong.  I have not looked 
at the patch in question, but I seriously doubt that this is the easiest, 
most elegant (and yes, backwards-compatible) solution.

Typically, it is not a good sign when you _require_ newer and newer 
features and versions of components you use.

Whatever,
Dscho

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] Remove dependency on IO::String from Git.pm test, Michael Hendricks, (Wed Jun 18, 6:37 am)
Re: [PATCH] Remove dependency on IO::String from Git.pm test, Rafael Garcia-Suarez, (Wed Jun 18, 8:10 am)
Re: [PATCH] Remove dependency on IO::String from Git.pm test, Johannes Schindelin, (Wed Jun 18, 10:00 am)
Re: [PATCH] Remove dependency on IO::String from Git.pm test, Jakub Narebski, (Wed Jun 18, 10:52 am)
Re: [PATCH] Remove dependency on IO::String from Git.pm test, Junio C Hamano, (Wed Jun 18, 12:13 pm)
Re: [PATCH] Remove dependency on IO::String from Git.pm test, Johannes Schindelin, (Wed Jun 18, 12:35 pm)
Re: [PATCH] Remove dependency on IO::String from Git.pm test, Johannes Schindelin, (Wed Jun 18, 12:37 pm)