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

Re: [PATCH 1/3] Prepare for non-interactive merge-preserving rebase

From
Stephen Haberman <stephen@exigencecorp.com>
Date
Sep 27, 2008, 19:20 UTC
Message-ID
<20080927142007.a75a31a0.stephen@exigencecorp.com>
In-Reply-To
<48DE7386.2080808@op5.se>
> Stephen, are you using this in production?

Kind of--I have not distributed a patched version of pull. But I have written test cases on our side and manually executing `GIT_EDITOR=: git rebase -i -p` works very well.

Past occurrences aside, no one has needed to rebase a local merge yet.
> How's it turning out?

I think it's great, but the primary problem will be getting devs to actually remember to use it. E.g. I don't think they will type out:

    git pull --rebase --preserve-rebase
Every time they pull. And they definitely don't do our current hack:
    git fetch
    GIT_EDITOR=: git rebase -i -p

I do have a wrapper shell script for people to use, but it hasn't seen wide adoption yet. We have a draconian hook script that tries to detect merges that should have been rebases and reject them, but it's disabled for tweaking right now--when it gets turned back on, I think more people will use the script.

In the long term, having "branch.name.preservemerges" and "branch.autosetuppreservemerges" config options to parallel the "branch.name.rebase" option and get us back to just "git pull" would be great.

I've been meaning to submit patches for these two config options--I figure I can hunt down how "branch.name.rebase" works and do the appropriate copy/paste, but I haven't dedicated any time to it yet.

Thanks, Stephen

Previous: Andreas EricssonNext: Shawn O. Pearce
Message 9 of 13 in “Prepare for non-interactive merge-preserving rebase”
  1. 1/3 Prepare for non-interactive merge-preserving rebaseAndreas Ericsson, Sep 23, 2008
  2. Stephen HabermanSep 23, 2008
  3. Andreas EricssonSep 23, 2008
  4. SZEDER GáborSep 24, 2008
  5. Johannes SixtSep 24, 2008
  6. Andreas EricssonSep 24, 2008
  7. Stephen HabermanOct 15, 2008
  8. Andreas EricssonSep 27, 2008
  9. Stephen HabermanSep 27, 2008
  10. Shawn O. PearceSep 29, 2008
  11. Andreas EricssonSep 29, 2008
  12. Shawn O. PearceSep 29, 2008
  13. Add branch.autosetuppreservemerges and branch.<name>.preservemerges.Stephen Haberman, Oct 1, 2008

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.