Thu May 8 15:41:00 GMT 2003
Thanks Igor. I have fixed the typo in the readme and added the strip step.
The new files can be downloaded from:
David Hudson | The Capital Group Companies
| Location: BRO | Extension: 51887
Outside: 714-672-1887 | E-mail:
[ Mailing: 135 S State College Blvd.
Brea, CA 92821-5704 USA ]
<email@example.com To: David Hudson/INV/CG/CAPITAL@CG
du> cc: firstname.lastname@example.org
Sent by: Subject: Re: a2ps (Was Re: [UPDATE] Pending package status (06 May 2003))
Please respond to
Yes. Turns out I downloaded it, but got distracted by real work and
forgot to review it. Sorry about that.
The CRLF issue and the permission bits were indeed fixed, in both the
binary and the source packages.
There's a minor typo in the readme:
This will build, creat teh binary package and re-create the source
I'm still a bit wary of the "all" step in the build script, in that it
will remove the source directory, but that's my personal feeling, and
the consensus here seems to be to leave it the way it is...
However, your build script seems to be missing the "strip" step (compared
to, say, a similar script in cygutils). FYI,
$ ls -l a2ps.exe
-rwx--x--x 1 igor Administ 890050 Apr 16 12:19 a2ps.exe*
$ strip a2ps.exe
$ ls -l a2ps.exe
-rwx--x--x 1 igor Administ 306688 May 7 16:56 a2ps.exe*
Otherwise this looks good.
On Wed, 7 May 2003 David_Hudson@capgroup.com wrote:
> Both issues that were raised (CRLF and exec bits) had been rectified in
> final package posted on 4/16.
> On Wed, May 07, 2003 at 10:49:10AM -0400, Igor Pechtchanski wrote:
> >FYI, the two comments below are really addressed to the general
> >cygwin-apps audience, not you personally.
> >Hmm, first off, the first link below (underlined) refers to aspell,
> >than a2ps (Elfyn?).
> >Secondly, I thought
> ><http://cygwin.com/ml/cygwin-apps/2003-04/msg00273.html> (the fourth
> >was a review (at least, it was written as such).
> >If that's unacceptable, what constitutes a formal review? Or do I
> >need to update it for the latest posted release?
> That was certainly a formal enough review. I didn't see any indication
> that your issues were rectified; at least not in that thread. The CRLF
> issue, at least, deserves repackaging.
> Hello all,
> I have uploaded the (hopefully) final version of the a2ps-4.12-1 package.
> used Max's suggestions on the build file to keep it as similar as
> with the build file in the Package Guidelines.
> Thanks for all the feedback on this process. I'll start looking at
|\ _,,,---,,_ email@example.com
ZZZzz /,`.-'`' -. ;-;;,_ firstname.lastname@example.org
|,4- ) )-,_. ,\ ( `'-' Igor Pechtchanski
'---''(_/--' `-'\_) fL a.k.a JaguaR-R-R-r-r-r-.-.-. Meow!
Knowledge is an unending adventure at the edge of uncertainty.
-- Leto II
More information about the Cygwin-apps