Re: [RFC/PATCH 0/4] Remove deprecated commands

2013-11-11 Thread John Keeping
On Mon, Nov 11, 2013 at 11:13:45AM -0800, Jonathan Nieder wrote:
> John Keeping wrote:
> > On Mon, Nov 11, 2013 at 10:25:51AM -0800, Junio C Hamano wrote:
> >> John Keeping  writes:
> 
> >>> "git repo-config", "git tar-tree", "git lost-found" and "git
> >>> peek-remote" have all been deprecated since at least Git 1.5.4.
> [...]
> >> Probably good material to discuss during the next cycle.
> [...]
> > I was assuming these would be queued as a "held until 2.0" branch, but
> 
> Please no. :)  We already have a nice set of features for 2.0 and I
> hope people have as few excuses not to upgrade as possible.  Anything
> that actually needs the same kind of treatment that is introduced now
> should wait for 3.0.
> 
> Removing repo-config, tar-tree, and peek-remote sounds fine to me
> (though I haven't thought much about it either way) and I agree that
> it wouldn't need to wait for an x.0 release.

For "git repo-config", the 1.5.4 release notes say that it will "be
removed in the next feature release".  I'm not sure what a "feature
release" is, but if 1.6.0 wasn't one, then I think 2.0 will be the next
one.

Although, I now see that howto/maintain-git.txt says a feature release
is numbered vX.Y.Z, so perhaps it should have been removed long before
now...
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC/PATCH 0/4] Remove deprecated commands

2013-11-11 Thread Jonathan Nieder
John Keeping wrote:
> On Mon, Nov 11, 2013 at 10:25:51AM -0800, Junio C Hamano wrote:
>> John Keeping  writes:

>>> "git repo-config", "git tar-tree", "git lost-found" and "git
>>> peek-remote" have all been deprecated since at least Git 1.5.4.
[...]
>> Probably good material to discuss during the next cycle.
[...]
> I was assuming these would be queued as a "held until 2.0" branch, but

Please no. :)  We already have a nice set of features for 2.0 and I
hope people have as few excuses not to upgrade as possible.  Anything
that actually needs the same kind of treatment that is introduced now
should wait for 3.0.

Removing repo-config, tar-tree, and peek-remote sounds fine to me
(though I haven't thought much about it either way) and I agree that
it wouldn't need to wait for an x.0 release.

Thanks,
Jonathan
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC/PATCH 0/4] Remove deprecated commands

2013-11-11 Thread John Keeping
On Mon, Nov 11, 2013 at 10:25:51AM -0800, Junio C Hamano wrote:
> John Keeping  writes:
> 
> > "git repo-config", "git tar-tree", "git lost-found" and "git
> > peek-remote" have all been deprecated since at least Git 1.5.4.
> >
> > With Git 2.0 approaching, I think that would be a good point to remove
> > then completely, which is what this series does.
> 
> Probably good material to discuss during the next cycle.
>
> As a totally unscientific sanity check, I asked Google to show these
> since Nov 13, 2011:
> 
> [git repo-config] vs [git config]: 136 vs 232
> 
> [git peek-remote] vs [git ls-remote]: 62 vs 133
> 
> [git tar-tree] vs [git archive]: 73 vs 189
> 
> [git lost-found] vs [git fsck --lost-found]: 96 vs 83
> 
> So I think 1, 2, and 4 are OK to ship in whatever version that comes
> after the upcoming 1.8.5, but we might have to hold onto lost-found
> a bit longer.  The command does show a deprecation warning, so there
> is nothing to change at this point.

I was assuming these would be queued as a "held until 2.0" branch, but
if you think some can go earlier I can re-send this once 1.8.5 is out of
the way.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC/PATCH 0/4] Remove deprecated commands

2013-11-11 Thread Junio C Hamano
John Keeping  writes:

> "git repo-config", "git tar-tree", "git lost-found" and "git
> peek-remote" have all been deprecated since at least Git 1.5.4.
>
> With Git 2.0 approaching, I think that would be a good point to remove
> then completely, which is what this series does.

Probably good material to discuss during the next cycle.

As a totally unscientific sanity check, I asked Google to show these
since Nov 13, 2011:

[git repo-config] vs [git config]: 136 vs 232

[git peek-remote] vs [git ls-remote]: 62 vs 133

[git tar-tree] vs [git archive]: 73 vs 189

[git lost-found] vs [git fsck --lost-found]: 96 vs 83

So I think 1, 2, and 4 are OK to ship in whatever version that comes
after the upcoming 1.8.5, but we might have to hold onto lost-found
a bit longer.  The command does show a deprecation warning, so there
is nothing to change at this point.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[RFC/PATCH 0/4] Remove deprecated commands

2013-11-10 Thread John Keeping
"git repo-config", "git tar-tree", "git lost-found" and "git
peek-remote" have all been deprecated since at least Git 1.5.4.

With Git 2.0 approaching, I think that would be a good point to remove
then completely, which is what this series does.

John Keeping (4):
  repo-config: remove deprecated alias for "git config"
  tar-tree: remove deprecated command
  lost-found: remove deprecated command
  peek-remote: remove deprecated alias of ls-remote

 .gitignore |  4 --
 Documentation/git-lost-found.txt   | 74 --
 Documentation/git-peek-remote.txt  | 43 --
 Documentation/git-repo-config.txt  | 23 --
 Documentation/git-tar-tree.txt | 82 --
 Makefile   |  3 --
 builtin.h  |  1 -
 builtin/config.c   |  6 ---
 builtin/tar-tree.c | 62 -
 command-list.txt   |  4 --
 contrib/completion/git-completion.bash |  4 --
 contrib/examples/git-whatchanged.sh|  4 +-
 git-lost-found.sh  | 33 --
 git.c  |  3 --
 t/t4116-apply-reverse.sh   |  4 +-
 t/t5000-tar-tree.sh| 16 ++-
 t/t5001-archive-attr.sh| 10 -
 17 files changed, 7 insertions(+), 369 deletions(-)
 delete mode 100644 Documentation/git-lost-found.txt
 delete mode 100644 Documentation/git-peek-remote.txt
 delete mode 100644 Documentation/git-repo-config.txt
 delete mode 100644 Documentation/git-tar-tree.txt
 delete mode 100755 git-lost-found.sh

-- 
1.8.5.rc0.170.g772b8ec

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html