From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Thomas Hallgren <thhal(at)mailblocks(dot)com> |
Cc: | PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] dynamic_library_path on Win32 |
Date: | 2004-06-10 02:49:06 |
Message-ID: | 200406100249.i5A2n6m28775@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Your patch has been added to the PostgreSQL unapplied patches list at:
http://momjian.postgresql.org/cgi-bin/pgpatches
I will try to apply it within the next 48 hours.
I will try to clean up the path/dir confusion too.
---------------------------------------------------------------------------
Thomas Hallgren wrote:
> Attached is a patch that takes care of the PATHSEP issue. I made a more
> extensive change then what was suggested. I found the file path.c that
> contained a lot of "Unix/Windows" agnostic functions so I added a function
> there instead and removed the PATHSEP declaration in exec.c altogether. All
> to keep things from scattering all over the code.
>
> I also took the liberty of changing the name of the functions
> "first_path_sep" and "last_path_sep". Where I come from (and I'm apparently
> not alone given the former macro name PATHSEP), they should be called
> "first_dir_sep" and "last_dir_sep". The new function I introduced, that
> actually finds path separators, is now the "first_path_sep". The patch
> contains changes on all affected places of course.
>
> I also changed the documentation on dynamic_library_path to reflect the
> chagnes.
>
> Kind regards,
>
> Thomas Hallgren
>
>
> "Bruce Momjian" <pgman(at)candle(dot)pha(dot)pa(dot)us> wrote in message
> news:200405300307(dot)i4U37sE17714(at)candle(dot)pha(dot)pa(dot)us(dot)(dot)(dot)
> >
> > I can do it but will be a few days until I get to it.
> >
> > --------------------------------------------------------------------------
> -
> >
> > Tom Lane wrote:
> > > Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
> > > > You've found a bug. Clearly we need to adjust the parsing of
> > > > dynamic_library_path and probably preload_libraries for Win32.
> > >
> > > Yup. Using PATHSEP sounded reasonable to me. Any volunteer to fix
> > > this? (Don't forget to patch the docs for these variables, too.)
> > >
> > > regards, tom lane
> > >
> > > ---------------------------(end of broadcast)---------------------------
> > > TIP 4: Don't 'kill -9' the postmaster
> > >
> >
> > --
> > Bruce Momjian | http://candle.pha.pa.us
> > pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
> > + If your life is a hard drive, | 13 Roberts Road
> > + Christ can be your backup. | Newtown Square, Pennsylvania
> 19073
> >
> > ---------------------------(end of broadcast)---------------------------
> > TIP 6: Have you searched our list archives?
> >
> > http://archives.postgresql.org
> >
[ Attachment, skipping... ]
--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
From | Date | Subject | |
---|---|---|---|
Next Message | Joshua D. Drake | 2004-06-10 02:54:25 | Re: pg_dump and schema namespace notes |
Previous Message | Bruce Momjian | 2004-06-10 02:42:43 | Re: Re : Win32 binaries test / pg_dump problem |