Re: [PATCH 3/3] chkdupexe: reimplement command in C

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Apr 01, 2012 at 08:19:32PM +0200, Sami Kerola wrote:
> The new version of the command is nearly identical to old.  Major
> differences are:
> 
> * New output is a little rough what comes to alignment.  This is
>   flagged as FIXME item in source.
> * The command is much quicker than perl implementation which used
>   to exec ls -ldU.
> * Size of chkdupexe binary is about five times bigger than perl
>   script.

I have doubts about chkdupexe, do we really need this command?

 * it provides very rarely used functionality (for example nobody
   complains that the command is missing in Fedora/RHEL ;-)

 * it seems that you can replace it by

    ls $(echo "$PATH" | tr : ' ') | sort | uniq -d

   or we can extend which(1) or so...


 I see two possible ways:

 1) mark chkdupexe as deprecated and remove it from distros

 2) or add new command (lspath ?) and get much more information from
 $PATH (like duplicates, per-directory stats, per-owner stats,
 per-device stats, selinux summary, suids, etc.) and make chkdupexe as
 alias to the new command (just for backward compatibility).

 If 2) makes sense then it would be better to have it in coreutils ;-)


 CC: Padraig

 ... let's flame :-)

    Karel

-- 
 Karel Zak  <kzak@xxxxxxxxxx>
 http://karelzak.blogspot.com
--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux