Re: [PATCH 5/6] configure: allow picking up a dedicated dwarf version from command line

[Jacek Caban]

Hi Eric,

On 9/28/21 3:44 PM, Eric Pouech wrote:
> @@ -2040,9 +2044,10 @@ char*f(const char *h,char n) {return strchr(h,n);}]])],[ac_cv_c_logicalop_noisy=
>     esac
> dnl Default to dwarf-2 debug info
> - AC_SUBST(DEBUGFORMAT)
>  for ac_flag in $CFLAGS; do
>    case $ac_flag in
> -
>  -g) WINE_TRY_CFLAGS([-gdwarf-2])
> -
>  -g) WINE_TRY_CFLAGS([-g${DEBUGFORMAT:-dwarf-2}])
>       WINE_TRY_CFLAGS([-gstrict-dwarf]) ;;
> esac
> done

Maybe we could make it a bit smarter, by scanning CFLAGS for explicit 
-gdwarf-X like CROSSDEBUG does. That should in fact make exposing 
DEBUGFORMAT redundant. In case of CROSSDEBUG, we also have split debug 
option that somehow needs to be passed to configure, but we don't have 
that for native parts. (Still, DEBUGFORMAT may be useful for convenience).

Thanks,

Jacek