Re: pgAdmin 4 - Update for Server dialog

From: Susan Douglas <susan(dot)douglas(at)enterprisedb(dot)com>
To: Dave Page <dave(dot)page(at)enterprisedb(dot)com>, pgadmin-hackers(at)postgresql(dot)org
Cc: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>, Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Subject: Re: pgAdmin 4 - Update for Server dialog
Date: 2017-12-18 10:51:11
Message-ID: 63A30F0D-83C6-4A9E-85B3-7DBDC676BE55@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi All,

Khushboo has helped generate the attached patch for the server_dialog.rst and pgadmin_tabbed_browser.rst changes. When generated on my system, the patch fails, but when generated on her system, the patch applies. In both cases, we're using the same files and commands to generate the patch.

Let me know if there are any further problems!

Thanks!

-- Susan

> On Dec 18, 2017, at 4:11 AM, Dave Page <dave(dot)page(at)enterprisedb(dot)com> wrote:
>
> Hi
>
> For some reason I usually find Susan's patches will only apply using "patch -p1" and not "git apply". It's very odd.
>
> In this case, patch seems to think it's reversed (i.e. already applied):
>
> piranha:pgadmin4 dpage$ patch -p1 < ~/Downloads/server_dialog.patch
> (Stripping trailing CRs from patch.)
> patching file docs/en_US/server_dialog.rst
> Reversed (or previously applied) patch detected! Assume -R? [n]
>
> On Fri, Dec 15, 2017 at 4:48 AM, Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com <mailto:khushboo(dot)vashi(at)enterprisedb(dot)com>> wrote:
> Hi Susan,
>
> Now, getting another error as attached.
>
> Thanks,
> Khushboo
>
> On Thu, Dec 14, 2017 at 5:44 PM, Susan Douglas <susan(dot)douglas(at)enterprisedb(dot)com <mailto:susan(dot)douglas(at)enterprisedb(dot)com>> wrote:
>
> > On Dec 13, 2017, at 11:39 PM, Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com <mailto:khushboo(dot)vashi(at)enterprisedb(dot)com>> wrote:
> >
> > Hi susan,
> >
> > I am getting an error while applying the patch, attached the screen-shot for the same.
> >
> > Thanks,
> > Khushboo
> >
>
> Hi Khushboo,
>
> The error message is complaining about line 3 (GIT binary patch), so I regenerated the patch without the --cached --binary directives... it's attached below. Can you try this patch?
>
> -- Susan
>
>
>
>
>
>
> --
> Dave Page
> VP, Chief Architect, Tools & Installers
> EnterpriseDB: http://www.enterprisedb.com <http://www.enterprisedb.com/>
> The Enterprise PostgreSQL Company
>
> Blog: http://pgsnake.blogspot.com <http://pgsnake.blogspot.com/>
> Twitter: @pgsnake

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2017-12-18 11:00:15 pgAdmin 4 commit: Doc updates for server connection dialogue and relate
Previous Message Dave Page 2017-12-18 10:36:01 pgAdmin 4 commit: Bump version for some dev builds.