[GHC] #12225: Warn if test setting has no effect (e.g. compile_timeout_multiplier on run_command)

#12225: Warn if test setting has no effect (e.g. compile_timeout_multiplier on run_command) -------------------------------------+------------------------------------- Reporter: ezyang | Owner: Type: bug | Status: new Priority: low | Milestone: Component: Test Suite | Version: 8.0.1 Keywords: | Operating System: Unknown/Multiple Architecture: | Type of failure: None/Unknown Unknown/Multiple | Test Case: | Blocked By: Blocking: | Related Tickets: Differential Rev(s): | Wiki Page: -------------------------------------+------------------------------------- I recently wrote this line in an `all.T`. Can you figure out how it's wrong? {{{ test('tc266', [extra_clean(['Tc266.hi-boot', 'Tc266.o-boot', 'Tc266a.hi', 'Tc266a.o', 'Tc266.hi', 'Tc266.o']), compile_timeout_multiplier(0.01)] , run_command, ['$MAKE -s --no-print-directory tc266']) }}} That's right, it should be `run_timeout_multiplier`, not compile timeout multiplier, which is literally a no-op for this test. There should be a warning if this happens. -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/12225 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#12225: Warn if test setting has no effect (e.g. compile_timeout_multiplier on run_command) -------------------------------------+------------------------------------- Reporter: ezyang | Owner: thomie Type: bug | Status: new Priority: low | Milestone: Component: Test Suite | Version: 8.0.1 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: | Unknown/Multiple Type of failure: None/Unknown | Test Case: Blocked By: | Blocking: Related Tickets: | Differential Rev(s): Wiki Page: | -------------------------------------+------------------------------------- Changes (by thomie): * owner: => thomie Comment: Great idea, I'll do that. Keep 'em coming. -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/12225#comment:1 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler
participants (1)
-
GHC