public inbox for dtrace@lists.linux.dev
 help / color / mirror / Atom feed
* [PATCH] configure: add support for --(disable|enable)-dependency-tracking
@ 2026-02-17 16:35 Kris Van Hees
  2026-02-17 20:44 ` [DTrace-devel] " Eugene Loh
  0 siblings, 1 reply; 4+ messages in thread
From: Kris Van Hees @ 2026-02-17 16:35 UTC (permalink / raw)
  To: dtrace, dtrace-devel

Signed-off-by: Kris Van Hees <kris.van.hees@oracle.com>
---
 Makefunctions | 2 ++
 configure     | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/Makefunctions b/Makefunctions
index 15836dc3..1f308967 100644
--- a/Makefunctions
+++ b/Makefunctions
@@ -52,7 +52,9 @@ endef
 # Rule to build a C source file.
 # Syntax: $(call cc-template,primary,filename-without-dir,optional flags)
 
+ifndef WITHOUT_DEPENDENCIES
 cc-dependencies = -MP -MMD -MF $(obj-name).deps -MT $(obj-name)
+endif
 
 define cc-template
 $(obj-name): $(src-name) $(foreach dep,$($(1)_SRCDEPS),$(call src-name,$(1),$(dep))) $(foreach prov,$($(1)_PROV),$(call hdr-name,$(1),$(prov))) $(CONFIG_H)
diff --git a/configure b/configure
index 04c28369..fadac05e 100755
--- a/configure
+++ b/configure
@@ -172,6 +172,8 @@ for option in "$@"; do
         HAVE_VALGRIND=*) write_config_var VALGRIND "$option";;
         HAVE_BPFV3=*) write_config_var BPFV3 "$option";;
         HAVE_BPFMASM=*) write_config_var BPFMASM "$option";;
+        --disable-dependency-tracking) write_make_var WITHOUT_DEPENDENCIES "y";;
+        --enable-dependency-tracking) write_make_var WITHOUT_DEPENDENCIES "n";;
         *) echo "Unknown option $option" >&2
            exit 1;;
     esac
-- 
2.51.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [DTrace-devel] [PATCH] configure: add support for --(disable|enable)-dependency-tracking
  2026-02-17 16:35 [PATCH] configure: add support for --(disable|enable)-dependency-tracking Kris Van Hees
@ 2026-02-17 20:44 ` Eugene Loh
  2026-02-18  5:07   ` Kris Van Hees
  0 siblings, 1 reply; 4+ messages in thread
From: Eugene Loh @ 2026-02-17 20:44 UTC (permalink / raw)
  To: Kris Van Hees, dtrace, dtrace-devel

On 2/17/26 11:35, Kris Van Hees via DTrace-devel wrote:

> Signed-off-by: Kris Van Hees <kris.van.hees@oracle.com>
> ---
>   Makefunctions | 2 ++
>   configure     | 2 ++
>   2 files changed, 4 insertions(+)
>
> diff --git a/Makefunctions b/Makefunctions
> index 15836dc3..1f308967 100644
> --- a/Makefunctions
> +++ b/Makefunctions
> @@ -52,7 +52,9 @@ endef
>   # Rule to build a C source file.
>   # Syntax: $(call cc-template,primary,filename-without-dir,optional flags)
>   
> +ifndef WITHOUT_DEPENDENCIES
>   cc-dependencies = -MP -MMD -MF $(obj-name).deps -MT $(obj-name)
> +endif
>   
>   define cc-template
>   $(obj-name): $(src-name) $(foreach dep,$($(1)_SRCDEPS),$(call src-name,$(1),$(dep))) $(foreach prov,$($(1)_PROV),$(call hdr-name,$(1),$(prov))) $(CONFIG_H)
> diff --git a/configure b/configure
> index 04c28369..fadac05e 100755
> --- a/configure
> +++ b/configure
> @@ -172,6 +172,8 @@ for option in "$@"; do
>           HAVE_VALGRIND=*) write_config_var VALGRIND "$option";;
>           HAVE_BPFV3=*) write_config_var BPFV3 "$option";;
>           HAVE_BPFMASM=*) write_config_var BPFMASM "$option";;
> +        --disable-dependency-tracking) write_make_var WITHOUT_DEPENDENCIES "y";;
> +        --enable-dependency-tracking) write_make_var WITHOUT_DEPENDENCIES "n";;

Maybe add these earlier/higher?  Where the other write_make_var settings 
are?

Also, should these options be documented?  Like in the file's help()?

(Come to think of it, how about --with-libctf and --with-libfuse3? 
Should they also appear in --help?)

>           *) echo "Unknown option $option" >&2
>              exit 1;;
>       esac

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [DTrace-devel] [PATCH] configure: add support for --(disable|enable)-dependency-tracking
  2026-02-17 20:44 ` [DTrace-devel] " Eugene Loh
@ 2026-02-18  5:07   ` Kris Van Hees
  2026-02-18  5:38     ` Kris Van Hees
  0 siblings, 1 reply; 4+ messages in thread
From: Kris Van Hees @ 2026-02-18  5:07 UTC (permalink / raw)
  To: Eugene Loh; +Cc: Kris Van Hees, dtrace, dtrace-devel

