Project

General

Profile

Actions

Backport #7746

closed

Backport NSGetEnviron()/setproctitle.c fix for macosx

Added by docwhat (Christian Höltje) almost 12 years ago. Updated over 11 years ago.

Status:
Closed
[ruby-core:51713]

Description

The fix for issue #6576 should be back ported to 1.9.3 as well. Without it you cannot build shared ruby libraries that work reliably. An example of this breakage is compiling something that uses the embedded library like vim; it crashes the instance vim tries to use ruby.

Here's the patch needed:

diff --git a/missing/setproctitle.c b/missing/setproctitle.c
index 169ba8b..4dc6d03 100644
--- a/missing/setproctitle.c
+++ b/missing/setproctitle.c
@@ -48,6 +48,12 @@
#endif
#include <string.h>

+#if defined(APPLE)
+#include <crt_externs.h>
+#undef environ
+#define environ (_NSGetEnviron())
+#endif
+
#define SPT_NONE 0 /
don't use it at all /
#define SPT_PSTAT 1 /
use pstat(PSTAT_SETCMD, ...) /
#define SPT_REUSEARGV 2 /
cover argv with title information */

Updated by kosaki (Motohiro KOSAKI) almost 12 years ago

Confused. Mac OS X has native setproctile(). I think missing/setproctitle.c shouldn't be used.

Updated by Anonymous over 11 years ago

kosaki (Motohiro KOSAKI) wrote:

Confused. Mac OS X has native setproctile(). I think missing/setproctitle.c shouldn't be used.

At least 10.6 does not have setproctitle().

Updated by kosaki (Motohiro KOSAKI) over 11 years ago

  • Status changed from Open to Assigned
  • Assignee changed from kosaki (Motohiro KOSAKI) to usa (Usaku NAKAMURA)

OK, ack then.
Usa-san, I think r36151 can be apply cleanly.

Actions #4

Updated by usa (Usaku NAKAMURA) over 11 years ago

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

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


merge revision(s) 36151: [Backport #7746]

* missing/setproctitle.c (environ): use (*_NSGetEnviron()) instead of
  environ on Darwin for namespace cleanness, same as [ruby-core:00537].
  [ruby-core:45615] [Bug #6576]
Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0