>>This looks like a good cleanup, but should be split into another cleanup >>commit. It looks unrelated. Agree. Will split up in next patchset. >>You've got all sorts of mixed space/tab indent here. Yes. Will fix wrong indents in next patchset. >>Isn't accepting http and https the default? I think it's NOT after I took a view at function `fetch_pack_config`. This may be optimized in another patch I think and also will listen to some other suggestions. Thank you.