git@vger.kernel.org list mirror (unofficial, one of many)
 help / color / mirror / code / Atom feed
* [PATCH 0/1] Drop last MakeMaker reference
@ 2019-02-25 19:27 Johannes Schindelin via GitGitGadget
  2019-02-25 19:27 ` [PATCH 1/1] mingw: drop " Johannes Schindelin via GitGitGadget
  2019-03-03  1:19 ` [PATCH 0/1] Drop last " Junio C Hamano
  0 siblings, 2 replies; 9+ messages in thread
From: Johannes Schindelin via GitGitGadget @ 2019-02-25 19:27 UTC (permalink / raw)
  To: git; +Cc: Junio C Hamano

Back when we stopped using MakeMaker, we forgot one reference...

Johannes Schindelin (1):
  mingw: drop MakeMaker reference

 config.mak.uname | 1 -
 1 file changed, 1 deletion(-)


base-commit: 8104ec994ea3849a968b4667d072fedd1e688642
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-146%2Fdscho%2Fno-perl-makemaker-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-146/dscho/no-perl-makemaker-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/146
-- 
gitgitgadget

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

* [PATCH 1/1] mingw: drop MakeMaker reference
  2019-02-25 19:27 [PATCH 0/1] Drop last MakeMaker reference Johannes Schindelin via GitGitGadget
@ 2019-02-25 19:27 ` Johannes Schindelin via GitGitGadget
  2019-02-25 20:02   ` Jonathan Nieder
  2019-03-03  1:19 ` [PATCH 0/1] Drop last " Junio C Hamano
  1 sibling, 1 reply; 9+ messages in thread
From: Johannes Schindelin via GitGitGadget @ 2019-02-25 19:27 UTC (permalink / raw)
  To: git; +Cc: Junio C Hamano, Johannes Schindelin

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(-)

diff --git a/config.mak.uname b/config.mak.uname
index b37fa8424c..b88d8451fe 100644
--- a/config.mak.uname
+++ b/config.mak.uname
@@ -529,7 +529,6 @@ ifneq (,$(findstring MINGW,$(uname_S)))
 	NO_STRTOUMAX = YesPlease
 	NO_MKDTEMP = YesPlease
 	NO_SVN_TESTS = YesPlease
-	NO_PERL_MAKEMAKER = YesPlease
 	RUNTIME_PREFIX = YesPlease
 	HAVE_WPGMPTR = YesWeDo
 	NO_ST_BLOCKS_IN_STRUCT_STAT = YesPlease
-- 
gitgitgadget

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

* Re: [PATCH 1/1] mingw: drop MakeMaker reference
  2019-02-25 19:27 ` [PATCH 1/1] mingw: drop " Johannes Schindelin via GitGitGadget
@ 2019-02-25 20:02   ` Jonathan Nieder
  2019-02-25 22:07     ` Johannes Schindelin
  0 siblings, 1 reply; 9+ messages in thread
From: Jonathan Nieder @ 2019-02-25 20:02 UTC (permalink / raw)
  To: Johannes Schindelin via GitGitGadget
  Cc: git, Junio C Hamano, Johannes Schindelin

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>

Is there a way to automate checking for make variables that are set
but never used?

Thanks,
Jonathan

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

* Re: [PATCH 1/1] mingw: drop MakeMaker reference
  2019-02-25 20:02   ` Jonathan Nieder
@ 2019-02-25 22:07     ` Johannes Schindelin
  0 siblings, 0 replies; 9+ messages in thread
From: Johannes Schindelin @ 2019-02-25 22:07 UTC (permalink / raw)
  To: Jonathan Nieder; +Cc: Johannes Schindelin via GitGitGadget, git, Junio C Hamano

Hi Jonathan,

On Mon, 25 Feb 2019, Jonathan Nieder wrote:

> 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

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

* Re: [PATCH 0/1] Drop last MakeMaker reference
  2019-02-25 19:27 [PATCH 0/1] Drop last MakeMaker reference Johannes Schindelin via GitGitGadget
  2019-02-25 19:27 ` [PATCH 1/1] mingw: drop " Johannes Schindelin via GitGitGadget
@ 2019-03-03  1:19 ` Junio C Hamano
  2019-03-07 10:10   ` Johannes Schindelin
  1 sibling, 1 reply; 9+ messages in thread
From: Junio C Hamano @ 2019-03-03  1:19 UTC (permalink / raw)
  To: Johannes Schindelin via GitGitGadget; +Cc: git

"Johannes Schindelin via GitGitGadget" <gitgitgadget@gmail.com>
writes:

> Back when we stopped using MakeMaker, we forgot one reference...
>
> Johannes Schindelin (1):
>   mingw: drop MakeMaker reference
>
>  config.mak.uname | 1 -
>  1 file changed, 1 deletion(-)
>
>
> base-commit: 8104ec994ea3849a968b4667d072fedd1e688642
> Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-146%2Fdscho%2Fno-perl-makemaker-v1
> Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-146/dscho/no-perl-makemaker-v1
> Pull-Request: https://github.com/gitgitgadget/git/pull/146

Good ;-)
Thanks.

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

* Re: [PATCH 0/1] Drop last MakeMaker reference
  2019-03-03  1:19 ` [PATCH 0/1] Drop last " Junio C Hamano
