From: Ramsay Jones <ramsay@ramsayjones.plus.com>
To: David Turner <dturner@twopensource.com>
Cc: Junio C Hamano <gitster@pobox.com>,
GIT Mailing-list <git@vger.kernel.org>
Subject: t7900-*.sh tesr #5 failure
Date: Tue, 3 May 2016 22:34:58 +0100 [thread overview]
Message-ID: <57291982.2000904@ramsayjones.plus.com> (raw)
Hi David,
Test t7900.5 fails for me, thus:
$ ./t7900-index-helper.sh -i -v -x -d
...
+ test -S .git/index-helper.sock
+ git status
On branch master
Untracked files:
(use "git add <file>..." to include in what will be committed)
err
nothing added to commit but untracked files present (use "git add" to track)
+ test -S .git/index-helper.sock
+ grep -q . err
error: last command exited with $?=1
not ok 5 - index-helper autorun works
#
# rm -f .git/index-helper.sock &&
# git status &&
# test_path_is_missing .git/index-helper.sock &&
# test_config indexhelper.autorun true &&
# git status &&
# test -S .git/index-helper.sock &&
# git status 2>err &&
# test -S .git/index-helper.sock &&
# ! grep -q . err &&
# git index-helper --kill &&
# test_config indexhelper.autorun false &&
# git status &&
# test_path_is_missing .git/index-helper.sock
#
$ cd trash\ directory.t7900-index-helper/
$ ls
err x.t
$ cat err
warning: We requested watchman support from index-helper, but it doesn't support it. Please use a version of git index-helper with watchman support.
$
[Yes, that is one long line in err!]
[At least, this is one of the failures, I have also seen git status failing
with a SIGPIPE.]
Note that I do not have the watchman libraries etc., so USE_WATCHMAN is
not defined. Note also, that I had an instance of git-index-helper still
running after the test failure (which I kill-ed).
I haven't spent any time debuging this, but some questions spring to
mind:
- can index-helper be used without watchman support?
- why is index-helper requesting watchman support, when it was
built without USE_WATCHMAN being defined?
- why is read-cache.o exporting the verify_index and
write_watchman_ext symbols?
- is index-helper any use/help without watchman support?
- is '! grep -q . err' meant to determine if the err file is
empty (ie git status did not issue an error message)?
[if yes, maybe 'test_must_be_empty err &&' would read better!]
Unfortunately, I have to run now, ...
ATB,
Ramsay Jones
next reply other threads:[~2016-05-03 21:35 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-03 21:34 Ramsay Jones [this message]
2016-05-05 21:34 ` t7900-*.sh tesr #5 failure David Turner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=57291982.2000904@ramsayjones.plus.com \
--to=ramsay@ramsayjones.plus.com \
--cc=dturner@twopensource.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
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 read-only IMAP folder(s) and NNTP newsgroup(s).