# [PATCH] clean cached refs when calling set_git_dir() 4 messages from 2011-03-10 to 2011-03-10. Participants: Julius Plenz, Nguyen Thai Ngoc Duy, Lars Hjemli. Thread: https://gitlist.dev/t/26703 ## Julius Plenz, 2011-03-10 15:19 Subject: [PATCH] clean cached refs when calling set_git_dir() Message-ID: <1299770345-32055-1-git-send-email-plenz@cis.fu-berlin.de> URL: https://gitlist.dev/e/1299770345-32055-1-git-send-email-plenz%40cis.fu-berlin.de ``` If you use libgit.a to perform reference resolutions on two or more repositories that contain packed refs, cached_refs will store the packed refs for the first repository to contain a packed-refs file only. Subsequent calls to get_packed_refs() will return the *cached* list, even when you changed to a different repository with set_git_dir(). Thus, references will almost certainly resolve to non-existent commits. The patch explicitly invalidates the cached list when calling set_git_dir() which calls setup_git_env(). Signed-off-by: Julius Plenz --- cache.h | 1 + environment.c | 1 + refs.c | 2 +- 3 files changed, 3 insertions(+), 1 deletions(-) diff --git a/cache.h b/cache.h index 08a9022..95491fe 100644 --- a/cache.h +++ b/cache.h @@ -881,6 +881,7 @@ extern int check_leading_path(const char *name, int len); extern int has_dirs_only_path(const char *name, int len, int prefix_len); extern void schedule_dir_for_removal(const char *name, int len); extern void remove_scheduled_dirs(void); +extern void invalidate_cached_refs(void); extern struct alternate_object_database { struct alternate_object_database *next; diff --git a/environment.c b/environment.c index 9564475..3a7e1dc 100644 --- a/environment.c +++ b/environment.c @@ -110,6 +110,7 @@ static void setup_git_env(void) git_graft_file = git_pathdup("info/grafts"); if (getenv(NO_REPLACE_OBJECTS_ENVIRONMENT)) read_replace_refs = 0; + invalidate_cached_refs(); } int is_bare_repository(void) diff --git a/refs.c b/refs.c index e3c0511..bbe34fb 100644 --- a/refs.c +++ b/refs.c @@ -171,7 +171,7 @@ static void free_ref_list(struct ref_list *list) } } -static void invalidate_cached_refs(void) +void invalidate_cached_refs(void) { struct cached_refs *ca = &cached_refs; -- 1.7.3.1 ``` ## Nguyen Thai Ngoc Duy, 2011-03-10 15:59 Subject: Re: [PATCH] clean cached refs when calling set_git_dir() Message-ID: URL: https://gitlist.dev/e/AANLkTik1HMQqKc98AF-6DTGJgrk8HOjZTimGnK-XBv-q%40mail.gmail.com In-Reply-To: <1299770345-32055-1-git-send-email-plenz@cis.fu-berlin.de> ``` On Thu, Mar 10, 2011 at 10:19 PM, Julius Plenz wrote: > If you use libgit.a to perform reference resolutions on two or more > repositories that contain packed refs, cached_refs will store the packed > refs for the first repository to contain a packed-refs file only. If you use libgit.a to do anything on more than one repository, you get a lot more messed up that just that. Spawning a new process or accessing with libgit2 may be safer. Why do you want to do that in the first place? -- Duy ``` ## Julius Plenz, 2011-03-10 16:09 Subject: Re: [PATCH] clean cached refs when calling set_git_dir() Message-ID: <20110310160948.GL15960@plenz.com> URL: https://gitlist.dev/e/20110310160948.GL15960%40plenz.com In-Reply-To: ``` Hi! * Nguyen Thai Ngoc Duy [2011-03-10 17:00]: > On Thu, Mar 10, 2011 at 10:19 PM, Julius Plenz wrote: > > If you use libgit.a to perform reference resolutions on two or more > > repositories that contain packed refs, cached_refs will store the packed > > refs for the first repository to contain a packed-refs file only. > > If you use libgit.a to do anything on more than one repository, you > get a lot more messed up that just that. Spawning a new process or > accessing with libgit2 may be safer. Why do you want to do that in the > first place? I was working on the cgit repository browser, where you have to peek into several repositories to create the overview page. Since cgit links to libgit.a I fixed the bug there. Not sure if that's the proper way to do it, though... Julius ``` ## Lars Hjemli, 2011-03-10 17:14 Subject: Re: [PATCH] clean cached refs when calling set_git_dir() Message-ID: URL: https://gitlist.dev/e/AANLkTikrTva2KFxh0O%3DRyn1zWZR2rjCSVkGjHQ0w0xL0%40mail.gmail.com In-Reply-To: <20110310160948.GL15960@plenz.com> ``` On Thu, Mar 10, 2011 at 17:09, Julius Plenz wrote: > * Nguyen Thai Ngoc Duy [2011-03-10 17:00]: >> On Thu, Mar 10, 2011 at 10:19 PM, Julius Plenz wrote: >> > If you use libgit.a to perform reference resolutions on two or more >> > repositories that contain packed refs, cached_refs will store the packed >> > refs for the first repository to contain a packed-refs file only. >> >> If you use libgit.a to do anything on more than one repository, you >> get a lot more messed up that just that. Spawning a new process or >> accessing with libgit2 may be safer. Why do you want to do that in the >> first place? > > I was working on the cgit repository browser, where you have to peek > into several repositories to create the overview page. Since cgit > links to libgit.a I fixed the bug there. Not sure if that's the proper > way to do it, though... I never got around to answer your earlier email about this, but here is what I would have said ;) Since libgit.a doesn't work with multiple repos [1], cgit supports the use of an "agefile" (that's the option name in cgitrc) to obtain the last commit-date per repo. The content of such an agefile can be generated by a command like the following in one (or more) of your git hooks: git for-each-ref --sort=-authordate --format='%(authordate:iso8601)' --count=1 Hth. -- larsh [1] Even if libgit.a did support working with multiple repos in a single run, I probably would avoid this particular usage since it is bound to perform badly when lots of repos are involved (like on cgit.freedesktop.org), and even more so when the list is sorted by idle-time. Simply reading a small textfile per repo is much faster. ```