X-Git-Url: https://arthur.barton.de/cgi-bin/gitweb.cgi?p=bup.git;a=blobdiff_plain;f=HACKING;h=da846f9a611c4d29eae9b43c901ac92268d9cbd2;hp=88f5b416837be2a73766285cc0496807272b3ff9;hb=HEAD;hpb=bfa418544ef05efd8b8cca9150d23922f2318205 diff --git a/HACKING b/HACKING index 88f5b41..0c8c719 100644 --- a/HACKING +++ b/HACKING @@ -7,10 +7,6 @@ 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. @@ -23,27 +19,44 @@ 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.33 release, we're working on 0.34, 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.) + - Migrating hashsplitting to C. + + - Automatically splitting trees to avoid having to save large tree + objects for large directories even if only a few files have + changed or been added (e.g. maildirs). + + - Moving all of the compoents of the index to sqlite. Right now the + main index is an mmapped file, and the hard link and metadata + databases are pickled. As a result the index isn't transactional + and suffers from bugs caused by "skew" across the components. + + - Better VFS performance for large repositories (i.e. fuse, ls, + web...). + + - Better VFS caching. -We'd like to try to release (0.25) soon, so we're limiting the scope -of prospective changes -- definitely in-scope: + - Index improvements. - - fixes to the new metadata support - - fixes for regressions (portability included) - - fixes for "serious" bugs - - "simple" fixes - - documentation improvements + - Incremental indexing via inotify. + + - Smarter (and quieter) handling of cross-filesystem metadata. + + - Encryption. + + - Support for alternate remote storage APIs. + +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 @@ -55,16 +68,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: @@ -93,7 +137,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. @@ -107,3 +151,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:. + +