Commit Graph

327 Commits (pu)
 

Author SHA1 Message Date
Elijah Newren e18e114919 WIP 5 years ago
Elijah Newren e817c0147c More LFS notes 5 years ago
Elijah Newren 1773b8d00f debuggery (of lfs-conversion) 5 years ago
Elijah Newren d7106b1c74 contrib README: other planned tools 5 years ago
Elijah Newren d73c18386b FIXME: Questionable bfg-ish WIP (make --convert-to-git-lfs more general) 5 years ago
Elijah Newren 01ca8ffe5c bfg-ish: WIP towards adding --convert-to-git-lfs 5 years ago
Elijah Newren 030d907869 contrib: new filter-repo demo named lfs-conversion
This rewrites history to upload some files to a git-lfs server.  It
is similar to BFG's --convert-to-git-lfs, github.com/bozaro/git-lfs-migrate,
and probably some other tools.

FIXME: Need to implement most of this still.
5 years ago
Elijah Newren faef23b2f1 filter-repo: make original changes available to callbacks
Scripts making use of filter-repo as a library may want to be able to
compare the list of file changes in a commit to the original list of
file changes before other filtering occurred.  For example, an
lfs-conversion script might want to know that e.g. --path selection and
--filename-callback said a bunch of files should be removed, so that the
lfs-conversion script can instead load those files up to git lfs.

FIXME: This slows down rails rewriting by over 60%, which is unacceptable;
need to find a different way to introduce this change.  Maybe only when
explicitly requested somehow?

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 731cd93b9a short term todo notes
Also, record which git.git testcases to run with filter-lamely
5 years ago
Elijah Newren 4de3861685 contrib: clarify a few points of usage
Make it clearer that absolute paths should not be used for pathnames
within a git repository.  Also, fix the comment about how the
insert-beginning script could be implemented as a one-liner; the
example commented-out code should have used bytestrings.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 76d48d03cc Merge branch 'en/flesh-out-docs' into master
The prerequisites and installation docs were not quite detailed enough,
and no code of conduct or contribution guidelines were included.  Flesh
out the docs to cover these issues.
5 years ago
Elijah Newren 50bda494b0 Contributing.md: mention testsuite line coverage
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 356d6f8cfa run_coverage: prefer coverage3 to python3-coverage
Some of the systems I ran on had a 'python3-coverage' and some had a
'coverage3' program.  More were of the latter name, but more
importantly, the upstream tarball only creates the latter name;
apparently the former was just added by some distros.  So, switch to the
more official name of the program.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 0a2a09d550 filter-repo (README): add code of conduct and contributing guidelines
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 4092ead1e5 Explain how to use a python3 executable not named "python3"
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 88811532c0 filter-repo (README): add sections on prerequisites and installation
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 63cec074c3 Merge branch 'mh/generated-readme-typo-fix' into master 5 years ago
Matthisk Heimensen 83a39f9233
filter-repo: fix typo in generated analysis README
Signed-off-by: Matthisk Heimensen <m@tthisk.nl>
5 years ago
Elijah Newren 3334c5cb01 Merge branch 'bf/installation-fixes' into master 5 years ago
Benoit Fouletier 2cbd4a46a7 Makefile: fix path installation issues
- quote paths that may have spaces
- force ln in case the file already exists

Signed-off-by: Benoit Fouletier <bennews@free.fr>
5 years ago
Benoit Fouletier ca2fd07dfa Makefile: fix documentation installation
- correct paths to including missing "Documentation/" prefix
- use fully specified "origin/docs" branch in case the "docs" branch is
not checked out locally

Signed-off-by: Benoit Fouletier <bennews@free.fr>
5 years ago
Elijah Newren 8d8410e2b2 Makefile: use the right token environment variable
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 84fddfe262 git-filter-repo.txt: fix typesetting of --partial
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren ceb924ea8f filter-repo (README): add link to predecessor project
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 2fc5596455 filter-repo (README): add note about requiring a recent version of git
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 904e03f963 filter-repo: workaround Windows' insistence that command args be strings
It appears that in addition to Windows requiring cwd be a string (and
not a bytestring), it also requires the command line arguments to be
unicode strings.  This appears to be a python-on-Windows issue at the
surface (attempts to quote things that assumes the arguments are all
strings), but whether it's solely a python-on-Windows issue or there is
also a deeper Windows issue, we can workaround this brain-damage by
extending the SubprocessWrapper slightly.  As with the cwd changes, only
apply this on Windows and not elsewhere because there are perfectly
legitimate reasons to pass non-unicode parameters (e.g. filenames that
are not valid unicode).

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren f2729153fe filter-repo: workaround Windows' insistence that cwd not be a bytestring
Unfortunately, it appears that Windows does not allow the 'cwd' argument
of various subprocess calls to be a bytestring.  That may be functional
on Windows since Windows-related filesystems are allowed to require that
all file and directory names be valid unicode, but not all platforms
enforce such restrictions.  As such, I certainly cannot change
   cwd=directory
