X-Git-Url: https://arthur.barton.de/gitweb/?a=blobdiff_plain;f=HACKING;h=372bc732a5188e164e0b33f2d7a8a390e7d3d6d7;hb=1dc1f11918198d4a192e285de2c393dd6dc33a8b;hp=f5952cdf39604a5a59a6dfc7c0eb69b73525c6b1;hpb=61c70d68c62526570948a76b11f1d218bca52148;p=bup.git diff --git a/HACKING b/HACKING index f5952cd..372bc73 100644 --- a/HACKING +++ b/HACKING @@ -7,38 +7,47 @@ Code Branching Model The master branch is what we consider the main-line of development, and the last, non-rc tag on master is the most recent stable release. -Of course, in all fairness, it has been a *long* time since the last -stable release, but we're working fairly hard to fix that -- no, -seriously. - Any branch with a "tmp/" prefix might be rebased (often), so keep that in mind when using or depending on one. +Any branch with a "tmp/review/" prefix corresponds to a patchset +submitted to the mailing list. We try to maintain these branches to +make the review process easier for those not as familiar with patches +via email. + Current Trajectory ================== -At the moment, the primary goal is to test master in preparation for a -0.25 release, which among many other things will include more complete -support for filesystem metadata. +Now that we've finished the 0.32 release, we're working on 0.33, and +although we're not certain which new features will be included, we're +considering: -If you have the time and inclination, please help review patches -posted to the list for inclusion in 0.25. (See "ways to help" below.) + - Better VFS performance for large repositories (i.e. fuse, ls, + web...). + + - Better VFS caching. + + - Index improvements. + + - Incremental indexing via inotify. -We'd like to try to release (0.25) soon, so we're limiting the scope -of prospective changes -- definitely in-scope: + - Smarter (and quieter) handling of cross-filesystem metadata. - - fixes to the new metadata support - - fixes for regressions (portability included) - - fixes for "serious" bugs - - "simple" fixes - - documentation improvements + - Encryption. + + - Support for alternate remote storage APIs. + + - Discontinuing Python 2 work, excepting perhaps some bugfixes. + +If you have the time and inclination, please help review patches +posted to the list, or post your own. (See "ways to help" below.) More specific ways to help ========================== -Testing -- yes please. +Testing -- yes please. With respect to patches, bup development is handled via the mailing list, and all patches should be sent to the list for review (see @@ -50,16 +59,47 @@ incorporating it into master, so reviews are an important way to help. We also love a good "Tested-by:" -- the more the merrier. +Testing +======= + +Individual tests can be run via + + ./pytest TEST + +For example: + + ./pytest test/int/test_git.py + ./pytest test/ext/test-ftp + +If you have the xdist module installed, then you can specify its `-n` +option to run the tests in parallel (e.g. `./pytest -nauto ...`), or +you can specify `-j` to make, which will be translated to xdist with +`-j` becoming `-nauto` and `-jN` becoming `-nN`. + +Internal tests that test bup's code directly are located in test/int, +and external tests that test bup from the outside, typically by +running the executable, are located in test/ext. + +Currently, all pytests must be located in either test/ext or test/int. +Internal test filenames must match test_*.py, and external tests must +be located in text/ext and their filenames must match test-* (see +test/ext/conftest.py for the handling of the latter). Any paths +matching those criteria will be automatically collected by pytest. + +Some aspects of the environment are automatically restored after each +test via fixtures in conftest.py, including the state of the +environment variables and the working directory; the latter is reset +to the top of the source tree. + Submitting patches ================== As mentioned, all patches should be posted to the mailing list for -review. +review, and must be "signed off" by the author before official +inclusion (see ./SIGNED-OFF-BY). You can create a "signed off" set of +patches in ./patches, ready for submission to the list, like this: -You can create a "signed off" (see ./SIGNED-OFF-BY) set of patches in -./pending, ready for submission to the list, like this: - - git format-patch -s -o pending origin/master + git format-patch -s -o patches origin/master which will include all of the patches since origin/master on your current branch. Then you can send them to the list like this: @@ -88,7 +128,7 @@ Of course, unless your machine is set up to handle outgoing mail locally, you may need to configure git to be able to send mail. See git-send-email(1) for further details. -Oh, and we do have a ./CODING-STYLE, hobgoblins and all, though don't +Oh, and we do have a ./CODINGSTYLE, hobgoblins and all, though don't let that scare you off. We're not all that fierce. @@ -102,3 +142,9 @@ ideas here aren't altogether terrible: In particular, we've been paying at least some attention to the bits regarding Acked-by:, Reported-by:, Tested-by: and Reviewed-by:. + +