Date: Sun, 4 Nov 2012 12:40:01 GMT From: Eitan Adler <lists@eitanadler.com> To: freebsd-bugs@FreeBSD.org Subject: Re: bin/113518: [patch] make(1): Prevent execution when command is a comment Message-ID: <201211041240.qA4Ce1Ym001190@freefall.freebsd.org>
index | next in thread | raw e-mail
The following reply was made to PR bin/113518; it has been noted by GNATS.
From: Eitan Adler <lists@eitanadler.com>
To: Jilles Tjoelker <jilles@stack.nl>, sjg@freebsd.org
Cc: bug-followup@freebsd.org
Subject: Re: bin/113518: [patch] make(1): Prevent execution when command is a comment
Date: Sun, 4 Nov 2012 07:35:01 -0500
On 4 November 2012 07:20, Jilles Tjoelker <jilles@stack.nl> wrote:
> The following reply was made to PR bin/113518; it has been noted by GNATS.
>
> From: Jilles Tjoelker <jilles@stack.nl>
> To: bug-followup@FreeBSD.org, ed@fxq.nl
> Cc: sjg@FreeBSD.org
> Subject: Re: bin/113518: [patch] make(1): Prevent execution when command is a
> comment
> Date: Sun, 4 Nov 2012 13:11:58 +0100
>
> In PR bin/113518:
> > [patch to teach make about the shell's comment character]
>
> The first thing here is whether there are actually real-life Makefiles
> containing shell commands consisting only of comments. I only find a
> few in /usr/ports/Mk:
>
> jilles@jaguar /usr/ports/Mk% grep $'^[ \t]\\+#' *
> bsd.gecko.mk: # XXX: make sure bsdtar(1) corrected symlinks
> bsd.kde4.mk: # Check that the component is valid.
> bsd.kde4.mk: # Skip meta-components (e.g., kdeprefix).
> bsd.kde4.mk: # Check if a dependency type is explicitly requested.
> bsd.kde4.mk: # If no dependency type is set, default to full dependency.
> bsd.kde4.mk: # Set real dependencies.
[10017 eitan@radar /usr/ports !130!]%find . -name Makefile\*|xargs
grep $'^[ \t]\\+#'|wc -l
420
> Secondly, I don't think such new features should be added to make(1)
> when it is planned to replace it with bmake (sjg@ cc'ed). Also, bmake
> generally knows less about the shell than make(1) does so it does not
> seem to fit in to add it to bmake.
Shall this bug be closed?
--
Eitan Adler
help
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201211041240.qA4Ce1Ym001190>