[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 1/5] substitute all uses of which by type
From: |
Juan Quintela |
Subject: |
[Qemu-devel] Re: [PATCH 1/5] substitute all uses of which by type |
Date: |
Wed, 20 Jan 2010 19:52:30 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Stefan Weil <address@hidden> wrote:
> Juan Quintela schrieb:
>> Except in one case, we are only interested in knowing if a command
>> exist, not is path. Just create prog_exists() function that does this
>> check.
>>
>> Signed-off-by: Juan Quintela <address@hidden>
>
> See comments below. The changes proposed are a suggestion, no real need.
>
> Regards,
> Stefan
>
>> ---
>> configure | 25 ++++++++++++++-----------
>> 1 files changed, 14 insertions(+), 11 deletions(-)
>>
>> diff --git a/configure b/configure
>> index 5631bbb..71edaf5 100755
>> --- a/configure
>> +++ b/configure
>> @@ -27,6 +27,11 @@ compile_prog() {
>> $cc $QEMU_CFLAGS $local_cflags -o $TMPE $TMPC $LDFLAGS $local_ldflags
>> > /dev/null 2> /dev/null
>> }
>>
>> +prog_exist() {
>> + prog="$1"
>> + type $prog > /dev/null 2> /dev/null
>
> I personally prefer
> + type $prog >/dev/null 2>&1
> because it is shorter (no whitespace after >, no duplicate /dev/null).
> Currently, all variants to write this pattern are used.
> Maybe we can reduce this.
Agreed. For next series if there is not needed a rebase.
Later, Juan.
- [Qemu-devel] [PATCH 0/5] Remove which use, Juan Quintela, 2010/01/20
- [Qemu-devel] [PATCH 1/5] substitute all uses of which by type, Juan Quintela, 2010/01/20
- [Qemu-devel] [PATCH 4/5] Remove check for grep, Juan Quintela, 2010/01/20
- [Qemu-devel] [PATCH 3/5] Add -static earlier to LDFLAGS for compile_prog(), Juan Quintela, 2010/01/20
- [Qemu-devel] [PATCH 2/5] Check for sdl-config before calling it, Juan Quintela, 2010/01/20
- [Qemu-devel] [PATCH 5/5] Fix missing symbols in .rel/.rela.plt sections, Juan Quintela, 2010/01/20