Re: BUG #6722: Debugger broken?

On 13.08.2012 10:42, Dave Page wrote:
> On Mon, Aug 13, 2012 at 7:22 AM, Heikki Linnakangas
> <heikki(dot)linnakangas(at)enterprisedb(dot)com> wrote:
>> On 09.08.2012 17:55, Dave Page wrote:
>>>
>>> On Thu, Aug 9, 2012 at 3:51 PM, Heikki Linnakangas
>>> <heikki(dot)linnakangas(at)enterprisedb(dot)com> wrote:
>>>>
>>>> Ok, I see. I just downloaded the 9.1.4 installer, and you're right, there
>>>> is
>>>> no pldbgapi.control file anywhere. Looking at the README.pldebugger file,
>>>> the version of pldebugger that the installer includes certainly should
>>>> include that file.
>>>>
>>>> Dave, is this a packaging issue? Which version of pldebugger was included
>>>> in the 9.1.4 installers?
>>>
>>> It uses the head of the PRE_9_2 branch.
>>
>> Ok. So, why is pldbgapi.control file not included? It's there in the
>> repository, and "make install" copies it to share/extension.
>
> If it's there in the repo, then it shouldn't be. The PRE_9_2 branch of
> code is supposed to be how things looked *before* you started your
> work - it's the stable branch that we're building the< 9.2 installers
> from, and certainly shouldn't have been "extensionised", or had any
> other changes made to it except for bug fixes.

Then that was a misunderstanding when we discussed that back in May then
(http://archives.postgresql.org/pgadmin-hackers/2012-05/msg00022.php)
How do you prefer to resolve that now? The extensionised code in PRE_9_2
branch should work on 9.1, if you include all the right files in the
installer, including the control file. Or, we can create a new "real"
pre-9.2 branch at the point before I started any of this work. Or at the
point just before the commit that turned it into an extension. All the
commits prior to that seem fairly harmless, but would need to test that
it works on all supported platforms and server versions.

--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com