On Tue, Feb 17, 2026 at 03:44:05PM -0500, Eugene Loh wrote:
> On 2/17/26 11:35, Kris Van Hees via DTrace-devel wrote:
> 
> > Signed-off-by: Kris Van Hees <kris.van.hees@oracle.com>
> > ---
> >   Makefunctions | 2 ++
> >   configure     | 2 ++
> >   2 files changed, 4 insertions(+)
> > 
> > diff --git a/Makefunctions b/Makefunctions
> > index 15836dc3..1f308967 100644
> > --- a/Makefunctions
> > +++ b/Makefunctions
> > @@ -52,7 +52,9 @@ endef
> >   # Rule to build a C source file.
> >   # Syntax: $(call cc-template,primary,filename-without-dir,optional flags)
> > +ifndef WITHOUT_DEPENDENCIES
> >   cc-dependencies = -MP -MMD -MF $(obj-name).deps -MT $(obj-name)
> > +endif
> >   define cc-template
> >   $(obj-name): $(src-name) $(foreach dep,$($(1)_SRCDEPS),$(call src-name,$(1),$(dep))) $(foreach prov,$($(1)_PROV),$(call hdr-name,$(1),$(prov))) $(CONFIG_H)
> > diff --git a/configure b/configure
> > index 04c28369..fadac05e 100755
> > --- a/configure
> > +++ b/configure
> > @@ -172,6 +172,8 @@ for option in "$@"; do
> >           HAVE_VALGRIND=*) write_config_var VALGRIND "$option";;
> >           HAVE_BPFV3=*) write_config_var BPFV3 "$option";;
> >           HAVE_BPFMASM=*) write_config_var BPFMASM "$option";;
> > +        --disable-dependency-tracking) write_make_var WITHOUT_DEPENDENCIES "y";;
> > +        --enable-dependency-tracking) write_make_var WITHOUT_DEPENDENCIES "n";;
> 
> Maybe add these earlier/higher?  Where the other write_make_var settings
> are?

I think it makes more sense to have them at the end because they are not
really common options that people would use.  They are being added because
the GNU Coding Standards require them (and having them means that distro
build systems that expect a standard configure script can pass typical
options without complaints).

> Also, should these options be documented?  Like in the file's help()?

I'll add them to the help.

> (Come to think of it, how about --with-libctf and --with-libfuse3? Should
> they also appear in --help?)

Should be done in a separate patch.

> >           *) echo "Unknown option $option" >&2
> >              exit 1;;
> >       esac

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [DTrace-devel] [PATCH] configure: add support for --(disable|enable)-dependency-tracking
  2026-02-18  5:07   ` Kris Van Hees
@ 2026-02-18  5:38     ` Kris Van Hees
  0 siblings, 0 replies; 4+ messages in thread
From: Kris Van Hees @ 2026-02-18  5:38 UTC (permalink / raw)
  To: Kris Van Hees; +Cc: Eugene Loh, dtrace, dtrace-devel

On Wed, Feb 18, 2026 at 12:07:09AM -0500, Kris Van Hees wrote:
> On Tue, Feb 17, 2026 at 03:44:05PM -0500, Eugene Loh wrote:
> > On 2/17/26 11:35, Kris Van Hees via DTrace-devel wrote:
> > 
> > > Signed-off-by: Kris Van Hees <kris.van.hees@oracle.com>
> > > ---
> > >   Makefunctions | 2 ++
> > >   configure     | 2 ++
> > >   2 files changed, 4 insertions(+)
> > > 
> > > diff --git a/Makefunctions b/Makefunctions
> > > index 15836dc3..1f308967 100644
> > > --- a/Makefunctions
> > > +++ b/Makefunctions
> > > @@ -52,7 +52,9 @@ endef
> > >   # Rule to build a C source file.
> > >   # Syntax: $(call cc-template,primary,filename-without-dir,optional flags)
> > > +ifndef WITHOUT_DEPENDENCIES
> > >   cc-dependencies = -MP -MMD -MF $(obj-name).deps -MT $(obj-name)
> > > +endif
> > >   define cc-template
> > >   $(obj-name): $(src-name) $(foreach dep,$($(1)_SRCDEPS),$(call src-name,$(1),$(dep))) $(foreach prov,$($(1)_PROV),$(call hdr-name,$(1),$(prov))) $(CONFIG_H)
> > > diff --git a/configure b/configure
> > > index 04c28369..fadac05e 100755
> > > --- a/configure
> > > +++ b/configure
> > > @@ -172,6 +172,8 @@ for option in "$@"; do
> > >           HAVE_VALGRIND=*) write_config_var VALGRIND "$option";;
> > >           HAVE_BPFV3=*) write_config_var BPFV3 "$option";;
> > >           HAVE_BPFMASM=*) write_config_var BPFMASM "$option";;
> > > +        --disable-dependency-tracking) write_make_var WITHOUT_DEPENDENCIES "y";;
> > > +        --enable-dependency-tracking) write_make_var WITHOUT_DEPENDENCIES "n";;
> > 
> > Maybe add these earlier/higher?  Where the other write_make_var settings
> > are?
> 
> I think it makes more sense to have them at the end because they are not
> really common options that people would use.  They are being added because
> the GNU Coding Standards require them (and having them means that distro
> build systems that expect a standard configure script can pass typical
> options without complaints).
> 
> > Also, should these options be documented?  Like in the file's help()?
> 
> I'll add them to the help.
> 
> > (Come to think of it, how about --with-libctf and --with-libfuse3? Should
> > they also appear in --help?)
> 
> Should be done in a separate patch.

Actually, this is already part of the help output, by virtue of the 
help-overrides make target.

> > >           *) echo "Unknown option $option" >&2
> > >              exit 1;;
> > >       esac

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2026-02-18  5:38 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-17 16:35 [PATCH] configure: add support for --(disable|enable)-dependency-tracking Kris Van Hees
2026-02-17 20:44 ` [DTrace-devel] " Eugene Loh
2026-02-18  5:07   ` Kris Van Hees
2026-02-18  5:38     ` Kris Van Hees

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox