* Pull request for sub-tree merge into /contrib/gitstats
@ 2008-10-29 23:06 Sverre Rabbelier
2008-10-29 23:12 ` Shawn O. Pearce
` (3 more replies)
0 siblings, 4 replies; 11+ messages in thread
From: Sverre Rabbelier @ 2008-10-29 23:06 UTC (permalink / raw)
To: Junio C Hamano, Git Mailinglist
My work is available in the git repository at:
git://repo.or.cz/git-stats.git master
Please use a subtree merge to put this in contrib/gitstats.
David Symonds (2):
Insert the git_stats path to the start of os.sys.path, not 1 element in.
Fix some spelling mistakes.
Sverre Rabbelier (237):
Created a script to setup a repo to test metrics on.
Converted the setupRepo script to python to allow creating the same repo.
Added the use cases in text form.
Added a README.
Initial commit for notes branch
Changelog for 28-05-2008
Changelog for 29-05-2008
Changelog for 30-05-2008
Changelog for 31-05-2008
Created a script that aggregates author activity in one part of
the content.
Updated the script to handle merges and to treat '-' as 0.
Modified author.activityInArea to take a list of strings instead
of a file.
Added a module containing the wrappers for the commands.
Replaced funny characters in use-cases.txt with whitespace.
Created a script that handles index related activities.
Added a script to handle data mining related to commit history.
Created a 'touched' command in the wrapper module.
Added pathsTouched to the history module.
Created a commitTouched function and a prettyPrint helper function
Wrapped the use-cases.txt document at ~72 characters.
Created a list of metrics that can be used to indentify fixes.
Added three more data mining functions to the history module
Gave the setupRepo script some major TLC.
Added commitdiffEqual functionality to the history module.
Creating the repository in a temp directory, use tempfile.gettempdir().
Improved the metrics documentation.
Use 'os.path.join(a, b)' rather than 'a + os.sep + b'.
Removed try/except from setupRepo but added a manual raise
Use the same configurations as test-lib.sh in setupRepo
Copied the test-lib.sh script from git.git/t
Modified test-lib.sh to use ~/code/git as git dir.
Added a basic test that sets up our test repository.
Added a test that verifies the HEAD revision hash
There is now a testcase to check HEAD, don't do it in setupRepo
Added a basic dispatcher to stats.py.
Moved things around a lot, and made commitTouches fixup nonrelative paths
Added basic dispatching functionality for diff
Added a __init__.py file since every module needs one
Made stats.py add the directory containing git_stats to the path
Moved stats.py up one directory
We no longer require running from the root of the repository
Prefixed private methods with _ and and made parseFileDiff
accept larger diffs
Change 'PATHS' to 'COMMIT COMMIT' in the usage text
Finished the dispatching function for commit.
Moved stats.py back to the git_stats directory
Removed the $GIT_STATS_PATH related code
Removed extra newlines and replaced import with 'from .. import ..'
Moved stats.py back up again.
Moved diff.diffContains to commit.logContains
Removed the executable bit on author.py and index.py
Added dispatching to branch and finished branchList
Added dispatching to the author module
Added dispatching to the index module
Started on testing author.py and found the first bug so far
Bugfix for the author filter in author.py
Created a test case to point out an unimplemented function
Added a testcase to point out wrong output for 'stats.py branch -c'
Bugfix for the branchList output in branch.py
Finished the branch testcases, but no test for -r switch yet
The testcases require two patches to git-python to run successfully
Changelog for 01-06-2008
Changelog for 02-06-2008
Changelog for 02-06-2008 (2)
Changelog for 03-06-2008
Changelog for 04-06-2008
Changelog for 05-06-2008
Changelog for 06-06-2008
Changelog for 07-06-2008
Changelog for 08-06-2008
Changelog for 09-06-2008
Changelog for 10-06-2008
Made GitStats compatible with the latest version of git-python
Patches to git-python, since its latest changes, are no longer required
Added testcases for 'commit'
Changelog for 10-06-2008 (2)
Changelog for 11-06-2008
Bugfix for commitdiffMatches
Usability enhancement: print help msg on 'stats.py subcmd'
Merge branch 'notes'
Moved the changelog into the doc subdirectory
Changelog for 12-06-2008
Added testcases for diff and pointed out a bug and some output problems
Bugfix for diff -i, also removed debug output
Improved the dispatching in index.py
Added tests for index.py
Improved the test descriptions
Expanded the setupRepo script to generate a more elaborate tree
Made use of with_keep_cwd now that git-python runs from the top dir
Added tests for "commit -r" and pointed out a bug
Added a todo to check the file passed to "stats.py commit"
Stripped down test-lib.sh to the bits we need
Changelog for 13-06-2008
Changelog for 14-06-2008
Refactored the custom parser into it's own file
Bugfix for commit -t, now checks if the specified file is sane
Changelog for 15-06-2008
Bugfix, created a missing test for "diff -n"
Slight usability improvement
Expanded setupRepo and refactored 'addFile' into it's own function
Bugfix for the testsuite, added tests for 'index -a'
Changelog for 16-06-2008
Created a remote branch in the setupRepo script
Bugfix, added a testcase for "branch -r -c"
Changelog for 17-06-2008
Extended the description of the 'branch contains' metric
Refactored setupRepo a bit to make room for a metrics repo
Added a little more documentation on the branch metric
Extended setupRepo to create the repo as specified in metrics.txt
Updated the metrics repo to have two commits on the topic branch
Initial draft of the 'belongs to' metric
Typo, 'dilate' should be 'dilute'
Changelog for 19-06-2008
Described a more complex tree in metrics.txt
Added a bumpTime to make the commit history look a bit more sane
Changelog for 20-06-2008
Added with_exceptions=False to 'git rev-parse --verify'
Added a GIT_STATS_PRETTY_PRINT environmental variable
Added a '-v' option to stats.py
Added more tags to the generated test repo
Bail out when the testrepo couldn't be set up
Use the new GIT_STATS_PRETTY_PRINT and added tags features
When specifying kwargs they should be in quotes when not part of
a function call.
Use Repo(".").git instead of Git(".")
Updated the metrics repo to match the documented tree
Changelog for 24-06-2008
Changelog for 27-06-2008
Made the 'belongs to' metric recursive
Added tests for the 'belongs to' metric
First gather all parentage information, then run metrics
Changelog for 28-06-2008
Moved isUnique into parse.py so that other modules may use it
Added a "author -f" aggregation, and a test case.
Changelog for 29-06-2008
Minor cleanups, added documentation
Added a switch that lists only reverts of a specified commit
Changelog for 02-07-2008
Changelog for 02-07-2008
When parsing the parent listing take into account parentless commits
Print the usage help sorted and added some documentation
Use a stack-based approach instead of a recursive algorithm
Added a simple 'stats.py bug' command
Disable debug output, print minimum dilution
Optimizations to the 'belongs to' metric
Bugfix to only ignore if dilution was 'worse'
Optimion to the 'belongs to' metric
Don't filter out subsets in the belongs to metric
Optimization in the 'belongs to' metric
Changelog for 06-07-2008
Improved the output of 'branch -b' with '-d'
Added aggregation and showing all activity
Add a config parser and an example configuration
Bugfix to _parseFileDiff so that it handles diffs with mode changes only
Made use of the new config parser in bug.py
Changelog for 10-07-2008
When the specified path was not found, return an empty dict
Don't die when an option is not specified at all
Moved the commit information into a seperate class
Restructured the commit and diff module so that diff depends on commit
Refactored bug.py to allow for a trivial aggregation function
Refactored stats.py to use a main function
Allow for checking deleted files in commitsThatTouched
Check for empty diffs
Add a diff memory to diff.py and use it in bug.py
Don't look before we leap in getting the commit diff
Add a 'commits that touch' memory to commit.py
Two more memories were added and some refactoring
Print authors sorted in 'author -e'
Added an option to disable line numbering in diff parsing
Added a proof of concept matcher
Changelog for 12-07-2008
Add an option to ignore parent in the 'belongs to' metric
Added an option to limit the amount of commits checked
Allow specifying True, False, or None in the config file
Allow specifying True, False, or None on the command line
General cleanups in bug.py
Bugfix for branchContains Fix the branch
Make use of the ignore_parents option to belongsTo
Changelog for 14-07-2008
Provide a default empty dict to pretty names
Refactored bug.py to use a Memory and Options class
Make config.py more versatile
Teach config.py to read multiple verses
Introduce a bugfixRating to bug.py
Changelog for 15-07-2008
Refactoring, cleanups and documentation
Changelog for 16-07-2008
Updated the README to include specific installation instructions
Added a 'net loc' to 'author -a'
Updated the tests to match the new output of 'author -a'
Official 0.1.4 release of git-python
Removed the redundant setupRepo.sh script
Added an option to print the path for setupRepo.py
Added a way to specify the path to use instead of the default
Don't assume the temp path is /tmp
Make use of the 'sorted' built-in
Changelog for 17-07-2008
Added a unit-test framework for GitStats
Use 'key in dict' instead of 'dict.has_key(key)'
Renamed fileDiff to FileDiff
Bugfix for diff.py, don't die on empty diffs
Make use of getattr instead of just trying
Use format specifiers instead of appending to a string
Replace backslashes with parens to do line continuation
Don't asume setupRepo.py is executable
Remove unneeded executionable bit on setupRepo.py
Replace check_file with checkFile
Use dashed_form for variable names instead of camelCase
Changelog for 23-07-2008
Made config.read take a bunch of strings instead of a path
Improved testing.py output and removed manual parsing of arguments
Added a module to dispatch unit-testing commands
Hooked up the test suite in stats.py
Do not require lines to end with a '\n' in the config parser
Refactor config.py to make it more testable
Wrote unittests for the config module and hooked them up
Changelog for 24-07-2008
Don't require stats.py to be in $PATH when running the regression tests
Renamed README to INSTALL
Expanded the INSTALL file to include usage information
Added a README describing GitStats purpose
Convert matcher.py to use optparse
Add a description about the branch module
Add a description about the author module
Add a description about the commit module
Add a general note about the 'stats.py'
Add a description about the diff module
Add a description about the index module
Add a description about the matcher module
Add a description about the tests module
Renamed the GitStats to include a gitstats- prefix
Add a description about the bug module
Renamed Memory->GitCache and Options->OptionList
Expanded the documentation of the bug module
Ran ispell on all non gitstats-* files in doc/
Ran ispell on all gitstats-* files in doc/
Added some example values to gitstats-bug.txt
Bugfix for the config parsing mechanism
INSTALL | 33 ++
README | 59 ++++
doc/changelog.txt | 335 +++++++++++++++++++++
doc/gitstats-author.txt | 30 ++
doc/gitstats-branch.txt | 35 +++
doc/gitstats-bug.txt | 109 +++++++
doc/gitstats-commit.txt | 44 +++
doc/gitstats-diff.txt | 24 ++
doc/gitstats-index.txt | 19 ++
doc/gitstats-matcher.txt | 19 ++
doc/gitstats-stats.txt | 36 +++
doc/gitstats-tests.txt | 8 +
doc/metrics.txt | 83 ++++++
doc/use-cases.txt | 160 ++++++++++
src/git_stats/author.py | 315 ++++++++++++++++++++
src/git_stats/branch.py | 420 ++++++++++++++++++++++++++
src/git_stats/bug.py | 306 +++++++++++++++++++
src/git_stats/commit.py | 343 ++++++++++++++++++++++
src/git_stats/config | 5 +
src/git_stats/config.py | 219 ++++++++++++++
src/git_stats/config_tests.py | 245 +++++++++++++++
src/git_stats/diff.py | 515 ++++++++++++++++++++++++++++++++
src/git_stats/index.py | 102 +++++++
src/git_stats/matcher.py | 130 ++++++++
src/git_stats/parse.py | 109 +++++++
src/git_stats/testing.py | 212 +++++++++++++
src/git_stats/tests.py | 96 ++++++
src/scripts/setupRepo.py | 652 +++++++++++++++++++++++++++++++++++++++++
src/scripts/setupRepo.sh | 31 --
src/stats.py | 112 +++++++
src/t/t8100-stats.sh | 333 +++++++++++++++++++++
src/t/t8101-metrics.sh | 163 ++++++++++
src/t/test-lib.sh | 372 +++++++++++++++++++++++
33 files changed, 5643 insertions(+), 31 deletions(-)
create mode 100644 INSTALL
create mode 100644 README
create mode 100644 doc/changelog.txt
create mode 100644 doc/gitstats-author.txt
create mode 100644 doc/gitstats-branch.txt
create mode 100644 doc/gitstats-bug.txt
create mode 100644 doc/gitstats-commit.txt
create mode 100644 doc/gitstats-diff.txt
create mode 100644 doc/gitstats-index.txt
create mode 100644 doc/gitstats-matcher.txt
create mode 100644 doc/gitstats-stats.txt
create mode 100644 doc/gitstats-tests.txt
create mode 100644 doc/metrics.txt
create mode 100644 doc/use-cases.txt
create mode 100644 src/git_stats/__init__.py
create mode 100644 src/git_stats/author.py
create mode 100644 src/git_stats/branch.py
create mode 100644 src/git_stats/bug.py
create mode 100644 src/git_stats/commit.py
create mode 100644 src/git_stats/config
create mode 100644 src/git_stats/config.py
create mode 100644 src/git_stats/config_tests.py
create mode 100644 src/git_stats/diff.py
create mode 100644 src/git_stats/index.py
create mode 100644 src/git_stats/matcher.py
create mode 100644 src/git_stats/parse.py
create mode 100644 src/git_stats/testing.py
create mode 100755 src/git_stats/tests.py
create mode 100644 src/scripts/setupRepo.py
delete mode 100755 src/scripts/setupRepo.sh
create mode 100755 src/stats.py
create mode 100755 src/t/t8100-stats.sh
create mode 100755 src/t/t8101-metrics.sh
create mode 100644 src/t/test-lib.sh
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Pull request for sub-tree merge into /contrib/gitstats
2008-10-29 23:06 Pull request for sub-tree merge into /contrib/gitstats Sverre Rabbelier
@ 2008-10-29 23:12 ` Shawn O. Pearce
2008-10-29 23:39 ` Sverre Rabbelier
2008-10-29 23:31 ` Nicolas Pitre
` (2 subsequent siblings)
3 siblings, 1 reply; 11+ messages in thread
From: Shawn O. Pearce @ 2008-10-29 23:12 UTC (permalink / raw)
To: sverre; +Cc: Junio C Hamano, Git Mailinglist
Sverre Rabbelier <alturin@gmail.com> wrote:
> Please use a subtree merge to put this in contrib/gitstats.
Yea, about that...
> Sverre Rabbelier (237):
> Created a script to setup a repo to test metrics on.
> Converted the setupRepo script to python to allow creating the same repo.
> Added the use cases in text form.
> Added a README.
> Initial commit for notes branch
> Changelog for 28-05-2008
> Changelog for 29-05-2008
> Changelog for 30-05-2008
> Changelog for 31-05-2008
...
How is this going to look in the "What's in git.git" email?
We don't use ChangeLog files in git.git and we don't have
notes branches, and we already have a README.
Most stuff in contrib/ has its commit messages with a prefix string
to make it more clear when looking at the shortlog what is being
impacted. Maybe this should be re-written with filter-branch to
include a prefix before it merges.
--
Shawn.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Pull request for sub-tree merge into /contrib/gitstats
2008-10-29 23:06 Pull request for sub-tree merge into /contrib/gitstats Sverre Rabbelier
2008-10-29 23:12 ` Shawn O. Pearce
@ 2008-10-29 23:31 ` Nicolas Pitre
2008-10-29 23:38 ` Sverre Rabbelier
2008-10-30 0:38 ` Sverre Rabbelier
2008-10-30 19:24 ` Junio C Hamano
3 siblings, 1 reply; 11+ messages in thread
From: Nicolas Pitre @ 2008-10-29 23:31 UTC (permalink / raw)
To: sverre; +Cc: Junio C Hamano, Git Mailinglist
On Thu, 30 Oct 2008, Sverre Rabbelier wrote:
> My work is available in the git repository at:
>
> git://repo.or.cz/git-stats.git master
>
> Please use a subtree merge to put this in contrib/gitstats.
[...]
Why do you have commits such as:
> Changelog for 01-06-2008
> Changelog for 02-06-2008
> Changelog for 02-06-2008 (2)
> Changelog for 03-06-2008
> Changelog for 04-06-2008
> Changelog for 05-06-2008
> Changelog for 06-06-2008
> Changelog for 07-06-2008
> Changelog for 08-06-2008
> Changelog for 09-06-2008
> Changelog for 10-06-2008
?
If those are not significant enough to have a proper description, then
I'd suggest you use 'git rebase -i' and its "squash" command to fold
them into the appropriate commit.
Nicolas
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Pull request for sub-tree merge into /contrib/gitstats
2008-10-29 23:31 ` Nicolas Pitre
@ 2008-10-29 23:38 ` Sverre Rabbelier
0 siblings, 0 replies; 11+ messages in thread
From: Sverre Rabbelier @ 2008-10-29 23:38 UTC (permalink / raw)
To: Nicolas Pitre; +Cc: Junio C Hamano, Git Mailinglist
On Thu, Oct 30, 2008 at 00:31, Nicolas Pitre <nico@cam.org> wrote:
> If those are not significant enough to have a proper description, then
> I'd suggest you use 'git rebase -i' and its "squash" command to fold
> them into the appropriate commit.
They are there because the work was done incrementally, having them as
seperate commits shows when the changelog was modified. Having a
description for a change to the changelog is pretty senseless, since
it would be the same text as what is in the diff.
That said, sure, I can squash those commits no problem.
--
Cheers,
Sverre Rabbelier
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Pull request for sub-tree merge into /contrib/gitstats
2008-10-29 23:12 ` Shawn O. Pearce
@ 2008-10-29 23:39 ` Sverre Rabbelier
0 siblings, 0 replies; 11+ messages in thread
From: Sverre Rabbelier @ 2008-10-29 23:39 UTC (permalink / raw)
To: Shawn O. Pearce; +Cc: Junio C Hamano, Git Mailinglist
On Thu, Oct 30, 2008 at 00:12, Shawn O. Pearce <spearce@spearce.org> wrote:
> Most stuff in contrib/ has its commit messages with a prefix string
> to make it more clear when looking at the shortlog what is being
> impacted. Maybe this should be re-written with filter-branch to
> include a prefix before it merges.
Sure, I'm fine with rewriting all commit messages to have a "gitstats:" prefix.
--
Cheers,
Sverre Rabbelier
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Pull request for sub-tree merge into /contrib/gitstats
2008-10-29 23:06 Pull request for sub-tree merge into /contrib/gitstats Sverre Rabbelier
2008-10-29 23:12 ` Shawn O. Pearce
2008-10-29 23:31 ` Nicolas Pitre
@ 2008-10-30 0:38 ` Sverre Rabbelier
2008-10-30 19:24 ` Junio C Hamano
3 siblings, 0 replies; 11+ messages in thread
From: Sverre Rabbelier @ 2008-10-30 0:38 UTC (permalink / raw)
To: Junio C Hamano, Git Mailinglist
On Thu, Oct 30, 2008 at 00:06, Sverre Rabbelier <alturin@gmail.com> wrote:
> My work is available in the git repository at:
Please use instead:
> git://repo.or.cz/git-stats.git for-junio
Which has the "gitstats:" prefix to all commit messages, and does not
have the changelog file.
--
Cheers,
Sverre Rabbelier
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Pull request for sub-tree merge into /contrib/gitstats
2008-10-29 23:06 Pull request for sub-tree merge into /contrib/gitstats Sverre Rabbelier
` (2 preceding siblings ...)
2008-10-30 0:38 ` Sverre Rabbelier
@ 2008-10-30 19:24 ` Junio C Hamano
2008-11-02 19:24 ` Sverre Rabbelier
3 siblings, 1 reply; 11+ messages in thread
From: Junio C Hamano @ 2008-10-30 19:24 UTC (permalink / raw)
To: sverre; +Cc: Git Mailinglist
I have a mixed feeling about this. From a longer-term perspective, do you
really want this to be a part of git.git repository?
I do not mind having notes to endorse and advocate "stats" as one of the
"Third party packages that may make your git life more pleasuable", just
like tig, stgit, guilt and topgit, but I cannot convince myself that
merging it as a subtree is the right thing to do at this point.
The "stats" tool, at least at the conceptual level, shares one important
property with tools like gitk and gitweb: it could be useful to people
whose sources are not in git repositories but in say Hg or Bzr, with some
effort. The code may need refactoring to make it easier to plug in
different backends and writing actual backends (aka "porting"), but that
is something you can expect people with different backends to help you
with.
However, it would be awkward for the contrib/ area in git.git to carry a
lot of code that are only needed to produce stat data from non-git
repositories, once such a porting effort begins.
It's perfectly fine if you are not interested in any of the other
backends, and tell the people that they are welcome to fork it never to
merge back. But if this were my brainchild, I'd imagine I'd be wishing to
be able to buy back the improvements to the "core stats" parts that are
done by people with other backends. I would imagine binding the current
code as part of git.git would make such improvements harder to manage,
both for you (who wants to buy back the changes made by others on
different backends) and for others on different backends (who want to
merge the changes made by you to their forks).
Perhaps pointing at your tree as a submodule would be the right thing to
do; then git.git proper will be just one of the users of "stats" tool.
How about making that as a mid-to-longer term goal? When we eject git-gui
and gitk from git.git and make them a submodule (wasn't that supposed to
happen in 1.8 or 2.0 timeframe?), we may also add "stats" as a submodule?
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Pull request for sub-tree merge into /contrib/gitstats
2008-10-30 19:24 ` Junio C Hamano
@ 2008-11-02 19:24 ` Sverre Rabbelier
2008-11-03 6:33 ` Johannes Schindelin
0 siblings, 1 reply; 11+ messages in thread
From: Sverre Rabbelier @ 2008-11-02 19:24 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Git Mailinglist
[Sorry for the late reply, have been travelling, sleeping, and
catching up with family in the past few days]
On Thu, Oct 30, 2008 at 20:24, Junio C Hamano <gitster@pobox.com> wrote:
> I have a mixed feeling about this. From a longer-term perspective, do you
> really want this to be a part of git.git repository?
My main reason for wanting to have it in git.git is getting additional
exposure, being in /contrib seems like a good way to do that.
> I do not mind having notes to endorse and advocate "stats" as one of the
> "Third party packages that may make your git life more pleasuable", just
> like tig, stgit, guilt and topgit, but I cannot convince myself that
> merging it as a subtree is the right thing to do at this point.
Heh, blame Johannes for that one; the main reason for not doing
something like this earlier was my uncertaincy as to -what- to do.
Dscho suggested to request-pull a subtree merge, which is what I did.
> The "stats" tool, at least at the conceptual level, shares one important
> property with tools like gitk and gitweb: it could be useful to people
> whose sources are not in git repositories but in say Hg or Bzr, with some
> effort. The code may need refactoring to make it easier to plug in
> different backends and writing actual backends (aka "porting"), but that
> is something you can expect people with different backends to help you
> with.
This is true, it uses a python wrapper around git, but with some work
it could be make to use a more abstract wrapper instead, that allows
the use of different backends.
> However, it would be awkward for the contrib/ area in git.git to carry a
> lot of code that are only needed to produce stat data from non-git
> repositories, once such a porting effort begins.
I reckon it would not be a lot of code, but I agree, that would be awkward.
> It's perfectly fine if you are not interested in any of the other
> backends, and tell the people that they are welcome to fork it never to
> merge back. But if this were my brainchild, I'd imagine I'd be wishing to
> be able to buy back the improvements to the "core stats" parts that are
> done by people with other backends. I would imagine binding the current
> code as part of git.git would make such improvements harder to manage,
> both for you (who wants to buy back the changes made by others on
> different backends) and for others on different backends (who want to
> merge the changes made by you to their forks).
This is true, if there is indeed interest from other backends to use
this kind of functionality, I would welcome the patches. In such a
case being in git.git/contrib might not be a good thing.
> Perhaps pointing at your tree as a submodule would be the right thing to
> do; then git.git proper will be just one of the users of "stats" tool.
Would a subdir in git.git for such submodules be a good idea? That way
we don't have to worry about a conflict between (for example) git-gui
as a subdir, and git-gui as a submodule.
> How about making that as a mid-to-longer term goal? When we eject git-gui
> and gitk from git.git and make them a submodule (wasn't that supposed to
> happen in 1.8 or 2.0 timeframe?), we may also add "stats" as a submodule?
I didn't know there was a timeframe for this, I thought your
suggestion tree to eject-and-make-into-submodule was somewhat ignored;
if there are indeed plans for this, I would be ok with having
git-stats as a submodule instead.
--
Cheers,
Sverre Rabbelier
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Pull request for sub-tree merge into /contrib/gitstats
2008-11-02 19:24 ` Sverre Rabbelier
@ 2008-11-03 6:33 ` Johannes Schindelin
2008-11-03 7:07 ` David Symonds
0 siblings, 1 reply; 11+ messages in thread
From: Johannes Schindelin @ 2008-11-03 6:33 UTC (permalink / raw)
To: Junio C Hamano; +Cc: sverre, git, dsymonds
Hi,
On Sun, 2 Nov 2008, Sverre Rabbelier wrote:
> On Thu, Oct 30, 2008 at 20:24, Junio C Hamano <gitster@pobox.com> wrote:
> > I have a mixed feeling about this. From a longer-term perspective, do
> > you really want this to be a part of git.git repository?
>
> My main reason for wanting to have it in git.git is getting additional
> exposure, being in /contrib seems like a good way to do that.
>
> > I do not mind having notes to endorse and advocate "stats" as one of
> > the "Third party packages that may make your git life more
> > pleasuable", just like tig, stgit, guilt and topgit, but I cannot
> > convince myself that merging it as a subtree is the right thing to do
> > at this point.
>
> Heh, blame Johannes for that one; the main reason for not doing
> something like this earlier was my uncertaincy as to -what- to do. Dscho
> suggested to request-pull a subtree merge, which is what I did.
Actually, I suggested that the end result be submitted as a single patch
against contrib/, but dsymonds objected that that would lose all history,
and I concurred that it would be nice to have all changes with the design
decisions recorded as commit messages, too.
Actually, in the meantime, I am of two minds. On the one hand, I would
just like to have the scripts in contrib/ (as a result of one commit), and
on the other hand, I would not like to lose history of gitstats.
So I roll the ball back into your (Junio's) half, albeit gently.
Ciao,
Dscho
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Pull request for sub-tree merge into /contrib/gitstats
2008-11-03 6:33 ` Johannes Schindelin
@ 2008-11-03 7:07 ` David Symonds
2008-11-03 8:40 ` Sverre Rabbelier
0 siblings, 1 reply; 11+ messages in thread
From: David Symonds @ 2008-11-03 7:07 UTC (permalink / raw)
To: Johannes Schindelin; +Cc: Junio C Hamano, sverre, git
On Sun, Nov 2, 2008 at 11:33 PM, Johannes Schindelin
<Johannes.Schindelin@gmx.de> wrote:
> Actually, I suggested that the end result be submitted as a single patch
> against contrib/, but dsymonds objected that that would lose all history,
> and I concurred that it would be nice to have all changes with the design
> decisions recorded as commit messages, too.
I chatted to Sverre separately, and I think we concluded that we could
probably just do a single patch (no history, no changelogs, etc.), and
just point to the complete git-stats.git repo for folk who want the
detail.
Dave.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Pull request for sub-tree merge into /contrib/gitstats
2008-11-03 7:07 ` David Symonds
@ 2008-11-03 8:40 ` Sverre Rabbelier
0 siblings, 0 replies; 11+ messages in thread
From: Sverre Rabbelier @ 2008-11-03 8:40 UTC (permalink / raw)
To: David Symonds; +Cc: Johannes Schindelin, Junio C Hamano, git
On Mon, Nov 3, 2008 at 08:07, David Symonds <dsymonds@gmail.com> wrote:
> I chatted to Sverre separately, and I think we concluded that we could
> probably just do a single patch (no history, no changelogs, etc.), and
> just point to the complete git-stats.git repo for folk who want the
> detail.
Either is fine with me, anything to get more exposure is fine with me :).
--
Cheers,
Sverre Rabbelier
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2008-11-03 8:42 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-10-29 23:06 Pull request for sub-tree merge into /contrib/gitstats Sverre Rabbelier
2008-10-29 23:12 ` Shawn O. Pearce
2008-10-29 23:39 ` Sverre Rabbelier
2008-10-29 23:31 ` Nicolas Pitre
2008-10-29 23:38 ` Sverre Rabbelier
2008-10-30 0:38 ` Sverre Rabbelier
2008-10-30 19:24 ` Junio C Hamano
2008-11-02 19:24 ` Sverre Rabbelier
2008-11-03 6:33 ` Johannes Schindelin
2008-11-03 7:07 ` David Symonds
2008-11-03 8:40 ` Sverre Rabbelier
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).