www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 5319] New: rt.critical/monitor: Add support for Solaris

reply d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=5319

           Summary: rt.critical/monitor: Add support for Solaris
           Product: D
           Version: D2
          Platform: All
        OS/Version: Solaris
            Status: NEW
          Severity: normal
          Priority: P2
         Component: druntime
        AssignedTo: sean invisibleduck.org
        ReportedBy: ibuclaw ubuntu.com


--- Comment #0 from Iain Buclaw <ibuclaw ubuntu.com> 2010-12-04 14:02:41 PST ---
Patch:

--- druntime-trunk~/src/rt/critical.c    2010-08-05 09:59:36.000000000 +0100
+++ druntime-trunk/src/rt/critical.c    2010-12-04 21:45:34.417486613 +0000
   -82,7 +82,7   

 /* ================================= linux ============================ */

-#if linux || __APPLE__ || __FreeBSD__
+#if linux || __APPLE__ || __FreeBSD__ || __sun&&__SVR4

 #include <stdio.h>
 #include <stdlib.h>
--- druntime-trunk~/src/rt/monitor.c    2010-08-05 09:59:36.000000000 +0100
+++ druntime-trunk/src/rt/monitor.c    2010-12-04 21:42:17.513486612 +0000
   -15,7 +15,7   
 #include <assert.h>

 #if _WIN32
-#elif linux || __APPLE__ || __FreeBSD__
+#elif linux || __APPLE__ || __FreeBSD__ || __sun&&__SVR4
 #define USE_PTHREADS    1
 #else
 #endif

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Dec 04 2010
next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=5319


Brad Roberts <braddr puremagic.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |braddr puremagic.com
         Resolution|                            |FIXED


--- Comment #1 from Brad Roberts <braddr puremagic.com> 2011-02-06 23:31:38 PST
---
https://github.com/D-Programming-Language/druntime/commit/1a7482679b8cf5df10d3b002beb51a285764a2a7

I used the version identifier that dmd uses, "Solaris".  I hope that's what GDC
and LDC use.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 06 2011
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=5319



--- Comment #2 from Brad Roberts <braddr puremagic.com> 2011-02-06 23:32:56 PST
---
And the same change to critical_.d:

https://github.com/D-Programming-Language/druntime/commit/f33862c7b6084af61e79f6ab73ff86d97833221b

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 06 2011
prev sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=5319



--- Comment #3 from Iain Buclaw <ibuclaw ubuntu.com> 2011-02-07 09:29:23 PST ---
Solaris isn't supported in core.sys.posix.pthreads though, is it? (Admittedly I
have a largely unfinished patch to add Solaris support in druntime lingering in
a VM somewhere, so maybe not too much of a worry).

Regards

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 07 2011