to
   cwd=decode(directory)
because that could break on other platforms (and perhaps even on Windows
if someone is trying to read a non-native filesystem).  Instead, create
a SubprocessWrapper class that will always call decode on the cwd
argument before passing along to the real subprocess class.  Use these
wrappers on Windows, and do not use them elsewhere.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren da2a969157 Makefile: add a few new targets to streamline my release workflow
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren d70b29a165 filter-repo: fix import sort order
During the python3 transition, StringIO was renamed to io -- but the
import wasn't moved to preserve appropriate sorting.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren e333be7b17 filter-repo: consistently use bytestrings for directory names
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren e6dd613e3f filter-repo: add a --version option
Note that this isn't a version *number* or even the more generalized
version string that folks are used to seeing, but a version hash (or
leading portion thereof).

A few import points:

  * These version hashes are not strictly monotonically increasing
    values.  Like I said, these aren't version numbers.  If that
    bothers you, read on...

  * This scheme has incredibly nice semantics satisfying a pair of
    properties that most version schemes would assume are mutually
    incompatible:
       This scheme works even if the user doesn't have a clone of
       filter-repo and doesn't require any build step to inject the
       version into the program; it works even if people just download
       git-filter-repo.py off GitHub without any of the other sources.
    And:
       This scheme means that a user is running precisely version X of
       the code, with the version not easily faked or misrepresented
       when third parties edit the code.
    Given the wonderful semantics provided by satisfying this pair of
    properties that all other versioning schemes seem to miss out on, I
    think I should name this scheme.  How about "Semantic Versioning"?
    (Hehe...)

  * The version hash is super easy to use; I just go to my own clone of
    filter-repo and run either:
        git show $VERSION_HASH
    or
        git describe $VERSION_HASH

  * A human consumable version might suggest to folks that this software
    is something they might frequently use and upgrade.  This program
    should only be used in exceptional cases (because rewriting history
    is not for the faint of heart).

  * A human consumable version (i.e. a version number or even the
    more relaxed version strings in more common use) might suggest to
    folks that they can rely on strict backward compatibility.  It's
    nice to subtly undercut any such assumption.

  * Despite all that, I will make releases (downloadable tarballs with
    real version numbers in the tarball name; I'm just going to re-use
    whatever version git is released with at the time).  But those
    version numbers won't be used by the --version option; instead the
    version hash will.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 1e21d6e2ec Add installation instructions
Try to make it a little more friendly for distros to package.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 62c311c69f filter-repo: fix an unmarked bytestring to be marked as such
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren e0140bb2ad git-filter-repo.txt: minor updates to docs
A few changes:
  * Include notes about git-2.24.0 changes
  * Make it clearer that messing with the first parent could have
    negative side-effects if the file_changes aren't also updated.
  * Fix wrapping of a line that was too long.

Also, update the README.md:
  * Note the upstream improvements made in (not yet released) git-2.24.0

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 320c85f941 filter-repo: improve support for partial history rewrites
Partial history rewrites were possible before with the (previously
hidden) --refs flag, but the defaults were wrong.  That could be worked
around with the --source or --target flags, but that disabled --no-data
for fast-export and thus slowed things down, and also would require
overridding --replace-refs.  And the defaults for --source and --target
may diverge further from what is wanted/needed for partial history
rewrites in the future.

So, add --partial as a first-class supported option with scary
documentation about how it permits mixing new and old history.  Make
--refs imply that flag.  Make the behavioral similarities (in regards to
which steps are skipped) between --source, --target, and --partial more
clear.  Add relevant documentation to round it out.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 509a624b6a filter-repo: fix issue with pruning of empty commits
In order to build the correct tree for a commit, git-fast-import always
takes a list of file changes for a merge commit relative to the first
parent.

