Re: [PATCH] Makefile: crash multi-target and multithread compile support

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

 



Hi Kazu,


On Tue, Feb 22, 2022 at 9:11 AM HAGIO KAZUHITO(萩尾 一仁)
<k-hagio-ab@xxxxxxx> wrote:
>
> Hi Tao Liu,
>
> thanks for tackling this issue.
>
> -----Original Message-----
> > > also multiple executions of configure can be removed with this way.
> > >
> > > +ifeq ($(findstring warn,$(MAKECMDGOALS)),warn)
> > > +CONF_TARGET_FLAG += -w
> > > +endif
> > > +ifeq ($(findstring Warn,$(MAKECMDGOALS)),Warn)
> > > +CONF_TARGET_FLAG += -W
> > > +endif
> >
> > I agree with your idea, but I didn't succeed in this way,
> > when making CONF_TARGET_FLAG += -W in
> > the makefile receipt as follows, I couldn't get the value
> > of ${CONF_TARGET_FLAG}:
> >
> > config: make_configure
> >     ifeq ($(findstring warn,$(MAKECMDGOALS)),warn)
> >     CONF_TARGET_FLAG += -w
> >     endif
> >     ifeq ($(findstring Warn,$(MAKECMDGOALS)),Warn)
> >     CONF_TARGET_FLAG += -W
> >     endif
> >     ....
> >     @echo ${CONF_TARGET_FLAG}      <------- output nothing
> >
>
> What I meant was the following and I've picked the last part from your v2.
> How about this?
>
> --- a/Makefile
> +++ b/Makefile
> @@ -223,6 +223,28 @@ ifneq ($(target),)
>  CONF_TARGET_FLAG="-t$(target)"
>  endif
>
> +ifeq ($(findstring warn,$(MAKECMDGOALS)),warn)
> +CONF_TARGET_FLAG += -w
> +endif
> +ifeq ($(findstring Warn,$(MAKECMDGOALS)),Warn)
> +CONF_TARGET_FLAG += -W
> +endif
> +ifeq ($(findstring nowarn,$(MAKECMDGOALS)),nowarn)
> +CONF_TARGET_FLAG += -n
> +endif
> +ifeq ($(findstring lzo,$(MAKECMDGOALS)),lzo)
> +CONF_TARGET_FLAG += -x lzo
> +endif
> +ifeq ($(findstring snappy,$(MAKECMDGOALS)),snappy)
> +CONF_TARGET_FLAG += -x snappy
> +endif
> +ifeq ($(findstring zstd,$(MAKECMDGOALS)),zstd)
> +CONF_TARGET_FLAG += -x zstd
> +endif
> +ifeq ($(findstring valgrind,$(MAKECMDGOALS)),valgrind)
> +CONF_TARGET_FLAG += -x valgrind
> +endif
> +
>  # To build the extensions library by default, uncomment the third command
>  # line below.  Otherwise they can be built by entering "make extensions".
>

Yeah! It's much better!

> @@ -305,33 +327,8 @@ install:
>  unconfig: make_configure
>         @./configure -u
>
> -warn: make_configure
> -       @./configure ${CONF_TARGET_FLAG} -w -b
> -       @$(MAKE) gdb_merge
> -
> -Warn: make_configure
> -       @./configure ${CONF_TARGET_FLAG} -W -b
> -       @$(MAKE) gdb_merge
> -
> -nowarn: make_configure
> -       @./configure ${CONF_TARGET_FLAG} -n -b
> -       @$(MAKE) gdb_merge
> -
> -lzo: make_configure
> -       @./configure -x lzo ${CONF_TARGET_FLAG} -w -b
> -       @$(MAKE) gdb_merge
> -
> -snappy: make_configure
> -       @./configure -x snappy ${CONF_TARGET_FLAG} -w -b
> -       @$(MAKE) gdb_merge
> -
> -zstd: make_configure
> -       @./configure -x zstd ${CONF_TARGET_FLAG} -w -b
> -       @$(MAKE) gdb_merge
> -
> -valgrind: make_configure
> -       @./configure -x valgrind ${CONF_TARGET_FLAG} -w -b
> -       @$(MAKE) gdb_merge
> +warn Warn nowarn lzo snappy zstd valgrind: all
> +       @true   # dummy
>
I will send the v3 patch for this. Thanks for the suggestions!

Thanks,
Tao Liu

>  main.o: ${GENERIC_HFILES} main.c
>         ${CC} -c ${CRASH_CFLAGS} main.c ${WARNING_OPTIONS} ${WARNING_ERROR}
>
>
> Thanks,
> Kazu
>


--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://listman.redhat.com/mailman/listinfo/crash-utility




[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux