[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Status report for Ruby bindings on Windows

From: Kouhei Sutou <kou_at_cozmixng.org>
Date: 2007-02-02 15:55:22 CET

Hi,

In <ae6cb1100702012007y7827bdc5jbda88a3505b2a1ce@mail.gmail.com>
  "Re: Status report for Ruby bindings on Windows" on Thu, 1 Feb 2007 20:07:28 -0800,
  "Joe Swatosh" <joe.swatosh@gmail.com> wrote:

> > > > > * Normalizing the line breaks. Upon reflection, I've decided that I
> > > > > started us down the wrong path when I modified the tests for the line
> > > > > ending issues. As I've thought about it more, I now believe that the
> > > > > bindings should do the conversions. My thinking is since Ruby treats
> > > > > line endings as "'\n" instead of "\r\n" the bindings should provide
> > > > > any multiline values Ruby expects. (This is where the 'b' comes from
> > > > > when calling File::open. Without the 'b' on windows there is a \r\n
> > > > > to \n conversion. With the 'b', no conversion. Maybe we should check
> > > > > out what the Perl and Python bindings do about this issue on
> > > > > windows?). (I've made zero effort to figure out what this will take).
> > > >
> > > > OK. We'll try the problem after we solve test environment
> > > > problem on Windows.
> > > >
> > >
> > > Removing all invocations and the definition of normalize_line_break
> > > from the test\*.rb files and the attached patch get us:
> >
> > It seems that the changes make cat ignore svn:eof-style. I
> > think tests have a problem. Could you try the attached
> > patch?
>
> The client tests pass with the patch. I (now) still think normalizing
> the line breaks in the test is the wrong solution. As you pointed
> out, my patch wasn't right for cat, but I hope you'll consider parts
> of it, or something like it for the the Core methods. Ruby on windows
> uses \n for line breaks, even though windows uses \r\n. This gives us
> a real problem to know when to pass the \r through and when not. I'll
> see if I can figure out how (and if) the perl bindings deal with the
> issue.

OK. Could you give me a time to think this issue?

> Unrelated, please
>
> svn propset svn:eol-style native
> subversion\bindings\swig\ruby\test\windows_util.rb

I've committed.

Thanks,

--
kou
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Feb 2 15:55:41 2007

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.