Thread (3 messages) flat view 3 messages, 3 authors, 2016-06-15

Re: [PATCH] t9010: svnadmin can fail even if available

From: Ramkumar Ramachandra <hidden>
Date: 2016-06-15 22:50:21

Hi,

Jonathan Nieder writes:
To do the same for t91* would be impossible.  If svn is broken or not
installed, svn-fe will run fine, but "git svn" will not.  On the other
hand, if svnadmin were broken but svn still worked, "git svn" would be
fine but that would be quite strange and I do not think it is worth
spending time to prepare for.
I don't think it's worth spending time preparing for every concievable
breakage. The patch A few more examples of possible breakages I've
encountered:
- APR compiled without threading support, SVN compiled with it, or
  viceversa.
- SVN is compiled against GNU iconv, but apr-iconv installed, or
  viceversa.
- Two different versions of a dependent library are installed, and SVN
  links to a different version in a different location.

One or many components of SVN may fail. So, I'm in favor of the
current approach: if SVN is installed, attempt to run all the t91*
tests. Any failure can either be interpreted as a real test failure or
malformed SVN installation.

-- Ram
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help