• psycotica0@lemmy.ca
    link
    fedilink
    arrow-up
    1
    ·
    edit-2
    3 days ago

    Interesting. Yeah, sounds like what git blame -C is for, so I’ve never made copies when splitting files, I’ve just moved lines between files naively. But I guess if one’s tools are limited and doesn’t have the ability to -C, then I guess I could respect the hack that is that solution?

    I mean, I’m 99% sure git doesn’t store blame or authorship info in the pack files, even as a convenience cache, and just guesses by traversing the patch log with heuristics live when you run blame anyway, so the history mostly doesn’t matter there, but the way you’ve done it does seem to have tricked the heuristics into doing what you want without relying on an option, so that’s neat! It’s an interesting hack, and I like interesting hacks 😛

    By the way, if there are down votes, they’re not from me!

    • JATothrim_v2@programming.dev
      link
      fedilink
      arrow-up
      1
      ·
      edit-2
      2 days ago

      I’m not a copyright-lawyer, but I think there are implications on who has authored the code, so preserving this detail can be important. The fancy copy reduced my blame by +90% on the final result.

      git blame output can be affected by e.g. ignoring white-space changes.

      • psycotica0@lemmy.ca
        link
        fedilink
        arrow-up
        1
        ·
        2 days ago

        Right, but what I’m saying is that git doesn’t store authorship information or line-by-line history, no matter how it’s done. Figuring out which line came from where is an algorithm the git blame command does every time you request it, and that algorithm can give different results depending on which options you give the blame command. And so what you’ve found here is a collection of commits that produces a situation the default blame algorithm can follow, without any optional flags, which is neat! Maybe not great for git history, but neat!