Project

General

Profile

Backport #7904

test_spawn_too_long_path(TestProcess) fails on PPC64

Added by vo.x (Vit Ondruch) about 6 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Normal
[ruby-core:52628]

Description

=begin
I observe following error on PPC64:
7) Failure:
test_spawn_too_long_path(TestProcess) [/builddir/build/BUILD/ruby-2.0.0-r39357/test/ruby/test_process.rb:1383]:
[Errno::ENOENT, Errno::E2BIG, nil] expected but nothing was raised.
8) Failure:
test_aspawn_too_long_path(TestProcess) [/builddir/build/BUILD/ruby-2.0.0-r39357/test/ruby/test_process.rb:1388]:
[Errno::ENOENT, Errno::E2BIG, nil] expected but nothing was raised.

I am quite sure, that there was no such test error testing with rev38848. Not sure if that might be related to 38934.
=end


Related issues

Related to Ruby trunk - Bug #4315: Process.spawn cause Segmentation Fault if take a long string. (windows)Closed01/25/2011Actions

Associated revisions

Revision 827e71bd
Added by nobu (Nobuyoshi Nakada) about 6 years ago

test_process.rb: fix test

  • test/ruby/test_process.rb (assert_fail_too_long_path): get rid of syntax error on sh, increase command line size until it exceeds the limit. [Bug #7904]

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@39417 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

Revision 39417
Added by nobu (Nobuyoshi Nakada) about 6 years ago

test_process.rb: fix test

  • test/ruby/test_process.rb (assert_fail_too_long_path): get rid of syntax error on sh, increase command line size until it exceeds the limit. [Bug #7904]

Revision 39417
Added by nobu (Nobuyoshi Nakada) about 6 years ago

test_process.rb: fix test

  • test/ruby/test_process.rb (assert_fail_too_long_path): get rid of syntax error on sh, increase command line size until it exceeds the limit. [Bug #7904]

Revision 39417
Added by nobu (Nobuyoshi Nakada) about 6 years ago

test_process.rb: fix test

  • test/ruby/test_process.rb (assert_fail_too_long_path): get rid of syntax error on sh, increase command line size until it exceeds the limit. [Bug #7904]

Revision 39417
Added by nobu (Nobuyoshi Nakada) about 6 years ago

test_process.rb: fix test

  • test/ruby/test_process.rb (assert_fail_too_long_path): get rid of syntax error on sh, increase command line size until it exceeds the limit. [Bug #7904]

Revision 39417
Added by nobu (Nobuyoshi Nakada) about 6 years ago

test_process.rb: fix test

  • test/ruby/test_process.rb (assert_fail_too_long_path): get rid of syntax error on sh, increase command line size until it exceeds the limit. [Bug #7904]

Revision 39417
Added by nobu (Nobuyoshi Nakada) about 6 years ago

test_process.rb: fix test

  • test/ruby/test_process.rb (assert_fail_too_long_path): get rid of syntax error on sh, increase command line size until it exceeds the limit. [Bug #7904]

Revision fa4e355e
Added by nagachika (Tomoyuki Chikanaga) about 6 years ago

merge revision(s) 39417: [Backport #7904]

test_process.rb: fix test

* test/ruby/test_process.rb (assert_fail_too_long_path): get rid of

syntax error on sh, increase command line size until it exceeds the
limit. [Bug #7904]

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_0_0@39665 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

Revision 39665
Added by nagachika (Tomoyuki Chikanaga) about 6 years ago

merge revision(s) 39417: [Backport #7904]

test_process.rb: fix test

* test/ruby/test_process.rb (assert_fail_too_long_path): get rid of

syntax error on sh, increase command line size until it exceeds the
limit. [Bug #7904]

History

Updated by ko1 (Koichi Sasada) about 6 years ago

  • Assignee set to nobu (Nobuyoshi Nakada)
  • Target version set to 2.1.0
  • Category set to core

Nobu, could you check it if it is critical or not for 2.0.0p0?

Updated by nobu (Nobuyoshi Nakada) about 6 years ago

  • Status changed from Open to Feedback

It seems a miss in the test probably.
Could you try with r39375?

Updated by vo.x (Vit Ondruch) about 6 years ago

=begin
Testing with r39383

TestProcess#test_aspawn_too_long_path = sh: -c: line 0: syntax error near unexpected token |'
sh: -c: line 0:
a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a|a

... snip ...

a|a|a|a|a|a|a|a|a|a|a|a|a|a|'
0.08 s = F

and

8) Failure:
test_aspawn_too_long_path(TestProcess) [/builddir/build/BUILD/ruby-2.0.0-r39383/test/ruby/test_process.rb:1388]:
[ruby-core:34833].

But one of the test errors (test_spawn_too_long_path) is gone.
=end

Updated by ngoto (Naohisa Goto) about 6 years ago

Failure of the same two tests were observed also on Solaris10 before r39374.
After r39375, failure of test_spawn_too_long_path also vanished on Solaris10.

On Solaris10, with r39376,

TestProcess#test_aspawn_too_long_path = sh: a: not found
sh: a: not found
sh: a: not found
sh: a: not found
(snip many lines)
sh: a: not found
sh: a: not found
sh: a: not found
sh: fork failed - too many processes
536.65 s = F

and

13) Failure:
test_aspawn_too_long_path(TestProcess) [/XXXXXXX/test/ruby/test_process.rb:1388]:
[ruby-core:34833].
[Errno::ENOENT, Errno::E2BIG] expected but nothing was raised.

#5

Updated by nobu (Nobuyoshi Nakada) about 6 years ago

  • Status changed from Feedback to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r39417.
Vit, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


test_process.rb: fix test

  • test/ruby/test_process.rb (assert_fail_too_long_path): get rid of syntax error on sh, increase command line size until it exceeds the limit. [Bug #7904]
#6

Updated by nagachika (Tomoyuki Chikanaga) about 6 years ago

  • Tracker changed from Bug to Backport
  • Project changed from Ruby trunk to Backport200
  • Category deleted (core)
  • Status changed from Closed to Assigned
  • Assignee changed from nobu (Nobuyoshi Nakada) to nagachika (Tomoyuki Chikanaga)
  • Target version deleted (2.1.0)
#7

Updated by nagachika (Tomoyuki Chikanaga) about 6 years ago

  • Status changed from Assigned to Closed

This issue was solved with changeset r39665.
Vit, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


merge revision(s) 39417: [Backport #7904]

test_process.rb: fix test

* test/ruby/test_process.rb (assert_fail_too_long_path): get rid of

syntax error on sh, increase command line size until it exceeds the
limit. [Bug #7904]

#8

Updated by naruse (Yui NARUSE) over 3 years ago

  • Related to Bug #4315: Process.spawn cause Segmentation Fault if take a long string. (windows) added

Also available in: Atom PDF