Running rack tests on Windows (both 1.9.3 and 2.0.0)

Running rack tests on Windows (both 1.9.3 and 2.0.0)

[Luis Lavena]

[-- Attachment #1: Type: text/plain, Size: 1086 bytes --]

Hello,

I've started to look into Rack (and its tests) on Windows just to ensure 
things are running as much properly as possible.

I found a few hardcoded values to temporary files (/tmp/rack_sendfile) that 
is blocking spec_sendfile.rb from executing.

Perhaps it will be good to use the system temporary directory instead?

Beyond that, and without installing any particular handlers, rack tests 
results in:

ruby 1.9.3p327 (2012-11-10) [i386-mingw32]
583 tests, 1761 assertions, 4 failures, 2 errors

ruby 2.0.0dev (2012-11-10 trunk 37612) [i386-mingw32]
583 tests, 1761 assertions, 5 failures, 2 errors

ruby 2.0.0dev (2012-11-10 trunk 37612) [x64-mingw32]
583 tests, 1761 assertions, 5 failures, 2 errors

See gist for full details:

https://gist.github.com/4054864

Do you think tests are incorrectly assuming details of the platform (test 
is not prepared to run on Windows) or do you think the failures are 
possible rack issues with Windows?

How would you like to proceed? I wanted to get the conversation going 
before invest more time on this.

Thank you.
--
Luis Lavena


[-- Attachment #2: Type: text/html, Size: 1465 bytes --]

Re: Running rack tests on Windows (both 1.9.3 and 2.0.0)

[James Tucker]

On Nov 11, 2012, at 5:20 AM, Luis Lavena <luislavena@gmail.com> wrote:

> Hello,
> 
> I've started to look into Rack (and its tests) on Windows just to ensure things are running as much properly as possible.

Awesome, thanks!

> I found a few hardcoded values to temporary files (/tmp/rack_sendfile) that is blocking spec_sendfile.rb from executing.

Those sound easy, if you get an issue open I'll make sure to get it done before I roll the next release.

> Perhaps it will be good to use the system temporary directory instead?

Agreed.

> Beyond that, and without installing any particular handlers, rack tests results in:
> 
> ruby 1.9.3p327 (2012-11-10) [i386-mingw32]
> 583 tests, 1761 assertions, 4 failures, 2 errors
> 
> ruby 2.0.0dev (2012-11-10 trunk 37612) [i386-mingw32]
> 583 tests, 1761 assertions, 5 failures, 2 errors
> 
> ruby 2.0.0dev (2012-11-10 trunk 37612) [x64-mingw32]
> 583 tests, 1761 assertions, 5 failures, 2 errors
> 
> See gist for full details:
> 
> https://gist.github.com/4054864
> 
> Do you think tests are incorrectly assuming details of the platform (test is not prepared to run on Windows) or do you think the failures are possible rack issues with Windows?

The multipart tests involve loading binary fixtures from the repo. They will become corrupted if you use autocrlf in your git settings (I VERY VERY strongly recommend against such settings for exactly this reason). It's also possible we're missing some binary file modes in the read calls, or there are additional encoding concerns. I'll take a deeper look. Again an issue on Github will help me remember to do so.

> How would you like to proceed? I wanted to get the conversation going before invest more time on this.

This is excellent feedback, if you have more time to prepare some patches, that's awesome, but if you just want to create some issues with this information, that's great too.

Many thanks

> 
> Thank you.
> --
> Luis Lavena
> 

Re: Running rack tests on Windows (both 1.9.3 and 2.0.0)

[Konstantin Haase]

In related news: Work on windows support for Travis CI has started. It might take a bit, though.

Konstantin

On Nov 11, 2012, at 3:20 PM, Luis Lavena <luislavena@gmail.com> wrote:

> Hello,
> 
> I've started to look into Rack (and its tests) on Windows just to ensure things are running as much properly as possible.
> 
> I found a few hardcoded values to temporary files (/tmp/rack_sendfile) that is blocking spec_sendfile.rb from executing.
> 
> Perhaps it will be good to use the system temporary directory instead?
> 
> Beyond that, and without installing any particular handlers, rack tests results in:
> 
> ruby 1.9.3p327 (2012-11-10) [i386-mingw32]
> 583 tests, 1761 assertions, 4 failures, 2 errors
> 
> ruby 2.0.0dev (2012-11-10 trunk 37612) [i386-mingw32]
> 583 tests, 1761 assertions, 5 failures, 2 errors
> 
> ruby 2.0.0dev (2012-11-10 trunk 37612) [x64-mingw32]
> 583 tests, 1761 assertions, 5 failures, 2 errors
> 
> See gist for full details:
> 
> https://gist.github.com/4054864
> 
> Do you think tests are incorrectly assuming details of the platform (test is not prepared to run on Windows) or do you think the failures are possible rack issues with Windows?
> 
> How would you like to proceed? I wanted to get the conversation going before invest more time on this.
> 
> Thank you.
> --
> Luis Lavena
> 

Re: Running rack tests on Windows (both 1.9.3 and 2.0.0)

[Luis Lavena]

[-- Attachment #1: Type: text/plain, Size: 358 bytes --]

On Sunday, November 11, 2012 8:25:54 PM UTC-3, Konstantin Haase wrote:
>
> In related news: Work on windows support for Travis CI has started. It 
> might take a bit, though. 
>
>
Please email me more details or include me in early alpha to help you crash 
the thing, or I will remove you from my friends list.

Really, I will do that.

:-D

--
Luis Lavena


[-- Attachment #2: Type: text/html, Size: 603 bytes --]

Re: Running rack tests on Windows (both 1.9.3 and 2.0.0)

[Tim Moore]

[-- Attachment #1: Type: text/plain, Size: 1846 bytes --]

Luis,

I've submitted a pull request to rack that fixes a race condition when 
starting two servers simultaneously with the same pid file.

https://github.com/rack/rack/pull/505

I don't have easy access to a Windows machine to make sure that the fix 
works correctly there. Would you be able to advise?

Thanks,
Tim Moore

On Monday, November 12, 2012 12:20:55 AM UTC+11, Luis Lavena wrote:
>
> Hello,
>
> I've started to look into Rack (and its tests) on Windows just to ensure 
> things are running as much properly as possible.
>
> I found a few hardcoded values to temporary files (/tmp/rack_sendfile) 
> that is blocking spec_sendfile.rb from executing.
>
> Perhaps it will be good to use the system temporary directory instead?
>
> Beyond that, and without installing any particular handlers, rack tests 
> results in:
>
> ruby 1.9.3p327 (2012-11-10) [i386-mingw32]
> 583 tests, 1761 assertions, 4 failures, 2 errors
>
> ruby 2.0.0dev (2012-11-10 trunk 37612) [i386-mingw32]
> 583 tests, 1761 assertions, 5 failures, 2 errors
>
> ruby 2.0.0dev (2012-11-10 trunk 37612) [x64-mingw32]
> 583 tests, 1761 assertions, 5 failures, 2 errors
>
> See gist for full details:
>
> https://gist.github.com/4054864
>
> Do you think tests are incorrectly assuming details of the platform (test 
> is not prepared to run on Windows) or do you think the failures are 
> possible rack issues with Windows?
>
> How would you like to proceed? I wanted to get the conversation going 
> before invest more time on this.
>
> Thank you.
> --
> Luis Lavena
>
>

-- 

--- 
You received this message because you are subscribed to the Google Groups "Rack Development" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rack-devel+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.



[-- Attachment #2: Type: text/html, Size: 2618 bytes --]

Re: Running rack tests on Windows (both 1.9.3 and 2.0.0)

[James Tucker]

[-- Attachment #1: Type: text/plain, Size: 1614 bytes --]

Provided your git settings are not modifying file data, you should be close
to a green run now.


On 11 November 2012 05:20, Luis Lavena <luislavena@gmail.com> wrote:

> Hello,
>
> I've started to look into Rack (and its tests) on Windows just to ensure
> things are running as much properly as possible.
>
> I found a few hardcoded values to temporary files (/tmp/rack_sendfile)
> that is blocking spec_sendfile.rb from executing.
>
> Perhaps it will be good to use the system temporary directory instead?
>
> Beyond that, and without installing any particular handlers, rack tests
> results in:
>
> ruby 1.9.3p327 (2012-11-10) [i386-mingw32]
> 583 tests, 1761 assertions, 4 failures, 2 errors
>
> ruby 2.0.0dev (2012-11-10 trunk 37612) [i386-mingw32]
> 583 tests, 1761 assertions, 5 failures, 2 errors
>
> ruby 2.0.0dev (2012-11-10 trunk 37612) [x64-mingw32]
> 583 tests, 1761 assertions, 5 failures, 2 errors
>
> See gist for full details:
>
> https://gist.github.com/4054864
>
> Do you think tests are incorrectly assuming details of the platform (test
> is not prepared to run on Windows) or do you think the failures are
> possible rack issues with Windows?
>
> How would you like to proceed? I wanted to get the conversation going
> before invest more time on this.
>
> Thank you.
> --
> Luis Lavena
>
>

-- 

--- 
You received this message because you are subscribed to the Google Groups "Rack Development" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rack-devel+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.



[-- Attachment #2: Type: text/html, Size: 2467 bytes --]