On 7/24/2014 3:41 AM, Junio C Hamano wrote:
Matthieu Moy matthieu@grenoble-inp.fr writes:
Tanay Abhra tanay...@gmail.com writes:
+test_expect_success 'check line errors for malformed values' '
+ cp .git/config .git/config.old
Should this be mv not cp? You will be overwriting
Tanay Abhra tanay...@gmail.com writes:
+ test_when_finished mv .git/config.old .git/config
+ echo [alias]\n br .git/config
Is the use of \n portable?
Yes, you are right, will replace with printf in the next patch.
... or a cat .git/config \EOF, since this is the construct used
Matthieu Moy matthieu@grenoble-inp.fr writes:
Tanay Abhra tanay...@gmail.com writes:
+ test_when_finished mv .git/config.old .git/config
+ echo [alias]\n br .git/config
Is the use of \n portable?
Yes, you are right, will replace with printf in the next patch.
... or a cat
Semantic errors (for example, for alias.* variables NULL values are
not allowed) in configuration files cause a die printing the line
number and file name of the offending value.
Add a test documenting that such errors cause a die printing the
accurate line number and file name.
Signed-off-by:
Tanay Abhra tanay...@gmail.com writes:
+test_expect_success 'check line errors for malformed values' '
+ cp .git/config .git/config.old
+ test_when_finished mv .git/config.old .git/config
+ echo [alias]\n br .git/config
+ test_expect_code 128 git br 2result
+ grep
Matthieu Moy matthieu@grenoble-inp.fr writes:
Tanay Abhra tanay...@gmail.com writes:
+test_expect_success 'check line errors for malformed values' '
+cp .git/config .git/config.old
Should this be mv not cp? You will be overwriting the file from
scratch in the later part of this
6 matches
Mail list logo