@ 2019-03-07 10:10   ` Johannes Schindelin
  2019-03-08  1:41     ` Junio C Hamano
  0 siblings, 1 reply; 9+ messages in thread
From: Johannes Schindelin @ 2019-03-07 10:10 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: Johannes Schindelin via GitGitGadget, git

Hi Junio,

On Sun, 3 Mar 2019, Junio C Hamano wrote:

> "Johannes Schindelin via GitGitGadget" <gitgitgadget@gmail.com>
> writes:
> 
> > Back when we stopped using MakeMaker, we forgot one reference...
> >
> > Johannes Schindelin (1):
> >   mingw: drop MakeMaker reference
> >
> >  config.mak.uname | 1 -
> >  1 file changed, 1 deletion(-)
> >
> >
> > base-commit: 8104ec994ea3849a968b4667d072fedd1e688642
> > Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-146%2Fdscho%2Fno-perl-makemaker-v1
> > Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-146/dscho/no-perl-makemaker-v1
> > Pull-Request: https://github.com/gitgitgadget/git/pull/146
> 
> Good ;-)
> Thanks.

Gentle reminder that this has not made it into `pu` yet...

Thanks,
Dscho

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

* Re: [PATCH 0/1] Drop last MakeMaker reference
  2019-03-07 10:10   ` Johannes Schindelin
@ 2019-03-08  1:41     ` Junio C Hamano
  2019-03-08 16:27       ` Johannes Schindelin
  0 siblings, 1 reply; 9+ messages in thread
From: Junio C Hamano @ 2019-03-08  1:41 UTC (permalink / raw)
  To: Johannes Schindelin; +Cc: Johannes Schindelin via GitGitGadget, git

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:

> Hi Junio,
>
> On Sun, 3 Mar 2019, Junio C Hamano wrote:
>
>> "Johannes Schindelin via GitGitGadget" <gitgitgadget@gmail.com>
>> writes:
>> 
>> > Back when we stopped using MakeMaker, we forgot one reference...
>> >
>> > Johannes Schindelin (1):
>> >   mingw: drop MakeMaker reference
>> >
>> >  config.mak.uname | 1 -
>> >  1 file changed, 1 deletion(-)
>> >
>> >
>> > base-commit: 8104ec994ea3849a968b4667d072fedd1e688642
>> > Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-146%2Fdscho%2Fno-perl-makemaker-v1
>> > Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-146/dscho/no-perl-makemaker-v1
>> > Pull-Request: https://github.com/gitgitgadget/git/pull/146
>> 
>> Good ;-)
>> Thanks.
>
> Gentle reminder that this has not made it into `pu` yet...

Thanks.

I'll try to make it a habit not to respond to 0/1 (but instead to
1/1) as it is quite inefficient to get to 1/1 from 0/1 at least for
me X-<.

Or perhaps GGG can learn to avoid 0/1 for a single-patch topic ;-)

Thanks anyway.  Will try to apply directly on top of master.

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

* Re: [PATCH 0/1] Drop last MakeMaker reference
  2019-03-08  1:41     ` Junio C Hamano
@ 2019-03-08 16:27       ` Johannes Schindelin
  2019-03-11 20:24         ` Jeff King
  0 siblings, 1 reply; 9+ messages in thread
From: Johannes Schindelin @ 2019-03-08 16:27 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: Johannes Schindelin via GitGitGadget, git

Hi Junio,

On Fri, 8 Mar 2019, Junio C Hamano wrote:

> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> 
> > On Sun, 3 Mar 2019, Junio C Hamano wrote:
> >
> >> "Johannes Schindelin via GitGitGadget" <gitgitgadget@gmail.com>
> >> writes:
> >> 
> >> > Back when we stopped using MakeMaker, we forgot one reference...
> >> >
> >> > Johannes Schindelin (1):
> >> >   mingw: drop MakeMaker reference
> >> >
> >> >  config.mak.uname | 1 -
> >> >  1 file changed, 1 deletion(-)
> >> >
> >> >
> >> > base-commit: 8104ec994ea3849a968b4667d072fedd1e688642
> >> > Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-146%2Fdscho%2Fno-perl-makemaker-v1
> >> > Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-146/dscho/no-perl-makemaker-v1
> >> > Pull-Request: https://github.com/gitgitgadget/git/pull/146
> >> 
> >> Good ;-)
> >> Thanks.
> >
> > Gentle reminder that this has not made it into `pu` yet...
> 
> Thanks.
> 
> I'll try to make it a habit not to respond to 0/1 (but instead to
> 1/1) as it is quite inefficient to get to 1/1 from 0/1 at least for
> me X-<.

Hehe. I do have something for you there:

https://github.com/git-for-windows/build-extra/blob/master/apply-from-public-inbox.sh

-- snip --

