Return-Path: X-Original-To: apmail-subversion-dev-archive@minotaur.apache.org Delivered-To: apmail-subversion-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id DF3901757D for ; Thu, 20 Aug 2015 07:28:38 +0000 (UTC) Received: (qmail 23509 invoked by uid 500); 20 Aug 2015 07:28:33 -0000 Delivered-To: apmail-subversion-dev-archive@subversion.apache.org Received: (qmail 23458 invoked by uid 500); 20 Aug 2015 07:28:33 -0000 Mailing-List: contact dev-help@subversion.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list dev@subversion.apache.org Received: (qmail 23439 invoked by uid 99); 20 Aug 2015 07:28:33 -0000 Received: from Unknown (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 20 Aug 2015 07:28:33 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id 00177182105 for ; Thu, 20 Aug 2015 07:28:33 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 2.881 X-Spam-Level: ** X-Spam-Status: No, score=2.881 tagged_above=-999 required=6.31 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=3, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, URIBL_BLOCKED=0.001] autolearn=disabled Authentication-Results: spamd3-us-west.apache.org (amavisd-new); dkim=pass (1024-bit key) header.d=qqmail.nl Received: from mx1-us-west.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id ps5-UiPQCqaE for ; Thu, 20 Aug 2015 07:28:24 +0000 (UTC) Received: from mail-wi0-f175.google.com (mail-wi0-f175.google.com [209.85.212.175]) by mx1-us-west.apache.org (ASF Mail Server at mx1-us-west.apache.org) with ESMTPS id 0B2882100B for ; Thu, 20 Aug 2015 07:28:24 +0000 (UTC) Received: by wicne3 with SMTP id ne3so7720925wic.1 for ; Thu, 20 Aug 2015 00:28:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qqmail.nl; s=google; h=message-id:mime-version:to:from:subject:date:importance:in-reply-to :references:content-type; bh=VNw62Tn+/pGRm2she6nHl/Uv3ZztWydyPM2lYa/V8m0=; b=MH11H58BoZh/VFXNNyAs7EYa0Ek3E9JT5FRcxeD3bdXbYqbLi+B7GFUNrlQIo2/bMV 9VyaQuKHaeo/8RUIyEwiLd335n4CWU3tNDE2jJXVMAad//ogwMAoQKp6zPlI/tlh5B5g RUCP/2+OnEh4KfH1+i2Si2PsS0Cl8QWKZo3qU= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:message-id:mime-version:to:from:subject:date :importance:in-reply-to:references:content-type; bh=VNw62Tn+/pGRm2she6nHl/Uv3ZztWydyPM2lYa/V8m0=; b=WBUxR+QRxYASIhO9+rA7hQ3KK1d2eN/MGuGZBhUe4/XLy4VYrn4++91H7dXQTOagVx o30Jheg7Hwcbj1v737hutyb424LjpoyBlS4ypRz5h7ZP651zFH1GP5pPtHHePM0PDVLC wTYc4Dk5TaBK91xfjRlHltTNGOGll44QBYMNlLeSkkC8nLT4gMpqxcumXCwc5MIo9iRt pYEWd3iIYCJc/s7EdqErjR9ZLr6bs3AacvTQvxt/6EPMoW7647EOxKK8yu+KrNkq2dkD ZzBmMoRKh1n207LJdX0WCF0gbI5zt58eadXV8Af5f6Cggu6laH76metTMbGV7CFvN2wu 9jwQ== X-Gm-Message-State: ALoCoQk8oyieRtut/jmdtlTlPeRZxcTqLS0lEKFW6sOOLeFhMFwa86gJplNYgjT0pTusvPTreZbA X-Received: by 10.194.85.163 with SMTP id i3mr3318706wjz.75.1440055702723; Thu, 20 Aug 2015 00:28:22 -0700 (PDT) Received: from ?IPv6:2001:610:66e:0:c233:5eff:fe29:3cff? ([2001:610:66e:0:c233:5eff:fe29:3cff]) by smtp.gmail.com with ESMTPSA id o3sm4251499wjx.16.2015.08.20.00.28.21 (version=TLSv1/SSLv3 cipher=OTHER); Thu, 20 Aug 2015 00:28:21 -0700 (PDT) Message-ID: <55d58195.2350c20a.67a0.ffff882b@mx.google.com> MIME-Version: 1.0 To: Thomas Singer , "dev@subversion.apache.org" From: Subject: RE: JavaHL, 1.9: "Bad file descriptor", "Stream doesn't support thiscapability" errors Date: Thu, 20 Aug 2015 09:28:24 +0200 Importance: normal X-Priority: 3 In-Reply-To: <55D5723B.1060000@syntevo.com> References: <55B5DB1F.2000400@syntevo.com> <55B5DC15.9030004@wandisco.com> <55CC805C.5000102@syntevo.com> <55CD1831.4020907@wandisco.com> <55CD9446.9020707@syntevo.com> <871tf6usx2.fsf@wandisco.com> <55D185FC.1000203@syntevo.com> <00e501d0d8ca$a7cfa0e0$f76ee2a0$@qqmail.nl> <55D5723B.1060000@syntevo.com> Content-Type: multipart/alternative; boundary="_F82741A9-E932-4D16-9F78-7249D946E387_" --_F82741A9-E932-4D16-9F78-7249D946E387_ Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Hi Thomas, The Subversion version of the fix is nominated for 1.9.1 [[ + * r1696222, r1696225 + Implement polling on named pipes on Windows to resolve svn+ssh:// probl= ems + Justification: + Without this patch operations like checkout just fail with an error. + This is a regression as older Subversion releases ignored the error. + Votes: ]] Apr just returns EBADF on any attempt to poll on a file or pipe on Windows.= This workaround on the Subversion side makes us support polling on pipes. = (In Subversion < 1.8 we always assumed that there was no data waiting, beca= use we never bothered to check for an error) The apr version of the patch is harder than I expected, because the poll fu= nction supports many flags of which we can really only implement one new va= riant, that can=E2=80=99t really be combined with any other flags. We need one more vote on this, before we can start the release process on 1= .9.1. I would hope we can also get r1696695 in the release, as that problem appea= rs to break Subversion for at least some Linux distributions. We might want to consider also fixing the polling in 1.8 (and 1.7?), as tha= t would allow exiting much earlier when a commit fails. Currently Subversio= n (when using svn+ssh:// on Windows) just goes on trying to commit on Windo= ws and only notices that svnserve returned an error when done. Bert Sent from Mail for Windows 10 From: Thomas Singer Sent: donderdag 20 augustus 2015 08:23 To: dev@subversion.apache.org Subject: Re: JavaHL, 1.9: "Bad file descriptor", "Stream doesn't support th= iscapability" errors Hi Bert, > I'll see if I can properly fix this (preferably in both Subversion and Ap= r), without reverting to just ignoring errors. Did you have any success fixing it yet? Is there already a bug entered=20 in the issue tracker? I'd rate it as blocker priority. -- Best regards, Thomas Singer =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D syntevo GmbH http://www.syntevo.com http://www.syntevo.com/blog --_F82741A9-E932-4D16-9F78-7249D946E387_ Content-Transfer-Encoding: quoted-printable Content-Type: text/html; charset="utf-8"

