Re: Some additions/#ifdefs to target new OS NetWare

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Ulrich Neumann <u_neumann(at)gne(dot)de>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Some additions/#ifdefs to target new OS NetWare
Date: 2002-03-04 15:27:29
Message-ID: 4018.1015255649@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Ulrich Neumann <u_neumann(at)gne(dot)de> writes:
> During my work ive found the following that need to be added to
> the sources in order to compile postgreSQL on NetWare.

We'd appreciate a patch (diff -c format), not random snippets of code.

> 2. in bootstrap.c/cleanup()
> #ifdef N_PLAT_NLM
> NWCleanUp();
> #endif

Unlikely to be the right place for it, if it's needed at all which I
doubt. (Surely NetWare can manage to provide a *standard* C execution
environment, in which any platform-specific startup and cleanup stuff
is done in the C library?)

> 4. in fd.c/filepath()

> #ifdef N_PLAT_NLM
> buf = (char *) palloc(strlen(filename) + 1);
> strcpy(buf, filename);
> #else

I don't believe this either.

> 7. in dynamic_loader.h
> #ifndef N_PLAT_NLM
> extern void *pg_dlopen(char *filename);
> extern PGFunction pg_dlsym(void *handle, char *funcname);
> extern void pg_dlclose(void *handle);
> extern char *pg_dlerror(void);
> #endif

Nope. Make a platform-specific implementation of pg_dlopen and friends,
just like all the other platforms have done.

regards, tom lane

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2002-03-04 16:19:11 Re: permanent EuroDates
Previous Message Tom Lane 2002-03-04 15:08:13 Re: Bug or Feature?