On Mon, Dec 2, 2024 at 3:21 PM Richard Purdie via lists.openembedded.org <[email protected]> wrote: > > buildhistory used to result in many individual objects in the repo which > could affect performance. This gc call was added over a decade ago to avoid > those performance issues. Modern git calls git-gc when needed and this kind > of workaround should no longer be needed. > > There is a small chance this is contributing to problems with the autobuilder > buildhistory repositories, so simplifying the code may help that too. >
I do run into situations on git repos often that requires running git gc to fix them but I think repo optimization should be left to user on those grounds I agree to remove it. > Signed-off-by: Richard Purdie <[email protected]> > --- > meta/classes/buildhistory.bbclass | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/meta/classes/buildhistory.bbclass > b/meta/classes/buildhistory.bbclass > index ce3abaa69d4..d735dd5fb5d 100644 > --- a/meta/classes/buildhistory.bbclass > +++ b/meta/classes/buildhistory.bbclass > @@ -877,7 +877,6 @@ END > # Porcelain output looks like "?? packages/foo/bar" > # Ensure we commit metadata-revs with the first commit > buildhistory_single_commit "$CMDLINE" "$HOSTNAME" > dummy > - git gc --auto --quiet > else > buildhistory_single_commit "$CMDLINE" "$HOSTNAME" > fi > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#208167): https://lists.openembedded.org/g/openembedded-core/message/208167 Mute This Topic: https://lists.openembedded.org/mt/109891145/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
