[PATCH 1 of 4] keyword: kwfiles --unknown instead of --untracked

Christian Ebert blacktrash at gmx.net
Wed Sep 30 06:18:51 CDT 2009


* Christian Ebert on Monday, September 28, 2009 at 09:59:57 +0200
> * Greg Ward on Sunday, September 27, 2009 at 22:06:39 -0400
>> On Sun, Sep 27, 2009 at 6:21 PM, Christian Ebert <blacktrash at gmx.net> wrote:
>>> # HG changeset patch
>>> # User Christian Ebert <blacktrash at gmx.net>
>>> # Date 1254071715 -7200
>>> # Node ID a0fb07222a11ad7c1f10b1ffae8556096a66c14c
>>> # Parent  7d6ac5d7917c84d5e8d1016b6bfe0cf726f80eac
>>> keyword: kwfiles --unknown instead of --untracked
>> 
>> Says in http://mercurial.selenic.com/wiki/CompatibilityRules#Extensions that:
>> 
>> "Extensions that are shipped in hgext/ follow the same compatibility
>> rules as core code"
> 
> Right. Although:
> 
> "Bear in mind that extensions are not "canonical", and their
> behavior may change or break core Mercurial functionality"
> 
> Which one is it then? ;-)
> 
> It's not a Config option:
> "We may deprecate some options, giving their replacements
> different names" 

Any opinions?

>> which means we can't just remove existing options that were present in 1.3.
>> 
>>> @@ -547,7 +547,8 @@
>>>         (files,
>>>          [('a', 'all', None, _('show keyword status flags of all files')),
>>>           ('i', 'ignore', None, _('show files excluded from expansion')),
>>> -          ('u', 'untracked', None, _('additionally show untracked files')),
>>> +          ('u', 'unknown', None,
>>> +           _('additionally show unknown (not tracked) files')),
>> 
>> ...which is what this looks like.  I think you need to add a
>> backwards-compatible --untracked option.
> 
> In the next patch the behaviour is changed too. I try to
> summarize again what the series changes:
> 
> old option names   old behaviour                      new option names    new behaviour 
> -u/--untracked     additionally show unknown files    -u/unknown          unknown files only (like status)
> -a/--all           show [keyword] files w/o unknown   -A/--all            all files, except ignored
> 
> I mainly thought of the command as interactive, not for parsing;
> but of course it /could/ be parsed.

I haven't tried yet how to deprecate a short/long option only.
However I'd like to know first whether the 2 rather contradictory
rules in

http://mercurial.selenic.com/wiki/CompatibilityRules#Extensions

should be interpreted that way.

THX

c
-- 
__   _    _         _  __  _               _    
\ \ | |__| |__ _ __| |_\ \| |_ _ _ __ _ __| |_  
 \ \| '_ \ / _` / _| / /\ \  _| '_/ _` (_-< ' \   http://www.blacktrash.org/
  \_\_.__/_\__,_\__|_\_\ \_\__|_| \__,_/__/_||_|  Jabber-ID: blacktrash at gmx.net


More information about the Mercurial-devel mailing list