This is the mail archive of the cygwin-apps@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [ITP] d 1.2.0


Hello Yaakov,

> http://mywebpage.netscape.com/yselkowitz/cygwin/d/d-1.2.0-1.tar.bz2
> http://mywebpage.netscape.com/yselkowitz/cygwin/d/d-1.2.0-1-src.tar.bz2
> http://mywebpage.netscape.com/yselkowitz/cygwin/d/setup.hint

> category: Utils
> requires: cygwin
> sdesc: "The Directory Lister"
> ldesc: "The Directory Lister is a optimized version of ls verbose mode
> with colorized output, space optimization, and many other features."

I vote pro.

Review:

Binary is not stripped:
$ ls -l usr/bin/d.exe
-rwxr-xr-x    1 gerrit   Domänen-   607133 Oct  1 11:19 usr/bin/d.exe*
$ strip usr/bin/d.exe
$ ls -l usr/bin/d.exe
-rwxr-xr-x    1 gerrit   Domänen-   376320 Oct  1  2003 usr/bin/d.exe*

Else the binary package looks ok.

Source package looks ok.

Readme:

Build instructions:
  unpack d-1.2.0-1-src.tar.bz2
    if you use setup to install this src package, it will be
         unpacked under /usr/src automatically
  cd /usr/src/d-1.2.0-1
  ./configure
  make
  make install

Would you mind to include all the options you used to build this binary
release?


Gerrit
-- 
=^..^=


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]