Re: [PATCH 1/1] tests: replace `test -(d|f)` with test_path_is_(dir|file)

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

Subject: Re: [PATCH 1/1] tests: replace `test -(d|f)` with test_path_is_(dir|file)
From: Rohit Ashiwal <rohit.ashiwal265@xxxxxxxxx>
Date: Wed, 27 Feb 2019 01:31:34 +0530
Cc: Martin Ågren <martin.agren@xxxxxxxxx>, Rohit Ashiwal via GitGitGadget <gitgitgadget@xxxxxxxxx>, Git Mailing List <git@xxxxxxxxxxxxxxx>, Junio C Hamano <gitster@xxxxxxxxx>, pclouds@xxxxxxxxx
In-reply-to: <nycvar.QRO.7.76.6.1902262051080.41@tvgsbejvaqbjf.bet>
References: <pull.152.git.gitgitgadget@gmail.com> <bf5eb045795579dd5d996e787e246996688cf4bf.1551188524.git.gitgitgadget@gmail.com> <CAN0heSqSp-a0zUKT5EaGLBYnRtESTnu9GKWtGARz2kaOAhc1HQ@mail.gmail.com> <CAL7ArXoau1ZfBsV9JaUDprwjSijyo6K5d9JyC1mdfc=KEvgJxw@mail.gmail.com> <nycvar.QRO.7.76.6.1902262051080.41@tvgsbejvaqbjf.bet>

Hey!

On Wed, Feb 27, 2019 at 1:22 AM Johannes Schindelin
<Johannes.Schindelin@xxxxxx> wrote:
>
> We already have `test_path_is_missing`. Why not use that instead of `!
> test -d` or `! test -f`?
>

Yes, I think this is better. It will satisfy all the requirements I guess.

Ciao
Rohit

Follow-Ups:
- Re: [PATCH 1/1] tests: replace `test -(d|f)` with test_path_is_(dir|file)
  - From: Martin Ågren

References:
- [PATCH 0/1] [GSoC][PATCH] tests: replace test -(d|f) with test_path_is_(dir|file)
  - From: Rohit Ashiwal via GitGitGadget
- [PATCH 1/1] tests: replace `test -(d|f)` with test_path_is_(dir|file)
  - From: Rohit Ashiwal via GitGitGadget
- Re: [PATCH 1/1] tests: replace `test -(d|f)` with test_path_is_(dir|file)
  - From: Martin Ågren
- Re: [PATCH 1/1] tests: replace `test -(d|f)` with test_path_is_(dir|file)
  - From: Rohit Ashiwal
- Re: [PATCH 1/1] tests: replace `test -(d|f)` with test_path_is_(dir|file)
  - From: Johannes Schindelin

Prev by Date: Re: Do test-path_is_{file,dir,exists} make sense anymore with -x?
Next by Date: Re: [PATCH v2 1/1] t3600: use test_path_is_dir and test_path_is_file
Previous by thread: Re: [PATCH 1/1] tests: replace `test -(d|f)` with test_path_is_(dir|file)
Next by thread: Re: [PATCH 1/1] tests: replace `test -(d|f)` with test_path_is_(dir|file)
Index(es):
- Date
- Thread

[Index of Archives] [Linux Kernel Development] [Gcc Help] [IETF Annouce] [DCCP] [Netdev] [Networking] [Security] [V4L] [Bugtraq] [Yosemite] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux SCSI] [Fedora Users]