> Or perhaps GGG can learn to avoid 0/1 for a single-patch topic ;-)

It is easier, and more consistent, to have a cover letter even then, for
things like the broader explanation of the context, the changes since the
previous iteration, and the range-diff (which would make v2, v3, v4, etc
utterly unreadable from my point of view if they were integrated into the
single patches, as I used to do with interdiffs).

> Thanks anyway.  Will try to apply directly on top of master.

Thank you!

While we're talking about "directly on top of master"... *after* it got
some review, I would love to ask for the same favor for
https://public-inbox.org/git/pull.160.git.gitgitgadget@gmail.com

On the other hand, it is an old bug, I know, and it will break all CI
builds for branches that are based off of older commits, anyway. I guess
we'll need some sort of horrible hack in the git-sdk-64-minimal thing, to
allow for patching this on the CI side, without adding commits to all of
those branches. :-(

So: I made up my mind, and that MSYS2 runtime v3.x patch does not need to
be fast-tracked; it would not fix all the CI builds...

Ciao,
Dscho

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

* Re: [PATCH 0/1] Drop last MakeMaker reference
  2019-03-08 16:27       ` Johannes Schindelin
@ 2019-03-11 20:24         ` Jeff King
  0 siblings, 0 replies; 9+ messages in thread
From: Jeff King @ 2019-03-11 20:24 UTC (permalink / raw)
  To: Johannes Schindelin
  Cc: Junio C Hamano, Johannes Schindelin via GitGitGadget, git

On Fri, Mar 08, 2019 at 05:27:36PM +0100, Johannes Schindelin wrote:

> > Or perhaps GGG can learn to avoid 0/1 for a single-patch topic ;-)
> 
> It is easier, and more consistent, to have a cover letter even then, for
> things like the broader explanation of the context, the changes since the
> previous iteration, and the range-diff (which would make v2, v3, v4, etc
> utterly unreadable from my point of view if they were integrated into the
> single patches, as I used to do with interdiffs).

Just my two cents:

As a reviewer, I generally prefer not to see a separate cover letter for
a single patch. At least for the first version (I agree it gets unwieldy
showing a range-diff after the "---" for subsequent versions, unless it
happens to be pretty short).

My reasoning is that I've noticed that many of the GGG-sent patches
often end up duplicating quite a bit of content between the cover letter
and the commit message of the patch (or worse, putting things only in
the cover letter that really could go into the commit message). That
doubles the time I spend reading and understanding the patch's rationale
(and I'm speaking subjectively here, of course; I didn't measure it).

I don't think it's an _inherent_ problem with a separate cover letter.
But something about the workflow causes people to write up over-long
cover letters. Which presumably is the fact that they're presented with
a PR textbox to write the rationale separately from when they wrote the
commit message. So they err on the side of repeating themselves, and
never see the two pieces "together" (like the reader will), which makes
the redundancy more obvious.

I'd say 99% of the time a single-patch doesn't need any cover letter
material at all. And even a multi-patch series really just needs a tl;dr
of the problem and a sketch of the solution. In both cases, the commit
messages should carry the meat.

(That's all specific to our project, of course; I know many projects
don't care about commit messages at all and expect PR descriptions to be
the first-class explanations).

-Peff

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

end of thread, other threads:[~2019-03-11 20:24 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-25 19:27 [PATCH 0/1] Drop last MakeMaker reference Johannes Schindelin via GitGitGadget
2019-02-25 19:27 ` [PATCH 1/1] mingw: drop " Johannes Schindelin via GitGitGadget
2019-02-25 20:02   ` Jonathan Nieder
2019-02-25 22:07     ` Johannes Schindelin
2019-03-03  1:19 ` [PATCH 0/1] Drop last " Junio C Hamano
2019-03-07 10:10   ` Johannes Schindelin
2019-03-08  1:41     ` Junio C Hamano
2019-03-08 16:27       ` Johannes Schindelin
2019-03-11 20:24         ` Jeff King

git@vger.kernel.org list mirror (unofficial, one of many)

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://public-inbox.org/git
	git clone --mirror http://ou63pmih66umazou.onion/git
	git clone --mirror http://czquwvybam4bgbro.onion/git
	git clone --mirror http://hjrcffqmbrq6wope.onion/git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V1 git git/ https://public-inbox.org/git \
		git@vger.kernel.org
	public-inbox-index git

Example config snippet for mirrors.
Newsgroups are available over NNTP:
	nntp://news.public-inbox.org/inbox.comp.version-control.git
	nntp://ou63pmih66umazou.onion/inbox.comp.version-control.git
	nntp://czquwvybam4bgbro.onion/inbox.comp.version-control.git
	nntp://hjrcffqmbrq6wope.onion/inbox.comp.version-control.git
	nntp://news.gmane.io/gmane.comp.version-control.git
 note: .onion URLs require Tor: https://www.torproject.org/

code repositories for project(s) associated with this inbox:

	https://80x24.org/mirrors/git.git

AGPL code for this site: git clone https://public-inbox.org/public-inbox.git