Mercurial Mercurial > mercurial / annotate
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | file | latest | revisions | annotate | diff | comparison | raw | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
tests/test-check-code.t
author Jun Wu <quark@fb.com>
Wed, 30 Nov 2016 19:25:18 +0000
changeset 30556 c059286a0f9c
parent 30541 342d0cb4f446
child 30633 fd244e047ec0
permissions -rw-r--r--
tests: replace "cp -r" with "cp -R" The POSIX documentation about "cp" [1] says: .... RATIONALE .... Earlier versions of this standard included support for the -r option to copy file hierarchies. The -r option is historical practice on BSD and BSD-derived systems. This option is no longer specified by POSIX.1-2008 but may be present in some implementations. The -R option was added as a close synonym to the -r option, selected for consistency with all other options in this volume of POSIX.1-2008 that do recursive directory descent. The difference between -R and the removed -r option is in the treatment by cp of file types other than regular and directory. It was implementation-defined how the - option treated special files to allow both historical implementations and those that chose to support -r with the same abilities as -R defined by this volume of POSIX.1-2008. The original -r flag, for historic reasons, did not handle special files any differently from regular files, but always read the file and copied its contents. This had obvious problems in the presence of special file types; for example, character devices, FIFOs, and sockets. .... .... Issue 6 The -r option is marked obsolescent. .... Issue 7 .... The obsolescent -r option is removed. .... (No "Issue 8" yet) Therefore it's clear that "cp -R" is strictly better than "cp -r". The issue was discovered when running tests on OS X after 0d87b1caed92. [1]: pubs.opengroup.org/onlinepubs/9699919799/utilities/cp.html
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
27368
409a20314c64 tests: move the '-hg' postfix for all style tests
Pierre-Yves David <pierre-yves.david@fb.com>
parents:
diff changeset 
     1
 
#require test-repo
409a20314c64 tests: move the '-hg' postfix for all style tests
Pierre-Yves David <pierre-yves.david@fb.com>
parents:
diff changeset 
     2
 







29219






3c9066ed557c
tests: silence test-repo obsolete warning



timeless <timeless@mozdev.org>


parents: 
28529

diff
changeset




     3


  $ . "$TESTDIR/helpers-testrepo.sh"







27368






409a20314c64
tests: move the '-hg' postfix for all style tests



Pierre-Yves David <pierre-yves.david@fb.com>


parents: 

diff
changeset




     4


  $ check_code="$TESTDIR"/../contrib/check-code.py












409a20314c64
tests: move the '-hg' postfix for all style tests



Pierre-Yves David <pierre-yves.david@fb.com>


parents: 

diff
changeset




     5


  $ cd "$TESTDIR"/..












409a20314c64
tests: move the '-hg' postfix for all style tests



Pierre-Yves David <pierre-yves.david@fb.com>


parents: 

diff
changeset




     6














409a20314c64
tests: move the '-hg' postfix for all style tests



Pierre-Yves David <pierre-yves.david@fb.com>


parents: 

diff
changeset




     7


New errors are not allowed. Warnings are strongly discouraged.












409a20314c64
tests: move the '-hg' postfix for all style tests



Pierre-Yves David <pierre-yves.david@fb.com>


parents: 

diff
changeset




     8


(The writing "no-che?k-code" is for not skipping this file when checking.)












409a20314c64
tests: move the '-hg' postfix for all style tests



Pierre-Yves David <pierre-yves.david@fb.com>


parents: 

diff
changeset




     9









30435






b86a448a2965
zstd: vendor python-zstandard 0.5.0



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30253

diff
changeset




    10


  $ hg locate -X contrib/python-zstandard | sed 's-\\-/-g' |







27368






409a20314c64
tests: move the '-hg' postfix for all style tests



Pierre-Yves David <pierre-yves.david@fb.com>


parents: 

diff
changeset




    11


  >   xargs "$check_code" --warnings --per-file=0 || false







28432






2377c4ac4eec
fsmonitor: dependencies for new experimental extension



Martijn Pieters <mjpieters@fb.com>


parents: 
28421

diff
changeset




    12


  Skipping hgext/fsmonitor/pywatchman/__init__.py it has no-che?k-code (glob)












2377c4ac4eec
fsmonitor: dependencies for new experimental extension



Martijn Pieters <mjpieters@fb.com>


parents: 
28421

diff
changeset




    13


  Skipping hgext/fsmonitor/pywatchman/bser.c it has no-che?k-code (glob)












2377c4ac4eec
fsmonitor: dependencies for new experimental extension



Martijn Pieters <mjpieters@fb.com>


parents: 
28421

diff
changeset




    14


  Skipping hgext/fsmonitor/pywatchman/capabilities.py it has no-che?k-code (glob)







28529






045cdf47851a
fsmonitor: flag msc_stdint as no-check-code



Augie Fackler <augie@google.com>


parents: 
28432

diff
changeset




    15


  Skipping hgext/fsmonitor/pywatchman/msc_stdint.h it has no-che?k-code (glob)







28432






2377c4ac4eec
fsmonitor: dependencies for new experimental extension



Martijn Pieters <mjpieters@fb.com>


parents: 
28421

diff
changeset




    16


  Skipping hgext/fsmonitor/pywatchman/pybser.py it has no-che?k-code (glob)







27368






409a20314c64
tests: move the '-hg' postfix for all style tests



Pierre-Yves David <pierre-yves.david@fb.com>


parents: 

diff
changeset




    17


  Skipping i18n/polib.py it has no-che?k-code (glob)












409a20314c64
tests: move the '-hg' postfix for all style tests



Pierre-Yves David <pierre-yves.david@fb.com>


parents: 

diff
changeset




    18


  Skipping mercurial/httpclient/__init__.py it has no-che?k-code (glob)












409a20314c64
tests: move the '-hg' postfix for all style tests



Pierre-Yves David <pierre-yves.david@fb.com>


parents: 

diff
changeset




    19


  Skipping mercurial/httpclient/_readers.py it has no-che?k-code (glob)







30253






b032a7b676c6
statprof: vendor statprof.py



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
29442

diff
changeset




    20


  Skipping mercurial/statprof.py it has no-che?k-code (glob)







30540






d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    21














d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    22


@commands in debugcommands.py should be in alphabetical order.












d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    23














d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    24


  >>> import re












d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    25


  >>> commands = []












d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    26


  >>> with open('mercurial/debugcommands.py', 'rb') as fh:












d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    27


  ...     for line in fh:












d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    28


  ...         m = re.match("^@command\('([a-z]+)", line)












d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    29


  ...         if m:












d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    30


  ...             commands.append(m.group(1))












d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    31


  >>> scommands = list(sorted(commands))












d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    32


  >>> for i, command in enumerate(scommands):












d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    33


  ...     if command != commands[i]:












d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    34


  ...         print('commands in debugcommands.py not sorted; first differing '












d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    35


  ...               'command is %s; expected %s' % (commands[i], command))












d955cebd8d6a
tests: add test that @commands in debugcommands.py are sorted



Gregory Szorc <gregory.szorc@gmail.com>


parents: 
30435

diff
changeset




    36


  ...         break














mercurial