{"thread":{"id":"38462","subject":"[PATCH] INSTALL: minor typo fix","startedAt":"2015-01-27T16:15:12Z","lastAt":"2015-01-27T17:17:44Z","messageCount":4,"participants":["Alexander Kuleshov","Chris Packham","Jeff King"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"255333","messageId":"1422375312-3798-1-git-send-email-kuleshovmail@gmail.com","threadId":"38462","inReplyTo":null,"subject":"[PATCH] INSTALL: minor typo fix","fromName":"Alexander Kuleshov","fromEmail":"kuleshovmail@gmail.com","sentAt":"2015-01-27T16:15:12Z","receivedAt":"2015-01-27T16:15:12Z","isPatch":true,"sender":{"key":"kuleshovmail@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2699235?v=4"},"body":"Signed-off-by: Alexander Kuleshov <kuleshovmail@gmail.com>\n---\n INSTALL | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/INSTALL b/INSTALL\nindex ffb071e..6f1c3d5 100644\n--- a/INSTALL\n+++ b/INSTALL\n@@ -53,7 +53,7 @@ or\n \n As a caveat: a profile-optimized build takes a *lot* longer since the\n git tree must be built twice, and in order for the profiling\n-measurements to work properly, ccache must be disabled and the test\n+measurements to work properly, cache must be disabled and the test\n suite has to be run using only a single CPU.  In addition, the profile\n feedback build stage currently generates a lot of additional compiler\n warnings.\n-- \n2.3.0-rc1\n"},{"id":"255335","messageId":"CAFOYHZBiMvU5=EpX7qn0wwrUwfZ2xzATKqRo4zPH+xmro7wVbA@mail.gmail.com","threadId":"38462","inReplyTo":"1422375312-3798-1-git-send-email-kuleshovmail@gmail.com","subject":"Re: [PATCH] INSTALL: minor typo fix","fromName":"Chris Packham","fromEmail":"judge.packham@gmail.com","sentAt":"2015-01-27T17:09:52Z","receivedAt":"2015-01-27T17:09:52Z","isPatch":true,"sender":{"key":"judge.packham@gmail.com","avatar":"https://avatars.githubusercontent.com/u/155667?v=4"},"body":"Hi Alexander\n\nOn Wed, Jan 28, 2015 at 5:15 AM, Alexander Kuleshov\n<kuleshovmail@gmail.com> wrote:\n> Signed-off-by: Alexander Kuleshov <kuleshovmail@gmail.com>\n> ---\n>  INSTALL | 2 +-\n>  1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> diff --git a/INSTALL b/INSTALL\n> index ffb071e..6f1c3d5 100644\n> --- a/INSTALL\n> +++ b/INSTALL\n> @@ -53,7 +53,7 @@ or\n>\n>  As a caveat: a profile-optimized build takes a *lot* longer since the\n>  git tree must be built twice, and in order for the profiling\n> -measurements to work properly, ccache must be disabled and the test\n> +measurements to work properly, cache must be disabled and the test\n>  suite has to be run using only a single CPU.  In addition, the profile\n>  feedback build stage currently generates a lot of additional compiler\n>  warnings.\n> --\n\nThat's not a typo ccache[1] is a compiler cache tool that can be used\nto speed up rebuilding object files. I don't know anything about the\nprofile enabled builds but I imagine it has something to do with\nneeding the 2nd build to produce difference object files from the\nfirst (with ccache enabled I imagine the 2nd build would result in\nidentical output since on source files are changed between builds).\n\n--\n[1] - https://ccache.samba.org/\n"},{"id":"255336","messageId":"CAFOYHZAWT+7tKYArtRK7hp5NTvLYQGv__9E+4-7zuVGnzQ7BzQ@mail.gmail.com","threadId":"38462","inReplyTo":"CAFOYHZBiMvU5=EpX7qn0wwrUwfZ2xzATKqRo4zPH+xmro7wVbA@mail.gmail.com","subject":"Re: [PATCH] INSTALL: minor typo fix","fromName":"Chris Packham","fromEmail":"judge.packham@gmail.com","sentAt":"2015-01-27T17:15:40Z","receivedAt":"2015-01-27T17:15:40Z","isPatch":true,"sender":{"key":"judge.packham@gmail.com","avatar":"https://avatars.githubusercontent.com/u/155667?v=4"},"body":"On Wed, Jan 28, 2015 at 6:09 AM, Chris Packham <judge.packham@gmail.com> wrote:\n> Hi Alexander\n>\n> On Wed, Jan 28, 2015 at 5:15 AM, Alexander Kuleshov\n> <kuleshovmail@gmail.com> wrote:\n>> Signed-off-by: Alexander Kuleshov <kuleshovmail@gmail.com>\n>> ---\n>>  INSTALL | 2 +-\n>>  1 file changed, 1 insertion(+), 1 deletion(-)\n>>\n>> diff --git a/INSTALL b/INSTALL\n>> index ffb071e..6f1c3d5 100644\n>> --- a/INSTALL\n>> +++ b/INSTALL\n>> @@ -53,7 +53,7 @@ or\n>>\n>>  As a caveat: a profile-optimized build takes a *lot* longer since the\n>>  git tree must be built twice, and in order for the profiling\n>> -measurements to work properly, ccache must be disabled and the test\n>> +measurements to work properly, cache must be disabled and the test\n>>  suite has to be run using only a single CPU.  In addition, the profile\n>>  feedback build stage currently generates a lot of additional compiler\n>>  warnings.\n>> --\n>\n> That's not a typo ccache[1] is a compiler cache tool that can be used\n> to speed up rebuilding object files. I don't know anything about the\n> profile enabled builds but I imagine it has something to do with\n> needing the 2nd build to produce difference object files from the\n\ns/difference/different/\n\n> first (with ccache enabled I imagine the 2nd build would result in\n> identical output since on source files are changed between builds).\n>\n> --\n> [1] - https://ccache.samba.org/\n\nThe commit that added that note f2d713fc[1] has a bit more of an\nexplanation of why profile enabled builds and ccache are incompatible.\n\n[1] - https://github.com/git/git/commit/f2d713fc3e8e0b7be89584f04b421808aa99c403\n"},{"id":"255337","messageId":"20150127171743.GA2313@peff.net","threadId":"38462","inReplyTo":"CAFOYHZBiMvU5=EpX7qn0wwrUwfZ2xzATKqRo4zPH+xmro7wVbA@mail.gmail.com","subject":"Re: [PATCH] INSTALL: minor typo fix","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2015-01-27T17:17:44Z","receivedAt":"2015-01-27T17:17:44Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Wed, Jan 28, 2015 at 06:09:52AM +1300, Chris Packham wrote:\n\n> On Wed, Jan 28, 2015 at 5:15 AM, Alexander Kuleshov\n> <kuleshovmail@gmail.com> wrote:\n> > Signed-off-by: Alexander Kuleshov <kuleshovmail@gmail.com>\n> > ---\n> >  INSTALL | 2 +-\n> >  1 file changed, 1 insertion(+), 1 deletion(-)\n> >\n> > diff --git a/INSTALL b/INSTALL\n> > index ffb071e..6f1c3d5 100644\n> > --- a/INSTALL\n> > +++ b/INSTALL\n> > @@ -53,7 +53,7 @@ or\n> >\n> >  As a caveat: a profile-optimized build takes a *lot* longer since the\n> >  git tree must be built twice, and in order for the profiling\n> > -measurements to work properly, ccache must be disabled and the test\n> > +measurements to work properly, cache must be disabled and the test\n> >  suite has to be run using only a single CPU.  In addition, the profile\n> >  feedback build stage currently generates a lot of additional compiler\n> >  warnings.\n> > --\n> \n> That's not a typo ccache[1] is a compiler cache tool that can be used\n> to speed up rebuilding object files. I don't know anything about the\n> profile enabled builds but I imagine it has something to do with\n> needing the 2nd build to produce difference object files from the\n> first (with ccache enabled I imagine the 2nd build would result in\n> identical output since on source files are changed between builds).\n\nYes, ccache creates a cache key for each file based on the command line\nflags to the compiler, and the output of the pre-processor when run on\nthe file. The profiling data generated by the first run is outside of\nwhat ccache knows about, so it can't include that information in its\ncache key (and consequently, we get false cache hits).\n\nSo yeah. Not a typo.\n\n-Peff\n"}]}