Re: [Qemu-devel] [PATCH] configure: factor out adding disas configure

[Peter Crosthwaite]

On Sat, Jul 18, 2015 at 2:09 AM, Peter Maydell <address@hidden> wrote:
> On 18 July 2015 at 07:25, Peter Crosthwaite <address@hidden> wrote:
>> Every arch adds its disas configury to both its own config as well
>> config_disas_all. Make a small function do to both at once.
>>
>> Signed-off-by: Peter Crosthwaite <address@hidden>
>> ---
>> Prepares support for multi-arch where the disas configury for
>> multi-arch will also need to be added where 3 more LOC are added
>> for each DISAS configury.
>> ---
>>  configure | 65 
>> ++++++++++++++++++++++++---------------------------------------
>>  1 file changed, 25 insertions(+), 40 deletions(-)
>>
>> diff --git a/configure b/configure
>> index cc0338d..096977b 100755
>> --- a/configure
>> +++ b/configure
>> @@ -5466,91 +5466,76 @@ fi
>>  cflags=""
>>  ldflags=""
>>
>> +disas_config() {
>> +  echo "$@" >> $config_target_mak
>> +  echo "$@" >> config-all-disas.mak
>> +}
>
> If we're calling it disas_config maybe we should make
> the argument just be ALPHA, not 'CONFIG_ALPHA_DIS=y' ?
>

Fixed.

Regards,
Peter

> -- PMM