Re: [perl #24177] [PATCH] Make Parrot dlcompat aware on OS X

2005-09-27 Thread Michael Scott

Sorry I was on holiday. Yes this can be closed.

On 21 Sep 2005, at 09:45, Joshua Hoblitt via RT wrote:


[mikescott - Thu Oct 09 11:49:45 2003]:

If someone happens to have dlcompat

http://www.opendarwin.org/projects/dlcompat/

installed on OS X then the following patch will let Parrot be  
aware of

this and use it.

The following changes are made:

config/auto/headers.pl
Add dlfcn.h to @extra_headers so it will be detected even if  
dlcompat

has been installed after Perl.

config/gen/platform/darwin.c
Add conditional code for PARROT_HAS_HEADER_DLFCN.

config/init/hints/darwin.pl
Add values for libs and so.

dynext.c
Replace SO_EXTENSION with PARROT_DLL_EXTENSION which is already
defined in config.h.

nci_test.c
Remove #include malloc.c because it is unused.

t/pmc/nci.t
Remove the $PConfig{jitcpuarch} eq 'i386' condition from the SKIP
conditional.

conf/gen/makefiles/root.in
Change libnci.so target to libnci and use $(SO).

Running

make libnci
perl t/harness --running-make-test --gc-debug t/pmc/nci.t

passes all tests with this patch applied.







Is this still an issue?

-J






Re: [perl #24177] [PATCH] Make Parrot dlcompat aware on OS X

2003-10-10 Thread Leopold Toetsch
Michael Scott [EMAIL PROTECTED] wrote:
 config/gen/platform/darwin.c
   Add conditional code for PARROT_HAS_HEADER_DLFCN.

Could you please rediff the patch w/o the whitespace changes. An indent
of 4 is ok, you seem to have 8 in place.

Thanks.

leo


Re: [perl #24177] [PATCH] Make Parrot dlcompat aware on OS X

2003-10-10 Thread Dan Sugalski
On Fri, 10 Oct 2003, Leopold Toetsch wrote:

 Michael Scott [EMAIL PROTECTED] wrote:
  config/gen/platform/darwin.c
  Add conditional code for PARROT_HAS_HEADER_DLFCN.

 Could you please rediff the patch w/o the whitespace changes. An indent
 of 4 is ok, you seem to have 8 in place.

Actually, I'd rather not do this. We linked against Fink under OS X
originally, to get library loading, but ended up not doing it any more
because too many people didn't have it, and it seemed to mess up some
people's systems.

I'll try and get the alternate dynaloader for OS X built this weekend.

Dan


Re: [perl #24177] [PATCH] Make Parrot dlcompat aware on OS X

2003-10-10 Thread Michael Scott
On Friday, Oct 10, 2003, at 14:22 Europe/Berlin, Dan Sugalski wrote:

On Fri, 10 Oct 2003, Leopold Toetsch wrote:

Michael Scott [EMAIL PROTECTED] wrote:
config/gen/platform/darwin.c
Add conditional code for PARROT_HAS_HEADER_DLFCN.
Could you please rediff the patch w/o the whitespace changes. An 
indent
of 4 is ok, you seem to have 8 in place.
Actually, I'd rather not do this. We linked against Fink under OS X
originally, to get library loading, but ended up not doing it any more
because too many people didn't have it, and it seemed to mess up some
people's systems.
I thought you might say that, which was why I made it a 
well-if-it's-there kind of thing.

I really just wanted to see the ncurses life example working, but that 
died with the following dlcompat error anyway:

	unable to open this file with RTLD_LOCAL

because it doesn't implement RTLD_LOCAL.

I'll try and get the alternate dynaloader for OS X built this weekend.
If I can be of any assistance with this let me know.

	Dan




[perl #24177] [PATCH] Make Parrot dlcompat aware on OS X

2003-10-09 Thread via RT
# New Ticket Created by  Michael Scott 
# Please include the string:  [perl #24177]
# in the subject line of all future correspondence about this issue. 
# URL: http://rt.perl.org/rt2/Ticket/Display.html?id=24177 


If someone happens to have dlcompat

http://www.opendarwin.org/projects/dlcompat/

installed on OS X then the following patch will let Parrot be aware of 
this and use it.

The following changes are made:

config/auto/headers.pl
Add dlfcn.h to @extra_headers so it will be detected even if dlcompat 
has been installed after Perl.

config/gen/platform/darwin.c
Add conditional code for PARROT_HAS_HEADER_DLFCN.

config/init/hints/darwin.pl
Add values for libs and so.

dynext.c
Replace SO_EXTENSION with PARROT_DLL_EXTENSION which is already 
defined in config.h.

nci_test.c
Remove #include malloc.c because it is unused.

t/pmc/nci.t
Remove the $PConfig{jitcpuarch} eq 'i386' condition from the SKIP 
conditional.

conf/gen/makefiles/root.in
Change libnci.so target to libnci and use $(SO).

Running

make libnci
perl t/harness --running-make-test --gc-debug t/pmc/nci.t

passes all tests with this patch applied.



-- attachment  1 --
url: http://rt.perl.org/rt2/attach/65884/49191/1922d1/dlcompat.patch



dlcompat.patch
Description: dlcompat.patch