On Mon, May 7, 2012 at 12:32 PM, Heikki Linnakangas
> On 07.05.2012 13:54, Dave Page wrote:
>> On Mon, May 7, 2012 at 11:46 AM, Heikki Linnakangas
>> <heikki(dot)linnakangas(at)enterprisedb(dot)com> wrote:
>>> On 07.05.2012 13:39, Dave Page wrote:
>>>> Can you create a "PRE-9_2" branch for the current architecture, and
>>>> apply Ashesh's patch to that if you think it's a reasonable fix
>>>> please? We can then move the older builds to that branch. Then the 9.2
>>>> changes can go on master.
>>> By current architecture, do you mean what we had in the CVS, or as the
>>> stands in the current tip of the git repository? I agree we should create
>>> branch like that, but I want to make sure we agree on where it should be
>>> branched off. I've rearranged and cleaned up the code quite extensively
>>> git master already, and I'm not sure if we want to include all that in
>>> back-branches. On one hand, I've fixed a lot of bugs with the changes, so
>>> would be nice to include those fixes, but OTOH, perhaps we shouldn't risk
>>> destabilizing back-branches.
>> What's at the current tip. I'm happy with those changes in the back
>> branches - they're really mostly just bug fixes and relatively minor
>> architectural tweaks as I understood them. It's the move to a single
>> library that I don't want in the back branches.
> Ok. I created branch PRE_9_2, and committed Ashesh's patch there. In master,
> I committed a patch to merge pldbgapi.so and plugin_debugger.so.
Ashesh; can you please update the back branch builds to use that
branch of the debugger code? Feel free to ask Dhiraj/Akshay/Dharam to
do it as I know you and Sachin are pretty busy right now.
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
In response to
pgadmin-hackers by date
|Next:||From: Sachin Srivastava||Date: 2012-05-08 12:37:21|
|Subject: Re: Re: PATCH: The linker complains about the
'-bundle_loader "../../src/backend/postgres"' against PostgreSQL 8.4 on OSX|
|Previous:||From: Clemens Eisserer||Date: 2012-05-07 21:55:40|
|Subject: Re: Limit the size of explainPopup|