git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 1/1] mingw: drop MakeMaker reference

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Feb 25, 2019, 22:07 UTC
Message-ID
<nycvar.QRO.7.76.6.1902252304180.41@tvgsbejvaqbjf.bet>
In-Reply-To
<20190225200225.GA16965@google.com>
Hi Jonathan,
On Mon, 25 Feb 2019, Jonathan Nieder wrote:
Show 16 quoted lines
> Johannes Schindelin via GitGitGadget wrote:
> 
> > From: Johannes Schindelin <johannes.schindelin@gmx.de>
> >
> > In 20d2a30f8ffe (Makefile: replace perl/Makefile.PL with simple make
> > rules, 2017-12-10), Git stopped using MakeMaker. Therefore, that
> > definition in the MINGW-specific section became useless.
> >
> > Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
> > ---
> >  config.mak.uname | 1 -
> >  1 file changed, 1 deletion(-)
> 
> Yay!
> 
> Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
Thanks!
> Is there a way to automate checking for make variables that are set
> but never used?
No, not that I know of...

The biggest problem is that we only add documentation to the beginning of the Makefile by convention, and there is not really any consistent way to parse that documentation to figure out what build variables are handled currently.

Ciao, Dscho

Previous: Jonathan NiederNext: Junio C Hamano
Message 4 of 9 in “Drop last MakeMaker reference”
  1. 0/1 Drop last MakeMaker referenceJohannes Schindelin via GitGitGadget, Feb 25, 2019
  2. 1/1 mingw: drop MakeMaker referenceJohannes Schindelin via GitGitGadget, Feb 25, 2019
  3. Jonathan NiederFeb 25, 2019
  4. Johannes SchindelinFeb 25, 2019
  5. Junio C HamanoMar 3, 2019
  6. Johannes SchindelinMar 7, 2019
  7. Junio C HamanoMar 8, 2019
  8. Johannes SchindelinMar 8, 2019
  9. Jeff KingMar 11, 2019

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.