public inbox for [email protected]
help / color / mirror / Atom feedFrom: Dave Page <[email protected]>
To: Andreas 'ads' Scherbaum <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: Updated Greenplum documentation
Date: Fri, 8 Jan 2016 15:41:42 +0000
Message-ID: <CA+OCxowz4sc6asb+QKyDmke5uZC2_DqzPaObN+i_iOg-Wuw1Sg@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
<CA+OCxoxtX52RXdaZor2pWis8TAt__snK4-uA5phnaq_ODnmP1Q@mail.gmail.com>
<[email protected]>
<CA+OCxozYRB-wbzCaSFD56_h8-Ti4w00isOeQJCzzffyTnUXgfA@mail.gmail.com>
<[email protected]>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgadmin-hackers>
On Fri, Jan 8, 2016 at 6:15 AM, Andreas 'ads' Scherbaum
<[email protected]> wrote:
> On 07.01.2016 15:45, Dave Page wrote:
>>
>> On Thu, Jan 7, 2016 at 2:43 PM, Andreas 'ads' Scherbaum
>> <[email protected]> wrote:
>>>
>>> On 07.01.2016 13:49, Dave Page wrote:
>>>>
>>>>
>>>>
>>>> What did you create the patch against?
>>>
>>>
>>>
>>> HEAD from git.postgresql from like 2 days ago.
>>
>>
>> Hmm, I would have expected that to work. Can you rebase it please, just in
>> case?
>>
>
> Just did that, and recreated the patch. Latest changes did not touch the
> files I modified, the new patch has the same checksum.
> Also, I can apply my patch to my working directory without any errors.
>
> Any idea why it is failing for you? Any reject files?
I get the following rejects (see attached as well). I wonder if it's a
line ending issue - what platform are you working on?
piranha:pgadmin3 dpage$ git apply --reject ~/Downloads/gp-doc.diff
/Users/dpage/Downloads/gp-doc.diff:9: trailing whitespace.
path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-5.0\\bin"));
/Users/dpage/Downloads/gp-doc.diff:10: trailing whitespace.
path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-4.4\\bin"));
/Users/dpage/Downloads/gp-doc.diff:18: trailing whitespace.
path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-5.0\\lib"));
/Users/dpage/Downloads/gp-doc.diff:19: trailing whitespace.
path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-4.4\\lib"));
/Users/dpage/Downloads/gp-doc.diff:27: trailing whitespace.
path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-5.0\\bin"));
Checking patch pgadmin/pgAdmin3.cpp...
error: while searching for:
// Ugly... Greenplum client releases have no predictable numbers,
because the path is the server version
if (!programFiles.IsEmpty())
{
path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-4.3\\bin"));
path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-4.2\\bin"));
path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-4.1\\bin"));
error: patch failed: pgadmin/pgAdmin3.cpp:1167
error: while searching for:
path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-3.3\\bin"));
path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-3.2\\bin"));
path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-4.3\\lib"));
path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-4.2\\lib"));
path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-4.1\\lib"));
error: patch failed: pgadmin/pgAdmin3.cpp:1174
error: while searching for:
if (!programFilesX86.IsEmpty())
{
path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-4.3\\bin"));
path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-4.2\\bin"));
path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-4.1\\bin"));
error: patch failed: pgadmin/pgAdmin3.cpp:1184
error: while searching for:
path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-3.3\\bin"));
path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-3.2\\bin"));
path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-4.3\\lib"));
path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-4.2\\lib"));
path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-4.1\\lib"));
error: patch failed: pgadmin/pgAdmin3.cpp:1191
error: while searching for:
// Generic Unix paths
path.Add(wxT("/usr/local/greenplum-clients-4.3/bin"));
path.Add(wxT("/opt/local/greenplum-clients-4.3/bin"));
path.Add(wxT("/usr/local/greenplum-clients-4.2/bin"));
error: patch failed: pgadmin/pgAdmin3.cpp:1206
error: while searching for:
path.Add(wxT("/usr/local/greenplum-clients-3.2/bin"));
path.Add(wxT("/opt/local/greenplum-clients-3.2/bin"));
path.Add(wxT("/usr/local/greenplum-clients-4.3/lib"));
path.Add(wxT("/opt/local/greenplum-clients-4.3/lib"));
path.Add(wxT("/usr/local/greenplum-clients-4.2/lib"));
error: patch failed: pgadmin/pgAdmin3.cpp:1219
error: while searching for:
if (edbHelpPath.IsEmpty())
edbHelpPath = wxT("http://www.enterprisedb.com/docs/en/current/server/";);
if (gpHelpPath.IsEmpty())
gpHelpPath = wxT("http://docs.gopivotal.com/gpdb/index.html";);
if (slonyHelpPath.IsEmpty())
slonyHelpPath = wxT("http://www.slony.info/documentation/";);
error: patch failed: pgadmin/pgAdmin3.cpp:1664
Checking patch pgadmin/utils/misc.cpp...
error: while searching for:
thePath.Lower().EndsWith(wxT(".zip")))
return thePath;
// In all othe cases we must have a directory
wxString sep;
// Figure out the appropriate seperator
error: patch failed: pgadmin/utils/misc.cpp:676
error: while searching for:
// the old help path (stored in the settings) is no longer working
static wxString gpHelpPath = settings->GetGpHelpPath();
if (gpHelpPath.CmpNoCase(wxT("http://www.greenplum.com/docs/3300/";)) == 0)
{
gpHelpPath = wxT("http://docs.gopivotal.com/gpdb/";);
// this is the old link, update the link to the new documentation link
// problem: this saves the link into the configuration file
settings->SetGpHelpPath(gpHelpPath);
error: patch failed: pgadmin/utils/misc.cpp:764
Applying patch pgadmin/pgAdmin3.cpp with 7 rejects...
Rejected hunk #1.
Rejected hunk #2.
Rejected hunk #3.
Rejected hunk #4.
Rejected hunk #5.
Rejected hunk #6.
Rejected hunk #7.
Applying patch pgadmin/utils/misc.cpp with 2 rejects...
Rejected hunk #1.
Rejected hunk #2.
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
--
Sent via pgadmin-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers
Attachments:
[application/octet-stream] pgAdmin3.cpp.rej (3.8K, 2-pgAdmin3.cpp.rej)
download
[application/octet-stream] misc.cpp.rej (1.5K, 3-misc.cpp.rej)
download
view thread (9+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected]
Subject: Re: Updated Greenplum documentation
In-Reply-To: <CA+OCxowz4sc6asb+QKyDmke5uZC2_DqzPaObN+i_iOg-Wuw1Sg@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox