Re: cygutils: replace ascii with ESR's?

2011-10-31 Thread Christian Franke

Yaakov (Cygwin/X) wrote:

ESR maintains a standalone ascii(1) which appears to have more features
than cygutils':

http://catb.org/~esr/ascii/

I have a package already in Ports:

ftp://ftp.cygwinports.org/pub/cygwinports/release-2/ascii

(The latest 3.11 release is already in my upload queue.)

Should we replace this?  Patch attached if so.




+1

Christian



Re: cygutils: replace ascii with ESR's?

2011-10-31 Thread Charles Wilson

On 10/30/2011 8:26 PM, Yaakov (Cygwin/X) wrote:

ESR maintains a standalone ascii(1) which appears to have more features
than cygutils':

http://catb.org/~esr/ascii/

Should we replace this?  Patch attached if so.


Well, I have mixed feelings.  On the PRO side:

1) smaller cygutils == fewer headaches for me. Also, since cygutils is 
now pulled in by several Base packages, the smaller it becomes, the better.


2) Following the same thought, if something is available (and 
maintained) elsewhere then cygutils shouldn't duplicate effort, unless 
its version really adds value -- or is unique to the cygwin platform 
(e.g. cygdrop).  [Hmm...dump.exe = 'od -Ax -tx2z'?]


3) show all names for a single character feature is...interesting.  I 
like the aliases listed for '' (includes gozinta -- and I haven't 
seen 'bra' and 'ket' used for '' and '' since Statistical Mechanics 
and Quantum Dynamics...tho technically it is '|' and '|')


However, on the CON side:

1) I really find the default table output of ESR's version rather ugly 
(and the hex only -x, decimal only -d, and octal only -o tables are 
downright hideous).


2) No long options (so -h/-? work, but --help doesn't. Ditto -v vs. 
--version).  And no --license.


3) Not sure if this matters, but ESR's version has no option to display 
the high-bit-set character codes (128..255).  I find this feature of 
cygutils-ascii much less useful these days, now that charset:oem (and 
rxvt) are as-good-as-dead, and other terminals with *real* charset 
support -- like mintty, xterm, or rxvt-unicode -- are gaining almost 
complete prevalence on cygwin.



I'll wait and see what other folks think before making a decision.

Comments, anyone?

--
Chuck


cygutils: replace ascii with ESR's?

2011-10-30 Thread Yaakov (Cygwin/X)
ESR maintains a standalone ascii(1) which appears to have more features
than cygutils':

http://catb.org/~esr/ascii/

I have a package already in Ports:

ftp://ftp.cygwinports.org/pub/cygwinports/release-2/ascii

(The latest 3.11 release is already in my upload queue.)

Should we replace this?  Patch attached if so.


Yaakov

2011-10-30  Yaakov Selkowitz  yselkowitz@...

	* Makefile.am: Remove mentions of ascii.
	* PROGLIST: Ditto.
	* README: Ditto.
	* src/ascii: Remove.

Index: Makefile.am
===
RCS file: /cvs/cygwin-apps/cygutils/Makefile.am,v
retrieving revision 1.37
diff -u -p -r1.37 Makefile.am
--- Makefile.am	28 Oct 2011 04:02:57 -	1.37
+++ Makefile.am	28 Oct 2011 06:00:53 -
@@ -31,7 +31,7 @@ if WITH_CYGWIN_SPECIFIC_PROGRAMS
 cygwin_specific_progs = src/cygdrop/cygdrop
 endif
 
-bin_PROGRAMS = src/ascii/ascii $(windows_progs) src/conv/conv \
+bin_PROGRAMS = $(windows_progs) src/conv/conv \
 	src/dump/dump $(ipc_progs) \
 	src/realpath/realpath $(cygwin_specific_progs)
 
@@ -112,7 +112,6 @@ DISTCLEANFILES = \
 	po/remove-potcdate.sed \
 	src/cygicons/cygicons.rc
 
-src_ascii_ascii_LDADD   = -lpopt
 src_banner_banner_LDADD = -lpopt -lkernel32 -lgdi32
 src_clip_getclip_LDADD  = -lpopt
 src_clip_putclip_LDADD  = -lpopt
Index: PROGLIST
===
RCS file: /cvs/cygwin-apps/cygutils/PROGLIST,v
retrieving revision 1.36
diff -u -p -r1.36 PROGLIST
--- PROGLIST	28 Apr 2011 17:41:03 -	1.36
+++ PROGLIST	28 Oct 2011 06:00:53 -
@@ -1,7 +1,3 @@
-ascii.exe
-  dumps the 256 ascii characters to stdout
-  using a nice format (GPLv3+)
-
 dump.exe [file]
   hex dump of file to stdout, using a nice
   format (GPLv3+)
Index: README
===
RCS file: /cvs/cygwin-apps/cygutils/README,v
retrieving revision 1.33
diff -u -p -r1.33 README
--- README	28 Apr 2011 17:41:03 -	1.33
+++ README	28 Oct 2011 06:00:53 -
@@ -47,8 +47,8 @@ files included in this package that fall
 What's in this package:
 ---
 GPLv3+:
- ascii banner cygdrop   cygicons.dll   cygstart
- dump  lprmkshortcutreadshortcut   realpath
+ banner cygdrop   cygicons.dll   cygstartdump
+ lprmkshortcutreadshortcut   realpath
 
  Clipboard:
  putclip   getclip