Opened 3 years ago

Closed 3 years ago

#5281 closed bug (fixed)

Bad compiler warning

Reported by: augustss Owned by: simonpj
Priority: high Milestone: 7.2.1
Component: Compiler Version: 7.1
Keywords: Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Difficulty:
Test Case: rename/should_fail/T5281 Blocked By:
Blocking: Related Tickets:

Description

In a jumble of messages from lots of parallel invocations of ghc I see this message:

Implicit import declaration:
    Warning: In the use of `catch'
             (imported from Prelude, but defined in System.IO.Error):
             Deprecated: "Please use the new exceptions variant, Control.Exception.catch"

That's nice, but what module did this message come from?

Change History (5)

comment:1 Changed 3 years ago by simonmar

  • Milestone set to 7.2.1
  • Priority changed from normal to high

comment:2 Changed 3 years ago by simonpj

  • Owner set to simonpj

comment:3 Changed 3 years ago by simonpj

I'm on this, but it'll have to wait till I'm back from holiday.

comment:4 Changed 3 years ago by simonpj@…

commit 3fdd294af643a86162e544f442b0e36c57e1db36

Author: Simon Peyton Jones <simonpj@microsoft.com>
Date:   Wed Jul 27 10:39:07 2011 +0100

    Give a more helpful src location (Trac #5281)
    
    We weren't giving a useful location for the
    *implicit* import of Prelude.  Lennart rightly
    complained. This fixes it.

 compiler/main/HeaderInfo.hs       |   14 +++++++-------
 compiler/main/HscMain.lhs         |    9 ++++++---
 compiler/rename/RnNames.lhs       |   16 +++++++++-------
 compiler/typecheck/TcRnDriver.lhs |   21 +++++++++++++--------
 4 files changed, 35 insertions(+), 25 deletions(-)

comment:5 Changed 3 years ago by simonpj

  • Resolution set to fixed
  • Status changed from new to closed
  • Test Case set to rename/should_fail/T5281

OK done! Try now

Simon

Note: See TracTickets for help on using tickets.