bug-make
[Top][All Lists]
Advanced

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

[bug #65273] Potential bug in the info function?


From: Paul D. Smith
Subject: [bug #65273] Potential bug in the info function?
Date: Fri, 9 Feb 2024 18:11:20 -0500 (EST)

Follow-up Comment #6, bug#65273 (group make):

I mean, there IS a reasonable way to get the same effect; I showed one
possibility in my response.  Yes, it looks slightly different but the result
is the same and there are no warnings.  Alternatively the Emacs makefiles
could disable these new warnings, if they are deemed more annoying than
helpful.

The problem is that at least 70% of the questions I've been answering on
StackOverflow in the last 5 years more or less boil down to misunderstanding
how make handles expansion, and a lot of those misunderstandings could be
helped if make complained about invalid variable names; for example:

all:
        echo $(git rev-parse HEAD)

will now show a warning (which can be either disabled or converted into an
error via options) because "git rev-parse HEAD" is not a valid variable name.

Leaving these checks disabled by default is not a good solution since the very
people who need this help most, will not benefit from them.

Of course we COULD add "special case" facilities for various projects' special
tricks but I'm not a fan of this approach.  And for this specific case, we
would be disabling warnings for situations like:

all:
        echo $ (CURDIR)

which admittedly is probably rare but still seems like it is a bizarre special
case to keep.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?65273>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]