Merge lp://qastaging/~jelmer/bzr-fastimport/remove-exporters into lp://qastaging/bzr-fastimport
Proposed by
Jelmer Vernooij
| Status: | Merged |
|---|---|
| Approved by: | Martin Packman |
| Approved revision: | 352 |
| Merged at revision: | 352 |
| Proposed branch: | lp://qastaging/~jelmer/bzr-fastimport/remove-exporters |
| Merge into: | lp://qastaging/bzr-fastimport |
| Diff against target: |
5423 lines (+20/-4960) 68 files modified
NEWS (+8/-0) README.txt (+0/-4) __init__.py (+1/-29) cmds.py (+11/-234) exporters/Makefile (+0/-15) exporters/__init__.py (+0/-326) exporters/darcs/.gitignore (+0/-3) exporters/darcs/Makefile (+0/-55) exporters/darcs/NEWS (+0/-26) exporters/darcs/README (+0/-187) exporters/darcs/TODO (+0/-8) exporters/darcs/asciidoc.conf (+0/-21) exporters/darcs/d2x (+0/-112) exporters/darcs/d2x.txt (+0/-27) exporters/darcs/darcs-fast-export (+0/-380) exporters/darcs/darcs-fast-export.txt (+0/-68) exporters/darcs/darcs-fast-import (+0/-375) exporters/darcs/darcs-fast-import.txt (+0/-57) exporters/darcs/git-darcs (+0/-279) exporters/darcs/git-darcs.txt (+0/-92) exporters/darcs/t/Makefile (+0/-9) exporters/darcs/t/bench-results/Makefile (+0/-5) exporters/darcs/t/bench-results/bench-results.gnu (+0/-6) exporters/darcs/t/bench-results/bench-results.py (+0/-23) exporters/darcs/t/bench-tailor.sh (+0/-59) exporters/darcs/t/bench.sh (+0/-38) exporters/darcs/t/lib-httpd.sh (+0/-67) exporters/darcs/t/lib.sh (+0/-337) exporters/darcs/t/test-bzr.sh (+0/-16) exporters/darcs/t/test-git-d2x.sh (+0/-19) exporters/darcs/t/test-git-incremental.sh (+0/-24) exporters/darcs/t/test-git-progress.sh (+0/-18) exporters/darcs/t/test-git.sh (+0/-18) exporters/darcs/t/test-hg-d2x.sh (+0/-12) exporters/darcs/t/test-hg.sh (+0/-16) exporters/darcs/t/test2-bzr-d2x.sh (+0/-19) exporters/darcs/t/test2-bzr-incremental.sh (+0/-21) exporters/darcs/t/test2-git-funny-tagname.sh (+0/-25) exporters/darcs/t/test2-git-http.sh (+0/-22) exporters/darcs/t/test2-git-incremental-specworkdir.sh (+0/-22) exporters/darcs/t/test2-git-incremental.sh (+0/-21) exporters/darcs/t/test2-git.sh (+0/-18) exporters/darcs/t/testimport-bzr-x2d.sh (+0/-15) exporters/darcs/t/testimport-bzr.sh (+0/-15) exporters/darcs/t/testimport-copy.sh (+0/-26) exporters/darcs/t/testimport-darcs.sh (+0/-17) exporters/darcs/t/testimport-deleteall.sh (+0/-31) exporters/darcs/t/testimport-git-incremental.sh (+0/-16) exporters/darcs/t/testimport-git-twoway-gd.sh (+0/-34) exporters/darcs/t/testimport-git-twoway.sh (+0/-30) exporters/darcs/t/testimport-git-x2d.sh (+0/-15) exporters/darcs/t/testimport-git.sh (+0/-15) exporters/darcs/t/testimport-gitsymlink.sh (+0/-45) exporters/darcs/t/testimport-hg-x2d.sh (+0/-15) exporters/darcs/t/testimport-hg.sh (+0/-15) exporters/darcs/t/testimport-rename.sh (+0/-25) exporters/darcs/x2d (+0/-125) exporters/darcs/x2d.txt (+0/-28) exporters/hg-fast-export.README (+0/-54) exporters/hg-fast-export.py (+0/-442) exporters/hg-fast-export.sh (+0/-100) exporters/hg2git.py (+0/-112) exporters/svn-archive.c (+0/-240) exporters/svn-fast-export.README (+0/-12) exporters/svn-fast-export.c (+0/-187) exporters/svn-fast-export.py (+0/-225) info.py (+0/-7) setup.py (+0/-1) |
| To merge this branch: | bzr merge lp://qastaging/~jelmer/bzr-fastimport/remove-exporters |
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Martin Packman (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
Remove the custom exporters from bzr-fastimport.
This is code that is imported from elsewhere which we're not really
maintaining. There are bzr command wrappers for them, but these wrappers only
expose a limited amount of the functionality of the actual exporters and don't
really add much value (other than a consistent, but fairly unuseful interface).
I'd really rather just point people at the actual exporters rather than
having to maintain a copy of them inside of bzr-fastimport.
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Reasoning for removing these makes sense to me. One thing I'd like to see is web links to exporters in the documentation here. Links do bitrot like code but generally at a slower rate and it's easier to fix them. Giving people a poke in the right direction to find the exporter they need rather than just assuming "widely available" means everyone will know how to google for the right thing would help.