<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Aug 18, 2021 at 6:01 PM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Joerg,<br>
<br>
> Hi Petr,<br>
<br>
> On 8/18/2021 11:12 AM, Petr Vorel wrote:<br>
> > "which" has been discontinued after 2.21 release in 2015 due this (git<br>
> > repository is empty [1]).<br>
> I am a bit against dropping this. which is widely used and I think as long<br>
> as it is available, it should behave as expected.<br>
First, thanks for your review of all patchset.<br>
<br>
I should have marked this commit as RFC. I have no problem to postpone<br>
deleting this test fairly long time (even for 1-2 years).<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">I just have a look at which01.sh that is written by the new API and so far so good.</div><div class="gmail_default" style="font-size:small">Maybe we can reserve it for a bit long time.<br></div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
But, if we really want to test basic shell commands (IMHO they should be part of<br>
particular shell implementation code and most of these projects have it: e.g.<br>
bash, busybox, coreutils, util-linux, ...), in this particular test it'd be more<br>
useful to test "type" or "command" than "which", which code hasn't been changed<br>
since 2015. Looking into openSUSE code [1], there is no patch on "which"<br>
command.<br>
<br>
Also, I might be a bit too aggressive when removing tests for legacy code, but<br>
in this case Debian has deprecated "which" from it's basic tools [2], proving<br>
there is a push to use "type" / "command -v" at least in some distros.<br>
<br>
Kind regards,<br>
Petr<br>
<br>
[1] <a href="https://build.opensuse.org/package/show/Base:System/which" rel="noreferrer" target="_blank">https://build.opensuse.org/package/show/Base:System/which</a><br>
[2] <a href="https://salsa.debian.org/debian/debianutils/-/commit/3a8dd10b4502f7bae8fc6973c13ce23fc9da7efb" rel="noreferrer" target="_blank">https://salsa.debian.org/debian/debianutils/-/commit/3a8dd10b4502f7bae8fc6973c13ce23fc9da7efb</a><br>
<br>
-- <br>
Mailing list info: <a href="https://lists.linux.it/listinfo/ltp" rel="noreferrer" target="_blank">https://lists.linux.it/listinfo/ltp</a><br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>