Mailing-List: contact cygwin-apps-help AT cygwin DOT com; run by ezmlm
Sender: cygwin-apps-owner AT cygwin DOT com
List-Subscribe: <mailto:cygwin-apps-subscribe AT cygwin DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin-apps/>
List-Post: <mailto:cygwin-apps AT cygwin DOT com>
List-Help: <mailto:cygwin-apps-help AT cygwin DOT com>, <http://sources.redhat.com/lists.html#faqs>
Mail-Followup-To: cygwin-apps AT cygwin DOT com
Delivered-To: mailing list cygwin-apps AT cygwin DOT com
Message-ID: <3D041CA7.4070708@ece.gatech.edu>
Date: Sun, 09 Jun 2002 23:27:35 -0400
From: Charles Wilson <cwilson AT ece DOT gatech DOT edu>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9.4) Gecko/20011019 Netscape6/6.2
X-Accept-Language: en-us
MIME-Version: 1.0
To: Volker Quetschke <quetschke AT scytek DOT de>
CC: cygwin-apps AT cygwin DOT com
Subject: Re: package offering: gnupg
References: <3D01D259 DOT 9080508 AT scytek DOT de>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit

You probably should add a postinstall script to install-info the .info 
files -- oh wait -- you could just rely on cgf's _install_info virtual 
package to handle that...

Other than that, it looks good (although I'm a gpg newbie, so I haven't 
actually tested *operation*...)

In the build script, don't just comment out the check() function -- you 
also need to comment out check) from the switch statement.  Better, just 
replace the guts of the check() function with "echo Check not implemented."

You should probably patch 'autogen' to call gettextize with the -c 
option.  Ditto autoconf and automake (and libtool, if that applies).

Other than those tiny things, it looks great.

--Chuck