git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Test 9500 - unhelpful without Perl::CGI

From
Junio C Hamano <gitster@pobox.com>
Date
Jun 15, 2011, 05:59 UTC
Message-ID
<7vmxhjk539.fsf@alter.siamese.dyndns.org>
In-Reply-To
<BANLkTikj0GQCN94PvZz6n6fJJb5sHMApig@mail.gmail.com>
Linus Torvalds <torvalds@linux-foundation.org> writes:
Show 7 quoted lines
> This has probably been going on forever, and the only reason I noticed
> is that I had a machine that doesn't have perl-CGI installed.
>
> I'm talking about t9500-gitweb-standalone-no-errors.
>
> Anyway, if you don't have that, the test fails at everything, and it
> does so without any clues about _why_ it fails.
Something like this would hopefully suffice.

It seems that we do test for the version of Encode, but we do not specifically test CGI or other modules that people do not have to slurp from CPAN?

 t/gitweb-lib.sh |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/t/gitweb-lib.sh b/t/gitweb-lib.sh
index 143eb1f..292753f 100644
--- a/t/gitweb-lib.sh
+++ b/t/gitweb-lib.sh
@@ -107,4 +107,9 @@ perl -MEncode -e '$e="";decode_utf8($e, Encode::FB_CROAK)' >/dev/null 2>&1 || {
 	test_done
 }
 
+perl -MCGI -MCGI::Util -MCGI::Carp -e 0 >/dev/null 2>&1 || {
+	skip_all='skipping gitweb tests, CGI module unusable'
+	test_done
+}
+
 gitweb_init
Previous: Linus TorvaldsNext: Sverre Rabbelier
Message 2 of 6 in “Test 9500 - unhelpful without Perl::CGI”
  1. Linus TorvaldsJun 15, 2011
  2. Junio C HamanoJun 15, 2011
  3. Sverre RabbelierJun 15, 2011
  4. Junio C HamanoJun 15, 2011
  5. Junio C HamanoJun 15, 2011
  6. Linus TorvaldsJun 15, 2011

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.