All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Stancek <jstancek@redhat.com>
To: ltp@lists.linux.it
Subject: [LTP] [RFC] Cleanup of tools/ directory
Date: Mon, 21 Nov 2016 04:10:57 -0500 (EST)	[thread overview]
Message-ID: <194063547.847310.1479719457351.JavaMail.zimbra@redhat.com> (raw)
In-Reply-To: <20161114140504.GE25055@rei.lan>





----- Original Message -----
> From: "Cyril Hrubis" <chrubis@suse.cz>
> To: ltp@lists.linux.it
> Sent: Monday, 14 November, 2016 3:05:04 PM
> Subject: [LTP] [RFC] Cleanup of tools/ directory
> 
> Hi!
> I've stumbled upon the tools/ directory and there is quite a lot of
> broken code that is not really useful for anything.
> 
> The Makefile skips netpipe*, pounder21 and top-LTP directories. Apart
> from automatic changes nobody touched the top-LTP and netpipe* code for
> more than six years, so I would say that it's safe enough to remove
> these. The pounder21 had some fixes that looks like somebody is using
> that so I guess that we should keep it.

I see pounder referenced occasionally in RH bugzilla reports, so
I'd keep it.

> 
> Then there are files that looks like they could be removed as well:
> 
> * STPfailure_report.pl - most likely no change since 2004, looks useless o me
> 
> * ltp_master, ltp_check, README, ltpoutput*
>   - remnants of some automated framework that used ltprun script that
>     used runalltests.sh script, I will remove these since there is no
>     reason to keep them now
> 
> In short has anybody anything agains removal of netpipe*, top-LTP,
> STPfailure_report.pl, ltp_master, ltp_check, README, and ltpoutput* from
> the tools/ directory?

My only hesitation was netpipe, which is referenced by couple scripts,
but looking at tools/Makefile comment and fact we are not building it for
so many years, I guess nobody will miss it.

Regards,
Jan

> 
> --
> Cyril Hrubis
> chrubis@suse.cz
> 
> --
> Mailing list info: https://lists.linux.it/listinfo/ltp
> 

  reply	other threads:[~2016-11-21  9:10 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-14 14:05 [LTP] [RFC] Cleanup of tools/ directory Cyril Hrubis
2016-11-21  9:10 ` Jan Stancek [this message]
2016-11-21 11:21   ` Cyril Hrubis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=194063547.847310.1479719457351.JavaMail.zimbra@redhat.com \
    --to=jstancek@redhat.com \
    --cc=ltp@lists.linux.it \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.