Re: PLUGINS Functionlity in Win32 build scripts

Поиск
Список
Период
Сортировка
Искать

Re: PLUGINS Functionlity in Win32 build scripts

От:
Alvaro Herrera <alvherre@commandprompt.com>
Дата:

Re: PLUGINS Functionlity in Win32 build scripts

От:
Tom Lane <tgl@sss.pgh.pa.us>
Дата:

Re: PLUGINS Functionlity in Win32 build scripts

От:
Tom Lane <tgl@sss.pgh.pa.us>
Дата:

Re: PLUGINS Functionlity in Win32 build scripts

От:
Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>
Дата:

Re: PLUGINS Functionlity in Win32 build scripts

От:
Peter Eisentraut <peter_e@gmx.net>
Дата:

Re: PLUGINS Functionlity in Win32 build scripts

От:
"Robert Haas" <robertmhaas@gmail.com>
Дата:

Re: PLUGINS Functionlity in Win32 build scripts

От:
"Robert Haas" <robertmhaas@gmail.com>
Дата:

Re: PLUGINS Functionlity in Win32 build scripts

От:
"Robert Haas" <robertmhaas@gmail.com>
Дата:

Re: PLUGINS Functionlity in Win32 build scripts

От:
MUHAMMAD ASIF <anaeem.it@hotmail.com>
Дата:
Robert,

I am sorry about this, This missed my attention. Here is my reply to your mail.
> Date: Fri, 28 Nov 2008 08:42:39 -0500
> From: robertmhaas@gmail.com
> To: anaeem.it@hotmail.com
> Subject: Re: [HACKERS] PLUGINS Functionlity in Win32 build scripts
> CC: pgsql-hackers@postgresql.org
>
> On Tue, Sep 9, 2008 at 3:04 AM, MUHAMMAD ASIF <anaeem.it@hotmail.com> wrote:
> > I am currently working in EnterpriseDB (www.enterprisedb.com). During the integration of pldebugger ( http://pgfoundry.org/projects/edb-debugger ) with postgres on windows I faced a problem that plugins are not being copied to the lib/plugins directory. Plugins should be copied in (Installation dir)lib/plugins to work properly.
> > To solve this problem I added PLUGINS logic in the Windows Perl build scripts of PostgreSQL 8.3.3. It searches for PLUGINS variable in the contrib Makefile and processes "PLUGINS" as "MODULES" and copies the generated plugin library to the (Installation dir)lib/plugins.
> > Please find the attached plugin.patch file. Thanks.
>
> Hi,
>
> I've been assigned to review this patch, but I confess I'm a little
> murky on what problem it's trying to solve. Can you explain what I
> need to do to recreate the problem?
 

In postgreSQL, Plugin modules should be installed in (Installation dir)lib/plugins to run properly. There is logic in src/makefiles/pgxs.mk for handling shared objects implemented as e.g. "Modules" variable that are installed in (Installation dir)/lib. There should be some way to handle plugins as well. We faced this issue during the integration of contrib module pldebugger ( http://pgfoundry.org/frs/?group_id=1000175 ) with the edb-postgresql code. pldebugger/Makefile handles plugins by itself, Unix/Linux don’t mind it because it build through make. On windows, perl scripts are used to build the system that relies on Makefiles that uses pgxs.mk variables e.g "Modules" etc. It fails to build pldebugger that implements its own plugin build logic in its Makefile. Right now for us there is only one plugin module but in future there may be more.Instead of any workaround we come up to a better solution to handle plugins automatically by pgxs.mk as other shared objects are being handled on Unix and Windows. We added PLUGIN logic in pgxs.mk and windows perl build scripts so that there is no need to handle plugins separately by new contrib modules.  
>
> There's also this comment on the Wiki (not sure why it wasn't posted
> to the mailing list...): "Dave Page says: This doesn't work with the
> edb-debugger plugin, which is the only such plugin around AFAIK. It
> needs to ignore comments on the PLUGINS line, and handle multiple
> targets (plugin_debugger, pldbgapi, targetinfo etc). Not sure if we
> want that complexity though." Any comments?
 

Dave was refering pldebugger code that uses comments in its Makefile.

pldebugger/Makefile : Line # 36

PLUGINS = plugin_debugger # plugin_profiler plugin_tracer

 

Make tool ignores the comments but windows perl scripts don’t understand "#" in process of extracting module names.

It is a separate functionality to enhance the scripts so that it can understand commented code with "#".I would love to make these changes separately but it don’t seems to be in the scope of PLUGINS functionality.  
>
> Thanks,
>
> ...Robert
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
 

I am sending you patches that i tested with postgresql-8.3.4. I am looking forward to your quries :).Thanks.

 

Best Regards,

M. Asif Naeem

www.enterprisedb.com

 


Discover the new Windows Vista Learn more!

Re: PLUGINS Functionlity in Win32 build scripts

От:
MUHAMMAD ASIF <anaeem.it@hotmail.com>
Дата:

PLUGINS Functionlity in Win32 build scripts

От:
MUHAMMAD ASIF <anaeem.it@hotmail.com>
Дата:


