ruby-core@ruby-lang.org archive (unofficial mirror)
 help / color / mirror / Atom feed
From: ngotogenome@gmail.com
To: ruby-core@ruby-lang.org
Subject: [ruby-core:69899] [Ruby trunk - Bug #11336] TestProcess#test_exec_fd_3_redirect failed on Solaris 10
Date: Wed, 08 Jul 2015 11:03:06 +0000	[thread overview]
Message-ID: <redmine.journal-53317.20150708110305.ac013fcf25cf53c3@ruby-lang.org> (raw)
In-Reply-To: redmine.issue-11336.20150707093603@ruby-lang.org

Issue #11336 has been updated by Naohisa Goto.


> Also, maybe the __APPLE__ || __HAIKU__ code above is suitable for all
OS, too.

I think so, and the following patch solves the failure ofTestProcess#test_exec_fd_3_redirect on Solaris 10.

~~~
Index: process.c
===================================================================
--- process.c   (revision 51194)
+++ process.c   (working copy)
@@ -2473,7 +2473,7 @@
     RB_GC_GUARD(execarg_obj);
 }
 
-#if defined(__APPLE__) || defined(__HAIKU__)
+#if defined(__APPLE__) || defined(__HAIKU__) || defined(__sun)
 static int rb_exec_without_timer_thread(const struct rb_execarg *eargp, char *errmsg, size_t errmsg_buflen);
 #endif
 
@@ -2563,7 +2563,7 @@
     rb_execarg_parent_start(execarg_obj);
     fail_str = eargp->use_shell ? eargp->invoke.sh.shell_script : eargp->invoke.cmd.command_name;
 
-#if defined(__APPLE__) || defined(__HAIKU__)
+#if defined(__APPLE__) || defined(__HAIKU__) || defined(__sun)
     rb_exec_without_timer_thread(eargp, errmsg, sizeof(errmsg));
 #else
     before_exec_async_signal_safe(); /* async-signal-safe */
@@ -3077,7 +3077,7 @@
     return -1;
 }
 
-#if defined(__APPLE__) || defined(__HAIKU__)
+#if defined(__APPLE__) || defined(__HAIKU__) || defined(__sun)
 static int
 rb_exec_without_timer_thread(const struct rb_execarg *eargp, char *errmsg, size_t errmsg_buflen)
 {
~~~


----------------------------------------
Bug #11336: TestProcess#test_exec_fd_3_redirect failed on Solaris 10
https://bugs.ruby-lang.org/issues/11336#change-53317

* Author: Naohisa Goto
* Status: Open
* Priority: Normal
* Assignee: 
* ruby -v: ruby 2.3.0dev (2015-07-06) [sparc64-solaris2.10]
* Backport: 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: UNKNOWN
----------------------------------------
On Solaris 10, TestProcess#test_exec_fd_3_redirect failed since r51146 (where the test was added).

~~~
  1) Failure:
TestProcess#test_exec_fd_3_redirect [/XXXXX-51146/test/ruby/test_process.rb:2049]:
<"."> expected but was
<nil>.
~~~





-- 
https://bugs.ruby-lang.org/

  parent reply	other threads:[~2015-07-08 10:36 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <redmine.issue-11336.20150707093603@ruby-lang.org>
2015-07-07  9:36 ` [ruby-core:69886] [Ruby trunk - Bug #11336] [Open] TestProcess#test_exec_fd_3_redirect failed on Solaris 10 ngotogenome
2015-07-07 16:50   ` [ruby-core:69889] " Eric Wong
2015-07-07 22:21 ` [ruby-core:69893] [Ruby trunk - Bug #11336] " ngotogenome
2015-07-07 22:33   ` [ruby-core:69894] " Eric Wong
2015-07-08  0:47 ` [ruby-core:69895] " ngotogenome
2015-07-08  2:55   ` [ruby-core:69897] " Eric Wong
2015-07-08 11:03 ` ngotogenome [this message]
2015-07-10  9:46 ` [ruby-core:69934] " ngotogenome
2015-07-10  9:52 ` [ruby-core:69935] " ngotogenome
2015-07-16 12:11 ` [ruby-core:70001] [Ruby trunk - Bug #11336] [Open] " ngotogenome
2015-07-16 21:26   ` [ruby-core:70005] " Eric Wong
2015-07-18  3:43     ` [ruby-core:70023] " Eric Wong
2015-07-20  8:13       ` [ruby-core:70050] " Eric Wong
2015-07-17  6:27 ` [ruby-core:70008] [Ruby trunk - Bug #11336] " ngotogenome
2015-07-19 11:27 ` [ruby-core:70041] " ngotogenome
2015-07-20  2:44   ` [ruby-core:70048] " Eric Wong
2015-07-31 13:29 ` [ruby-core:70204] " ngotogenome
2015-07-31 21:02   ` [ruby-core:70206] " Eric Wong
2015-08-14  8:33 ` [ruby-core:70386] [Ruby trunk - Bug #11336] [Open] " ngotogenome
2015-08-14  9:46   ` [ruby-core:70387] " Eric Wong
2015-08-14 15:27 ` [ruby-core:70390] [Ruby trunk - Bug #11336] [Closed] " ngotogenome

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.ruby-lang.org/en/community/mailing-lists/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=redmine.journal-53317.20150708110305.ac013fcf25cf53c3@ruby-lang.org \
    --to=ruby-core@ruby-lang.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).