-Original Message-
From: Andreas Pflug [mailto:[EMAIL PROTECTED]
Sent: 18 May 2005 18:02
To: Dave Page
Cc: pgadmin-hackers@postgresql.org
Subject: Re: [pgadmin-hackers] SVN Commit by dpage: r4203 -
trunk/pgadmin3/xtra/pgagent/pgaevent
[EMAIL PROTECTED] wrote:
Author:
Author: dpage
Date: 2005-05-19 09:31:32 +0100 (Thu, 19 May 2005)
New Revision: 4213
Modified:
trunk/pgadmin3/acinclude.m4
Log:
Attached is a new acinclude.m4, and a patch to our current acinclude,
that relies on wx-config for all the flags. Gone is all the
complicated stuff. The only
-Original Message-
From: [EMAIL PROTECTED]
[mailto:[EMAIL PROTECTED] On Behalf Of
Adam H. Pendleton
Sent: 18 May 2005 22:26
To: pgadmin-hackers@postgresql.org
Subject: [pgadmin-hackers] New acinclude.m4
Attached is a new acinclude.m4, and a patch to our current
acinclude,
Author: dpage
Date: 2005-05-19 10:27:24 +0100 (Thu, 19 May 2005)
New Revision: 4214
Modified:
trunk/pgadmin3/xtra/pgagent/job.cpp
trunk/pgadmin3/xtra/pgagent/pgagent.sql
Log:
Rename jslstarted - jslstart for consistency
Modified: trunk/pgadmin3/xtra/pgagent/job.cpp
Author: dpage
Date: 2005-05-19 10:42:43 +0100 (Thu, 19 May 2005)
New Revision: 4215
Modified:
trunk/pgadmin3/xtra/pgagent/job.cpp
trunk/pgadmin3/xtra/pgagent/pgagent.sql
Log:
No need to dup the dbname and code from the step into the step log.
Modified: trunk/pgadmin3/xtra/pgagent/job.cpp
Author: dpage
Date: 2005-05-19 10:53:11 +0100 (Thu, 19 May 2005)
New Revision: 4216
Modified:
trunk/pgadmin3/xtra/pgagent/job.cpp
Log:
Properly reference the job log from the step log
Modified: trunk/pgadmin3/xtra/pgagent/job.cpp
Author: dpage
Date: 2005-05-19 11:06:30 +0100 (Thu, 19 May 2005)
New Revision: 4217
Modified:
trunk/pgadmin3/xtra/pgagent/job.cpp
trunk/pgadmin3/xtra/pgagent/pgagent.sql
Log:
Include the step id in the step log
Modified: trunk/pgadmin3/xtra/pgagent/job.cpp
Author: dpage
Date: 2005-05-19 11:30:28 +0100 (Thu, 19 May 2005)
New Revision: 4218
Modified:
trunk/pgadmin3/src/agent/include/pgaJob.h
trunk/pgadmin3/src/agent/include/pgaStep.h
trunk/pgadmin3/src/agent/pgaJob.cpp
trunk/pgadmin3/src/agent/pgaStep.cpp
Log:
Display job step stats
On May 19, 2005, at 4:34 AM, Dave Page wrote:It's already mode 755 on a fresh checkout here.Hmmm, it seems that everything gets a mode of 644 when I use the "download tarball" option from the cvsweb interface (still waiting for a Fink binary for Tiger!).ahp
Author: dpage
Date: 2005-05-19 15:53:32 +0100 (Thu, 19 May 2005)
New Revision: 4219
Modified:
trunk/pgadmin3/xtra/pgagent/connection.cpp
trunk/pgadmin3/xtra/pgagent/include/connection.h
trunk/pgadmin3/xtra/pgagent/include/job.h
trunk/pgadmin3/xtra/pgagent/include/pgAgent.h
Dave Page wrote:
-Original Message-
From: [EMAIL PROTECTED]
[mailto:[EMAIL PROTECTED] On Behalf Of
Adam H. Pendleton
Sent: 18 May 2005 22:26
To: pgadmin-hackers@postgresql.org
Subject: [pgadmin-hackers] New acinclude.m4
Attached is a new acinclude.m4, and a patch to our current
Yes. Nice catch.
On May 19, 2005, at 1:07 PM, Raphaël Enrici wrote:
Dave Page wrote:
-Original Message-
From: [EMAIL PROTECTED]
[mailto:[EMAIL PROTECTED] On Behalf Of
Adam H. Pendleton
Sent: 18 May 2005 22:26
To: pgadmin-hackers@postgresql.org
Subject: [pgadmin-hackers] New
Message d'origine
Copie à: Dave Page dpage@vale-housing.co.uk,
De: Adam H.Pendleton [EMAIL PROTECTED]
Sujet: Re: [pgadmin-hackers] New acinclude.m4
Date: Thu, 19 May 2005 13:14:47 -0400
A: Raphaël Enrici [EMAIL PROTECTED]
Yes. Nice catch.
A small one compared to the work you did!
Raphaël Enrici wrote:
I have warning concerning ENABLE_STATIC in configure script. Shouldn't
we also remove it crom configure.ac ? (patch attached).
Does this mean the linking statically is not possible anymore?
Or has the check just moved to another place?
greetings, Florian Pflug
Florian G. Pflug wrote:
Raphaël Enrici wrote:
I have warning concerning ENABLE_STATIC in configure script. Shouldn't
we also remove it crom configure.ac ? (patch attached).
Does this mean the linking statically is not possible anymore?
Or has the check just moved to another place?
To me,
Raphaël Enrici wrote:
Florian G. Pflug wrote:
Raphaël Enrici wrote:
I have warning concerning ENABLE_STATIC in configure script. Shouldn't
we also remove it crom configure.ac ? (patch attached).
Does this mean the linking statically is not possible anymore?
Or has the check just moved to another
Hi Dave,
here is an update to the pkg/debian for REL-1_2_0_PATCHES branch. After
that and the latest acinclude.m4 discussion, it should be fine for me to
go for a release.
Cheers,
Raphaël
Index: pkg/debian/control
===
---
Florian G. Pflug wrote:
Raphaël Enrici wrote:
Florian G. Pflug wrote:
Raphaël Enrici wrote:
I have warning concerning ENABLE_STATIC in configure script. Shouldn't
we also remove it crom configure.ac ? (patch attached).
Does this mean the linking statically is not possible anymore?
Or has
On May 19, 2005, at 3:24 PM, Florian G. Pflug wrote:
Raphaël Enrici wrote:
Florian G. Pflug wrote:
Raphaël Enrici wrote:
I have warning concerning ENABLE_STATIC in configure script.
Shouldn't
we also remove it crom configure.ac ? (patch attached).
Does this mean the linking statically is not
On May 19, 2005, at 3:46 PM, Raphaël Enrici wrote:
Can you try the patch attached on OSX with a fresh checkout (don't
apply the configure.ac I just sent) ? (I need to rebuild wxWid to
generate static libs... it may take a moment).
Your attached patch isn't going to do anything. `wx-config
On May 19, 2005, at 2:13 PM, Florian G. Pflug wrote:
Raphaël Enrici wrote:
I have warning concerning ENABLE_STATIC in configure script.
Shouldn't
we also remove it crom configure.ac ? (patch attached).
Does this mean the linking statically is not possible anymore?
Or has the check just moved to
Adam H. Pendleton wrote:
On May 19, 2005, at 3:24 PM, Florian G. Pflug wrote:
Raphaël Enrici wrote:
Florian G. Pflug wrote:
Raphaël Enrici wrote:
I have warning concerning ENABLE_STATIC in configure script.
Shouldn't
we also remove it crom configure.ac ? (patch attached).
Does this mean the
Adam H. Pendleton wrote:
On May 19, 2005, at 3:46 PM, Raphaël Enrici wrote:
Can you try the patch attached on OSX with a fresh checkout (don't
apply the configure.ac I just sent) ? (I need to rebuild wxWid to
generate static libs... it may take a moment).
Your attached patch isn't
On May 19, 2005, at 4:30 PM, Raphaël Enrici wrote:
Adam H. Pendleton wrote:
I'm glad to be so rare... It seems I belong to the 1%: I have a
dynamic build. ;)
Please also note that the patch attached reintroduce your code
concerning the static link of the rest of the libs (libpq and sons).
However,
Adam H. Pendleton wrote:
On May 19, 2005, at 4:30 PM, Raphaël Enrici wrote:
Adam H. Pendleton wrote:
I'm glad to be so rare... It seems I belong to the 1%: I have a
dynamic build. ;)
Please also note that the patch attached reintroduce your code
concerning the static link of the rest of the
On May 19, 2005, at 4:54 PM, Raphaël Enrici wrote:
What I'm trying to say is this:
I've a dynamic only build. So, wx-config --libs is ok. wx-config
--static --libs gives an error.
According to what I understand from the wx-config --help output is:
if I had both built and would like to get
Raphaël Enrici wrote:
Adam H. Pendleton wrote:
On May 19, 2005, at 4:30 PM, Raphaël Enrici wrote:
Adam H. Pendleton wrote:
I'm glad to be so rare... It seems I belong to the 1%: I have a
dynamic build. ;)
Please also note that the patch attached reintroduce your code
concerning the static link of
To summarize :
- we reject the configure.ac.patch which removed ENABLE_STATIC and
should not have been included in trunk yet
- we keep the latest acinclude_static.patch which re-add support for
pg_static_build=yes + takes care of systems where wxWid installations
are both static dynamic.
On May 19, 2005, at 5:27 PM, Raphaël Enrici wrote:To summarize :- we reject the configure.ac.patch which removed ENABLE_STATIC andshould not have been included in trunk yetYou mean reject my entire patch? It did much more than just remove ENABLE_STATIC, so it should not be rejected.- we keep the
29 matches
Mail list logo