Hi,
      I am currently working in EnterpriseDB (www.enterprisedb.com). During the integration of pldebugger ( http://pgfoundry.org/projects/edb-debugger ) with postgres on windows  I faced a problem that plugins are not being copied to the lib/plugins directory. Plugins should be copied in (Installation dir)lib/plugins to work properly.
      To solve this problem I added PLUGINS logic in the Windows Perl build scripts of PostgreSQL 8.3.3. It searches for PLUGINS variable in the contrib Makefile and processes "PLUGINS" as "MODULES" and copies the generated plugin library to the (Installation dir)lib/plugins.
Please find the attached plugin.patch file. Thanks.

Best Regards,
Asif Naeem
www.enterprisedb.com

_________________________________________________________________
News, entertainment and everything you care about at Live.com. Get it now!
http://www.live.com/getstarted.aspx

Re: PLUGINS Functionlity in Win32 build scripts

От:
MUHAMMAD ASIF <anaeem.it@hotmail.com>
Дата:

Re: PLUGINS Functionlity in Win32 build scripts

От:
MUHAMMAD ASIF <anaeem.it@hotmail.com>
Дата:
Please checkout the patch that adds the PLUGINS functionality in src/makefiles/pgxs.mk it covers the Unix side of functionality. Thanks.

> Date: Sat, 13 Sep 2008 10:33:30 +0300
> To: anaeem.it@hotmail.com
> CC: pgsql-hackers@postgresql.org
> Subject: Re: [HACKERS] PLUGINS Functionlity in Win32 build scripts
> From: heikki.linnakangas@enterprisedb.com
>
> MUHAMMAD ASIF wrote:
> > During the integration of pldebugger ( http://pgfoundry.org/projects/edb-debugger ) with postgres on windows I faced a problem that plugins are not being copied to the lib/plugins directory. Plugins should be copied in (Installation dir)lib/plugins to work properly.
> > To solve this problem I added PLUGINS logic in the Windows Perl build scripts of PostgreSQL 8.3.3. It searches for PLUGINS variable in the contrib Makefile and processes 'PLUGINS' as 'MODULES' and copies the generated plugin library to the (Installation dir)lib/plugins.
> > Please find the attached plugin.patch file. Thanks.
>
> We'll need the same logic on Unix-platforms as well.
>
> I've added this to the November commitfest Wiki page.
>
> --
> Heikki Linnakangas
> EnterpriseDB http://www.enterprisedb.com
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers


Connect to the next generation of MSN Messenger  Get it now!

Re: PLUGINS Functionlity in Win32 build scripts

От:
MUHAMMAD ASIF <anaeem.it@hotmail.com>
Дата:

Purpose of this Plugins change is to add support in postgresql source to build plugin and install in (install dir)/lib/plugins. We made these changes for a plugin module i.e. pldebugger and we found this a useful feature that can help postgresql in adding plugins easily.
Attached file (foo.zip) is a test contrib module code that can be used to test PLUGINS logic ( patches.zip patch I have sent you in the last email). Following are the steps to build foo.zip i.e
    a. Apply patches plugins_make.patch and plugins_msvc.patch.
    b. Unzip and copy the foo directory in the contrib directory
    c. Build and Install the system
It will build foo library and place it in lib/plugins directory.You can test it on both windows and linux/unix platforms.Thanks.

Best Regards,
Asif Naeem

> Date: Tue, 9 Dec 2008 23:36:42 -0500
> From: robertmhaas@gmail.com
> To: anaeem.it@hotmail.com
> Subject: Re: [HACKERS] PLUGINS Functionlity in Win32 build scripts
> CC: pgsql-hackers@postgresql.org
> 
>>> I've been assigned to review this patch, but I confess I'm a little
>>> murky on what problem it's trying to solve. Can you explain what I
>>> need to do to recreate the problem?
>> In postgreSQL, Plugin modules should be installed in (Installation
>> dir)lib/plugins to run properly. There is logic in src/makefiles/pgxs.mk for
>> handling shared objects implemented as e.g. "Modules" variable that are
>> installed in (Installation dir)/lib. There should be some way to handle
>> plugins as well. We faced this issue during the integration of contrib
>> module pldebugger ( http://pgfoundry.org/frs/?group_id=1000175 ) with the
>> edb-postgresql code. pldebugger/Makefile handles plugins by itself,
>> Unix/Linux don't mind it because it build through make. On windows, perl
>> scripts are used to build the system that relies on Makefiles that uses
>> pgxs.mk variables e.g "Modules" etc. It fails to build pldebugger that
>> implements its own plugin build logic in its Makefile. Right now for us
>> there is only one plugin module but in future there may be more.Instead of
>> any workaround we come up to a better solution to handle plugins
>> automatically by pgxs.mk as other shared objects are being handled on Unix
>> and Windows. We added PLUGIN logic in pgxs.mk and windows perl build scripts
>> so that there is no need to handle plugins separately by new contrib
>> modules.
> 
> I am a bit dubious about the idea of adding supposedly generic
> functionality to cater to a single client, but the bigger problem is
> that even after reading this I still don't really know what I'm
> supposed to be looking at.  I think you're saying that pldebugger
> contains some code that could be made simpler and less easily broken
> if this patch were applied.  Is that correct?  If so, please tell me
> where to download the pldebugger code and which file(s) to look in for
> the code that could be improved.
> 
> ...Robert
> 
> -- 
> Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers

_________________________________________________________________
Discover the new Windows Vista
http://search.msn.com/results.aspx?q=windows+vista&mkt=en-US&form=QBRE

Re: PLUGINS Functionlity in Win32 build scripts

От:
MUHAMMAD ASIF <anaeem.it@hotmail.com>
Дата:
FAQ