public inbox for [email protected]
help / color / mirror / Atom feedFrom: Andreas 'ads' Scherbaum <[email protected]>
To: [email protected]
Subject: Re: Updated Greenplum documentation
Date: Thu, 14 Jan 2016 00:44:29 +0100
Message-ID: <[email protected]> (raw)
In-Reply-To: <CA+OCxowz4sc6asb+QKyDmke5uZC2_DqzPaObN+i_iOg-Wuw1Sg@mail.gmail.com>
References: <[email protected]>
<CA+OCxoxtX52RXdaZor2pWis8TAt__snK4-uA5phnaq_ODnmP1Q@mail.gmail.com>
<[email protected]>
<CA+OCxozYRB-wbzCaSFD56_h8-Ti4w00isOeQJCzzffyTnUXgfA@mail.gmail.com>
<[email protected]>
<CA+OCxowz4sc6asb+QKyDmke5uZC2_DqzPaObN+i_iOg-Wuw1Sg@mail.gmail.com>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgadmin-hackers>
On 08.01.2016 16:41, Dave Page wrote:
>
> 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"));
This complains about trailing whitespaces where I can't see any
whitespaces in my diff. Not sure why ...
Attached is an updated version of the diff, plus one with Mac line endings.
Regards,
--
Andreas 'ads' Scherbaum
German PostgreSQL User Group
European PostgreSQL User Group - Board of Directors
Volunteer Regional Contact, Germany - PostgreSQL Project
--
Sent via pgadmin-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers
Attachments:
[text/x-patch] gp-doc2.diff (5.4K, 2-gp-doc2.diff)
download | inline diff:
diff --git a/pgadmin/pgAdmin3.cpp b/pgadmin/pgAdmin3.cpp
index 3f9cf3f..87ee9f0 100644
--- a/pgadmin/pgAdmin3.cpp
+++ b/pgadmin/pgAdmin3.cpp
@@ -1167,6 +1167,8 @@ void pgAdmin3::InitXtraPaths()
// 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-5.0\\bin"));
+ path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-4.4\\bin"));
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"));
@@ -1174,6 +1176,8 @@ void pgAdmin3::InitXtraPaths()
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-5.0\\lib"));
+ path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-4.4\\lib"));
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"));
@@ -1184,6 +1188,8 @@ void pgAdmin3::InitXtraPaths()
if (!programFilesX86.IsEmpty())
{
+ path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-5.0\\bin"));
+ path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-4.4\\bin"));
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"));
@@ -1191,6 +1197,8 @@ void pgAdmin3::InitXtraPaths()
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-5.0\\lib"));
+ path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-4.4\\lib"));
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"));
@@ -1206,6 +1214,10 @@ void pgAdmin3::InitXtraPaths()
// Generic Unix paths
+ path.Add(wxT("/usr/local/greenplum-clients-5.0/bin"));
+ path.Add(wxT("/opt/local/greenplum-clients-5.0/bin"));
+ path.Add(wxT("/usr/local/greenplum-clients-4.4/bin"));
+ path.Add(wxT("/opt/local/greenplum-clients-4.4/bin"));
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"));
@@ -1219,6 +1231,10 @@ void pgAdmin3::InitXtraPaths()
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-5.0/lib"));
+ path.Add(wxT("/opt/local/greenplum-clients-5.0/lib"));
+ path.Add(wxT("/usr/local/greenplum-clients-4.4/lib"));
+ path.Add(wxT("/opt/local/greenplum-clients-4.4/lib"));
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"));
@@ -1664,7 +1680,7 @@ void pgAdmin3::InitHelp()
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");
+ gpHelpPath = wxT("http://gpdb.docs.pivotal.io/");
if (slonyHelpPath.IsEmpty())
slonyHelpPath = wxT("http://www.slony.info/documentation/");
diff --git a/pgadmin/utils/misc.cpp b/pgadmin/utils/misc.cpp
index 93b56f8..96464c0 100644
--- a/pgadmin/utils/misc.cpp
+++ b/pgadmin/utils/misc.cpp
@@ -676,7 +676,7 @@ wxString CleanHelpPath(const wxString &path)
thePath.Lower().EndsWith(wxT(".zip")))
return thePath;
- // In all othe cases we must have a directory
+ // In all other cases we must have a directory
wxString sep;
// Figure out the appropriate seperator
@@ -764,9 +764,21 @@ void DisplayHelp(const wxString &helpTopic, const HelpType helpType)
// the old help path (stored in the settings) is no longer working
static wxString gpHelpPath = settings->GetGpHelpPath();
+ // Note: never end the URL on "index.html"
+ // InitHelp() does obscure magic with this ending
+ if (gpHelpPath.CmpNoCase(wxT("http://docs.gopivotal.com/gpdb/")) == 0)
+ {
+ gpHelpPath = wxT("http://gpdb.docs.pivotal.io/");
+ // Replace the path to the old domain with the link to
+ // the new documentation path
+ // The old link is working for now, but there is no guarantee
+ // that it will stay this way
+ // Also the new link automatically redirects to the latest version
+ settings->SetGpHelpPath(gpHelpPath);
+ }
if (gpHelpPath.CmpNoCase(wxT("http://www.greenplum.com/docs/3300/")) == 0)
{
- gpHelpPath = wxT("http://docs.gopivotal.com/gpdb/");
+ gpHelpPath = wxT("http://gpdb.docs.pivotal.io/");
// 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);
[text/x-patch] gp-doc2-mac.diff (5.4K, 3-gp-doc2-mac.diff)
download | inline diff:
diff --git a/pgadmin/pgAdmin3.cpp b/pgadmin/pgAdmin3.cpp
index 3f9cf3f..87ee9f0 100644
--- a/pgadmin/pgAdmin3.cpp
+++ b/pgadmin/pgAdmin3.cpp
@@ -1167,6 +1167,8 @@ void pgAdmin3::InitXtraPaths()
// 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-5.0\\bin"));
+ path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-4.4\\bin"));
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"));
@@ -1174,6 +1176,8 @@ void pgAdmin3::InitXtraPaths()
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-5.0\\lib"));
+ path.Add(programFiles + wxT("\\Greenplum\\greenplum-clients-4.4\\lib"));
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"));
@@ -1184,6 +1188,8 @@ void pgAdmin3::InitXtraPaths()
if (!programFilesX86.IsEmpty())
{
+ path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-5.0\\bin"));
+ path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-4.4\\bin"));
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"));
@@ -1191,6 +1197,8 @@ void pgAdmin3::InitXtraPaths()
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-5.0\\lib"));
+ path.Add(programFilesX86 + wxT("\\Greenplum\\greenplum-clients-4.4\\lib"));
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"));
@@ -1206,6 +1214,10 @@ void pgAdmin3::InitXtraPaths()
// Generic Unix paths
+ path.Add(wxT("/usr/local/greenplum-clients-5.0/bin"));
+ path.Add(wxT("/opt/local/greenplum-clients-5.0/bin"));
+ path.Add(wxT("/usr/local/greenplum-clients-4.4/bin"));
+ path.Add(wxT("/opt/local/greenplum-clients-4.4/bin"));
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"));
@@ -1219,6 +1231,10 @@ void pgAdmin3::InitXtraPaths()
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-5.0/lib"));
+ path.Add(wxT("/opt/local/greenplum-clients-5.0/lib"));
+ path.Add(wxT("/usr/local/greenplum-clients-4.4/lib"));
+ path.Add(wxT("/opt/local/greenplum-clients-4.4/lib"));
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"));
@@ -1664,7 +1680,7 @@ void pgAdmin3::InitHelp()
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");
+ gpHelpPath = wxT("http://gpdb.docs.pivotal.io/");
if (slonyHelpPath.IsEmpty())
slonyHelpPath = wxT("http://www.slony.info/documentation/");
diff --git a/pgadmin/utils/misc.cpp b/pgadmin/utils/misc.cpp
index 93b56f8..96464c0 100644
--- a/pgadmin/utils/misc.cpp
+++ b/pgadmin/utils/misc.cpp
@@ -676,7 +676,7 @@ wxString CleanHelpPath(const wxString &path)
thePath.Lower().EndsWith(wxT(".zip")))
return thePath;
- // In all othe cases we must have a directory
+ // In all other cases we must have a directory
wxString sep;
// Figure out the appropriate seperator
@@ -764,9 +764,21 @@ void DisplayHelp(const wxString &helpTopic, const HelpType helpType)
// the old help path (stored in the settings) is no longer working
static wxString gpHelpPath = settings->GetGpHelpPath();
+ // Note: never end the URL on "index.html"
+ // InitHelp() does obscure magic with this ending
+ if (gpHelpPath.CmpNoCase(wxT("http://docs.gopivotal.com/gpdb/")) == 0)
+ {
+ gpHelpPath = wxT("http://gpdb.docs.pivotal.io/");
+ // Replace the path to the old domain with the link to
+ // the new documentation path
+ // The old link is working for now, but there is no guarantee
+ // that it will stay this way
+ // Also the new link automatically redirects to the latest version
+ settings->SetGpHelpPath(gpHelpPath);
+ }
if (gpHelpPath.CmpNoCase(wxT("http://www.greenplum.com/docs/3300/")) == 0)
{
- gpHelpPath = wxT("http://docs.gopivotal.com/gpdb/");
+ gpHelpPath = wxT("http://gpdb.docs.pivotal.io/");
// 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);
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]
Subject: Re: Updated Greenplum documentation
In-Reply-To: <[email protected]>
* 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