Hi Thomas,

 

The Subversion version of the fix is nominate= d for 1.9.1

[[=

+ * r1696222, r1696225

+= =C2=A0=C2=A0 Implement polling on named pipes on Windows to resolve svn+ssh= :// problems

+=C2=A0=C2=A0 Justification:

+=C2=A0=C2=A0=C2=A0=C2=A0 Without this patch operations like= checkout just fail with an error.

+=C2=A0=C2=A0=C2= =A0=C2=A0 This is a regression as older Subversion releases ignored the err= or.

+=C2=A0=C2=A0 Votes:

]= ]

 

Apr just returns = EBADF on any attempt to poll on a file or pipe on Windows. This workaround = on the Subversion side makes us support polling on pipes. (In Subversion &l= t; 1.8 we always assumed that there was no data waiting, because we never b= othered to check for an error)

<= o:p> 

The apr version of the pat= ch is harder than I expected, because the poll function supports many flags= of which we can really only implement one new variant, that can=E2=80=99t = really be combined with any other flags.

 

 

We need one more vote on this, before we = can start the release process on 1.9.1.

 

I would hope we = can also get r1696695 in the release, as that pro= blem appears to break Subversion for at least some Linux distributions.

 

 

 = ;

We might want to consider also fixi= ng the polling in 1.8 (and 1.7?), as that would allow exiting much earlier = when a commit fails. Currently Subversion (when using svn+ssh:// on Windows= ) just goes on trying to commit on Windows and only notices that svnserve r= eturned an error when done.

 

 

<= p>Bert

 

S= ent from Mail for Windows 10

 

 

 

 

=

Hi Bert,

 

=

> I'll see if I can properly fix this (preferably i= n both Subversion and Apr), without reverting to just ignoring errors.

<= p class=3DMsoNormal> 

Did you have = any success fixing it yet? Is there already a bug entered

in the issue tracker? I'd rate it as blocker priority.

 

--

Best regards,

Thomas Singer

=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D

syn= tevo GmbH

http://www.syntevo.com

http://www.syntevo.com/blog

 =

 

= --_F82741A9-E932-4D16-9F78-7249D946E387_--