| From: | Dave Page <dpage(at)pgadmin(dot)org> |
|---|---|
| To: | Adam Scott <adam(dot)c(dot)scott(at)gmail(dot)com> |
| Cc: | Magnus Hagander <magnus(at)hagander(dot)net>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
| Subject: | Re: Patch: New field in frmMain statusbar |
| Date: | 2015-09-24 15:25:38 |
| Message-ID: | CA+OCxowkgw+ppRhRYUqfqrsGYpHPOFtjYWvqqAn+3HjxV4UhqA@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgadmin-hackers |
Thanks - patch applied!
On Wed, Sep 23, 2015 at 8:15 PM, Adam Scott <adam(dot)c(dot)scott(at)gmail(dot)com> wrote:
> Thank you Dave! I rebased from master. To get it to build is going to
> take some research. It looks like it wants openssl, which I grabbed and
> configured for WIN32, but it's not happy. First error is below. But this
> patch is so straightforward, I'm sharing anyway for the sake of time, as it
> was working before.
>
> 2>c:\pgadmin3\pgadmin\libssh2\openssl.c(71): error C2037: left of 'e'
> specifies undefined struct/union 'rsa_st'
>
> Thanks again!
>
> Adam
>
>
> On Wed, Sep 23, 2015 at 8:35 AM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>> On Wed, Sep 23, 2015 at 2:55 PM, Adam Scott <adam(dot)c(dot)scott(at)gmail(dot)com>
>> wrote:
>> > Is the last patch okay? If there's something more required or anything
>> > please let me know.
>>
>> Hi
>>
>> The patch does not apply for me. Can you rebase it against master please?
>>
>> Thanks.
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Dave Page | 2015-09-24 15:29:32 | pgAdmin III commit: Make style run. |
| Previous Message | Dave Page | 2015-09-24 15:25:18 | pgAdmin III commit: Display the connection details for the current brow |