[PATCH] Record repo.snapshot-prefix in the per-repo config

[john at keeping.me.uk (John Keeping)]

On Tue, Jul 17, 2018 at 12:38:22PM -0400, Konstantin Ryabitsev wrote:
> Even if we find snapshot-prefix in the repo configuration, we are not
> writing it out into the rc- file, so setting the value does not have any
> effect.
> 
> Signed-off-by: Konstantin Ryabitsev <konstantin at linuxfoundation.org>

Reviewed-by: John Keeping <john at keeping.me.uk>

To clarify why this is a problem, the repo list is cached using our
normal caching mechanism so that we don't scan for repositories on every
(uncached) request.  Without this patch, only the initial request which
generates a new repo list will work and any subsequent requests using
the cached list will ignore the setting.

Of course, if caching is disabled then everything works even without
this change (guess how my test environment is set up...!).

> ---
>  cgit.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/cgit.c b/cgit.c
> index e2d7891..a39d83a 100644
> --- a/cgit.c
> +++ b/cgit.c
> @@ -830,6 +830,8 @@ static void print_repo(FILE *f, struct cgit_repo *repo)
>  		fprintf(f, "repo.snapshots=%s\n", tmp ? tmp : "");
>  		free(tmp);
>  	}
> +	if (repo->snapshot_prefix)
> +		fprintf(f, "repo.snapshot-prefix=%s\n", repo->snapshot_prefix);
>  	if (repo->max_stats != ctx.cfg.max_stats)
>  		fprintf(f, "repo.max-stats=%s\n",
>  		        cgit_find_stats_periodname(repo->max_stats));
> -- 
> 2.17.1