Re: What's cooking in git.git (Nov 2021, #03; Tue, 9)

2 messages, 2 authors, 2021-11-11 · open the first message on its own page

Re: What's cooking in git.git (Nov 2021, #03; Tue, 9)

From: Junio C Hamano <hidden>
Date: 2021-11-11 22:58:42

Emily Shaffer [off-list ref] writes:
quoted
* gc/remote-with-fewer-static-global-variables (2021-10-28) 6 commits
 - remote: add struct repository parameter to external functions
 - remote: die if branch is not found in repository
 - remote: remove the_repository->remote_state from static methods
 - remote: use remote_state parameter internally
 - remote: move static variables into per-repository struct
 - t5516: add test case for pushing remote refspecs

 Code clean-up to eventually allow information on remotes defined
 for an arbitrary repository to be read.

 Will merge to 'next'?
What's still blocking this from being merged? Anything? Release
schedule?
I do not think I saw anybody giving sufficient review on any
previous rounds or the latest round.  I did read some part and
commented on earlier rounds, during the -rc period, I don't have
enough concentration to judge a new design properly myself.

So, in short, these patches are unreviewed yet, if I am not
mistaken.

Thanks.

Re: What's cooking in git.git (Nov 2021, #03; Tue, 9)

From: Glen Choo <hidden>
Date: 2021-11-11 23:44:22

Junio C Hamano [off-list ref] writes:
Emily Shaffer [off-list ref] writes:
quoted
quoted
* gc/remote-with-fewer-static-global-variables (2021-10-28) 6 commits
 - remote: add struct repository parameter to external functions
 - remote: die if branch is not found in repository
 - remote: remove the_repository->remote_state from static methods
 - remote: use remote_state parameter internally
 - remote: move static variables into per-repository struct
 - t5516: add test case for pushing remote refspecs

 Code clean-up to eventually allow information on remotes defined
 for an arbitrary repository to be read.

 Will merge to 'next'?
What's still blocking this from being merged? Anything? Release
schedule?
I do not think I saw anybody giving sufficient review on any
previous rounds or the latest round.  I did read some part and
commented on earlier rounds, during the -rc period, I don't have
enough concentration to judge a new design properly myself.

So, in short, these patches are unreviewed yet, if I am not
mistaken.
Thanks for clarifying, Emily and I were under the impression that this
would be merged soon. I'll ask for more reviews.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help