On Tue, Nov 4, 2008 at 6:02 AM, Ashesh D Vashi
<[EMAIL PROTECTED]> 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

-- 
Sent via pgadmin-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Reply via email to