Re: JavaHL, 1.9: Bad file descriptor, Stream doesn't support thiscapability errors
The fix is alread on the 1.9.x branch and will be part of the 1.9.1 release Perfect, I can confirm that it is working now. Thanks. -- Tom On 25.08.2015 12:39, Branko Čibej wrote: On 25.08.2015 12:36, Thomas Singer wrote: Is this bug already reported in the issue tracker? I've searched but could not found. Should I report it? We don't need that. The fix is alread on the 1.9.x branch and will be part of the 1.9.1 release, which we'll probably expedite precisely because of this bug. -- Brane -- Best regards, Thomas Singer = syntevo GmbH http://www.syntevo.com http://www.syntevo.com/blog On 20.08.2015 09:28, b...@qqmail.nl wrote: 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:// problems + 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, because we never bothered to check for an error) The apr version of the patch is harder than I expected, because the poll function supports many flags of which we can really only implement one new variant, that can’t 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 appears 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 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 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 thiscapability errors Hi Bert, I'll see if I can properly fix this (preferably in both Subversion and Apr), without reverting to just ignoring errors. 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 = syntevo GmbH http://www.syntevo.com http://www.syntevo.com/blog
Re: JavaHL, 1.9: Bad file descriptor, Stream doesn't support thiscapability errors
On 25.08.2015 12:36, Thomas Singer wrote: Is this bug already reported in the issue tracker? I've searched but could not found. Should I report it? We don't need that. The fix is alread on the 1.9.x branch and will be part of the 1.9.1 release, which we'll probably expedite precisely because of this bug. -- Brane -- Best regards, Thomas Singer = syntevo GmbH http://www.syntevo.com http://www.syntevo.com/blog On 20.08.2015 09:28, b...@qqmail.nl wrote: 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:// problems + 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, because we never bothered to check for an error) The apr version of the patch is harder than I expected, because the poll function supports many flags of which we can really only implement one new variant, that can’t 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 appears 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 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 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 thiscapability errors Hi Bert, I'll see if I can properly fix this (preferably in both Subversion and Apr), without reverting to just ignoring errors. 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 = syntevo GmbH http://www.syntevo.com http://www.syntevo.com/blog
Re: JavaHL, 1.9: Bad file descriptor, Stream doesn't support thiscapability errors
Is this bug already reported in the issue tracker? I've searched but could not found. Should I report it? -- Best regards, Thomas Singer = syntevo GmbH http://www.syntevo.com http://www.syntevo.com/blog On 20.08.2015 09:28, b...@qqmail.nl wrote: 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:// problems + 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, because we never bothered to check for an error) The apr version of the patch is harder than I expected, because the poll function supports many flags of which we can really only implement one new variant, that can’t 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 appears 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 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 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 thiscapability errors Hi Bert, I'll see if I can properly fix this (preferably in both Subversion and Apr), without reverting to just ignoring errors. 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 = syntevo GmbH http://www.syntevo.com http://www.syntevo.com/blog
RE: JavaHL, 1.9: Bad file descriptor, Stream doesn't support thiscapability errors
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:// problems + 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, because we never bothered to check for an error) The apr version of the patch is harder than I expected, because the poll function supports many flags of which we can really only implement one new variant, that can’t 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 appears 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 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 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 thiscapability errors Hi Bert, I'll see if I can properly fix this (preferably in both Subversion and Apr), without reverting to just ignoring errors. 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 = syntevo GmbH http://www.syntevo.com http://www.syntevo.com/blog
