Postgres Pro
Downloads
Home   >   mailing lists

Re: Patch for removing TipsOfDay functionality - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Patch for removing TipsOfDay functionality
Date
Msg-id 937d27e10811040408k7f58ec1cu5d633c5e25b5709a@mail.gmail.com
Whole thread Raw
In response to Patch for removing TipsOfDay functionality  (Ashesh D Vashi <ashesh.vashi@enterprisedb.com>)
List pgadmin-hackers
Tree view 
On Tue, Nov 4, 2008 at 6:02 AM, Ashesh D Vashi
<ashesh.vashi@enterprisedb.com> wrote:
> Hi,
>
> As per Dave page suggestion, I worked on removing the "Tips Of the Day"
> functionality.
> I have attached the patch for it.
>
> Please review it.

Hi Ashesh,

An eyeball review looks good - you've fixed up a couple of places I'd
completely forgotten about. Just one change required I think; in the
Windows installer part of the patch, there are a bunch of hunks like:

                                 <Component Id="de_de_docs"
Guid="{BCC51FC8-18F9-46CB-A536-8EB8A55812F0}">
-                                    <File Id="de_de_docs.tips.txt"
Name="tips.txt" DiskId="1"
Source="$(var.BUILDTREE)/docs/de_DE/tips.txt" />
+                                    <CreateFolder/>
                                 </Component>

We should just remove those empty components altogether I think.

--
Dave Page
EnterpriseDB UK:   http://www.enterprisedb.com

pgadmin-hackers by date:

Previous
From: Ashesh D Vashi
Date:
Subject: Patch for removing TipsOfDay functionality
Next
From: svn@pgadmin.org
Date:
Subject: SVN Commit by dpage: r7496 - in trunk/pgadmin3: . docs docs/de_DE docs/en_US docs/es_ES docs/fi_FI docs/fr_FR docs/it_IT docs/sl_SI docs/zh_CN docs/zh_TW pgadmin pgadmin/frm pgadmin/include/frm pgadmin/include/utils pgadmin/utils pkg/win32/src