When the entire first-parent history of a merge commit is pruned away
and the merge had paths with no difference relative to the first parent
but which differed relative to later parents, then we really need to
generate a new list of file changes in order to have one of those other
parents become the new first parent.  An example might help clarify...

Let's say that there is a merge commit, and:

  * it resolved differences in pathA between its two parents by taking
    the version of pathA from the first parent.

  * pathB was added in the history of the second parent (it is not
    present in the first parent) and is NOT included in the merge commit
    (either being deleted, or via rename treated as deleted and added as
    something else)

For this merge commit, neither pathA nor pathB differ from the first
parent, and thus wouldn't appear in the list of file changes shown by
fast-export.  However, when our filtering rules determine that the first
parent (and all its parents) should be pruned away, then the second
parent has to become the new first parent of the merge commit.  But to
end up with the right files in the merge commit despite using a
different parent, we need a list of file changes that specifies the
changes for both pathA and pathB.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren cdec483573 filter-repo: use our own textdomain for translations
git.git wants to move more towards core-only rather than batteries
included, and as such, filter-repo will not be part of the git
distribution.  Therefore, due to keeping the projects apart, there will
need to be separate translation files (assuming filter-repo ever gains
any translations) and as such we will need a different textdomain
definition.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 71bb8d26a9 filter-repo: add a --state-branch option for incremental exporting
Allow folks to periodically update the export of a live repo without
re-exporting from the beginning.  This is a performance improvement, but
can also be important for collaboration.  For example, for sensitivity
reasons, folks might want to export a subset of a repo and update the
export periodically.  While this could be done by just re-exporting the
repository anew each time, there is a risk that the paths used to
specify the wanted subset might need to change in the future; making the
user verify that their paths (including globs or regexes) don't also
pick up anything from history that was previously excluded so that they
don't get a divergent history is not very user friendly.  Allowing them
to just export stuff that is new since the last export works much better
for them.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren b5b0cf4230 filter-repo (README): restructure and update
Since we now have a separate user manual and it does not make sense to
duplicate information in multiple places, restructure the README:
  * Refer to the actual manual early on
  * Limit the README to mostly be about why I wrote it and why folks
    might want to consider it instead of existing tools
  * Include a new section on upstream improvements, especially since it
    looks like inclusion of git-filter-repo in git.git is unlikely.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren a6a6a1b0f6 git-filter-repo.txt: add a manpage for the filter-repo command
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 622afca813 contrib: new filter-repo demo named bfg-ish
This implements most of BFG Repo Cleaner, with several additional
features and bugfixes included.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren df575fb181 contrib: new filter-repo demo named filter-lamely (or filter-branch-ish)
This is a re-implementation of git-filter-branch that is nearly
perfectly bug compatible (it can replace git-filter-branch and still
pass the git testsuite).  It deviates in one minor way that should not
matter to real world usecases, but allows it to run a few times faster
than filter-branch.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 65f0ecaef7 filter-repo: updates and minor fixes in option help and README
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 6d231c0a94 contrib: simple examples of tools based on filter-repo
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 2094221721 filter-repo: do not claim we are repacking if we are not
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 9fbe2569ec filter-repo: split repacking logic into a separate function for reuse
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 6ba30e9b98 filter-repo: use more versatile commit rename function
Being able to find the new commit hash for either an abbreviated commit
hash or a full commit hash is much more useful than only working for a
full commit hash.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 47406a6cc0 filter-repo: re-license
When I wrote git_fast_filter.py, I was unaware of and did not forsee
libgit2.  So, although the license said the project could be used under
whatever license git.git was, there was still a potential barrier for
usage by libgit2.  I'm not sure if libgit2 will ever want to use
filter-repo, but I don't want the barrier there and I would like to
avoid a repeat of this problem.  (Also, since filter-repo is for the
most part a one-shot usage tool, I doubt that the normal copyleft
provisions could provide much value.)

MIT is widely used, compatible with just about everyting, and is
preferred by Palantir (my current employer) for open source
contributions.  So, I contacted all other contributors (Jim is still at
Sandia) and got permission to relicense.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren 7a12d7a38b filter-repo: add ability to parse and dump encoding
Commit 346f2ba891 (filter-repo: make reencoding of commit messages
togglable, 2019-05-11) made reencoding of commit messages togglable but
forgot to add parsing and outputting of the encoding header itself.  Add
such ability now.

Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago
Elijah Newren e9678a367f filter-repo: support deleteall directive
Signed-off-by: Elijah Newren <newren@gmail.com>
5 years ago