git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 1/2] contrib/subtree: parse using --stuck-long

From
PWPatrik Weiskircher <patrik.weiskircher@nutrient.io>
Date
Jun 4, 2025, 15:43 UTC
Message-ID
<CANMzfzgOfb1O1cQTmDFRDWEDLO-J6E_iq-MRQimhpmgAkU8pYA@mail.gmail.com>
In-Reply-To
<xmqqcybjcy5q.fsf@gitster.g>
On Wed, Jun 4, 2025 at 11:41 AM Junio C Hamano <gitster@pobox.com> wrote:
Show 41 quoted lines
>
> "Kristoffer Haugsbakk" <kristofferhaugsbakk@fastmail.com> writes:
>
> > On Wed, Jun 4, 2025, at 15:56, Patrik Weiskircher wrote:
> >>>
> >>> Here we mention "-S", but that flag isn't implemented yet, right?
> >>>
> >>> Perhaps something like:
> >>>
> >>>     Optional parameter handling only works unambiguous with git rev-parse
> >>>     --parseopt when using the --stuck-long option. To prepare for future commits
> >>>     which add flags with optional parameters, parse with --stuck-long.
> >>>
> >>
> >> Makes sense! Changing that. What is a good policy to resubmit
> >> something? Should I wait longer? Sorry, very new here!
> >
> > • Force-push your branch to gitgitgadget
> > • Edit the PR description with something like “Changes since v1:” to
> >   summarize the changes
> > • (`/preview` comment)
> > • To send the next version: `/submit` comment again
> >
> > I think that’s it. :)
> >
> > I don’t think there’s a need to wait if you don’t want to.
>
> It would be nice for potential reviewers to give at least 24 hours
> to ensure people anywhere on the globe have a chance to comment, and
> a chance for you to respond to them, before sending your next
> iteration.
>
> Also, for future reference, when responding to a review comment that
> causes you to drastically change the course of the series, you can
> respond whenever you want to, but it is nice to other potential
> reviewers to give at least 24 hours to voice their opinions, before
> sending an updated series based on that comment, since suggested
> changes in such a comment may be controversial and after seeing you
> spend some time already to adjust to it, others may feel discouraged
> to make you redo your series again even whey they think the
> suggested changes are not taking us in the right direction.
Makes sense! I'll keep that in mind for the future!
Previous: Junio C HamanoNext: Patrik Weiskircher via GitGitGadget
Message 7 of 12 in “contrib/subtree: Add -S/-gpg-sign”
  1. 0/2 contrib/subtree: Add -S/-gpg-signPatrik Weiskircher via GitGitGadget, Jun 2, 2025
  2. 1/2 contrib/subtree: parse using --stuck-longPatrik Weiskircher via GitGitGadget, Jun 2, 2025
  3. D. Ben KnobleJun 3, 2025
  4. Patrik WeiskircherJun 4, 2025
  5. Kristoffer HaugsbakkJun 4, 2025
  6. Junio C HamanoJun 4, 2025
  7. Patrik WeiskircherJun 4, 2025
  8. 2/2 contrib/subtree: add -S/--gpg-signPatrik Weiskircher via GitGitGadget, Jun 2, 2025
  9. 0/2 contrib/subtree: Add -S/-gpg-signPatrik Weiskircher via GitGitGadget, Jun 4, 2025
  10. 1/2 contrib/subtree: parse using --stuck-longPatrik Weiskircher via GitGitGadget, Jun 4, 2025
  11. 2/2 contrib/subtree: add -S/--gpg-signPatrik Weiskircher via GitGitGadget, Jun 4, 2025
  12. Junio C HamanoJun 4, 2025

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.