Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#5734 closed bug (fixed)

Build failure on mips: ghc-cabal: Failed to read "target arch" value "ArchMipseb"

Reported by: nomeata Owned by: igloo
Priority: highest Milestone: 7.4.1
Component: Compiler Version: 7.3
Keywords: Cc:
Operating System: Linux Architecture: mips
Type of failure: Building GHC failed Test Case:
Blocked By: Blocking:
Related Tickets: Differential Rev(s):
Wiki Page:

Description

Hi,

the 7.4.1 release candidate failed to build on mips; previous versions built fine. It would be great if mips/mipsel support would not be dropped, if it can be helped:

"inplace/bin/ghc-cabal" configure --with-ghc="/build/buildd2-ghc_7.4.0.20111219-2-mips-qz6aVF/ghc-7.4.0.20111219/inplace/bin/ghc-stage1" --with-ghc-pkg="/build/buildd2-ghc_7.4.0.20111219-2-mips-qz6aVF/ghc-7.4.0.20111219/inplace/bin/ghc-pkg" --flag=include-ghc-prim --enable-library-profiling --disable-library-for-ghci --configure-option=CFLAGS=" -fno-stack-protector  " --configure-option=LDFLAGS=" -Wl,--hash-size=31 -Wl,--reduce-memory-overheads  " --configure-option=CPPFLAGS="   " --with-gcc="/usr/bin/gcc" --configure-option=--with-cc="/usr/bin/gcc" --with-ar="/usr/bin/ar" --with-ranlib="true" -- dist-install libraries/ghc-prim
Configuring ghc-prim-0.2.0.0...
ghc-cabal: Failed to read "target arch" value "ArchMipseb"
make[2]: *** [libraries/ghc-prim/dist-install/package-data.mk] Error 1

(https://buildd.debian.org/status/fetch.php?pkg=ghc&arch=mips&ver=7.4.0.20111219-2&stamp=1324654091)

and

"inplace/bin/ghc-cabal" configure --with-ghc="/build/buildd-ghc_7.4.0.20111219-2-mipsel-7rXnvK/ghc-7.4.0.20111219/inplace/bin/ghc-stage1" --with-ghc-pkg="/build/buildd-ghc_7.4.0.20111219-2-mipsel-7rXnvK/ghc-7.4.0.20111219/inplace/bin/ghc-pkg" --flag=include-ghc-prim --enable-library-profiling --disable-library-for-ghci --configure-option=CFLAGS=" -fno-stack-protector  " --configure-option=LDFLAGS=" -Wl,--hash-size=31 -Wl,--reduce-memory-overheads  " --configure-option=CPPFLAGS="   " --with-gcc="/usr/bin/gcc" --configure-option=--with-cc="/usr/bin/gcc" --with-ar="/usr/bin/ar" --with-ranlib="true" -- dist-install libraries/ghc-prim
Configuring ghc-prim-0.2.0.0...
ghc-cabal: Failed to read "target arch" value "ArchMipsel"
make[2]: *** [libraries/ghc-prim/dist-install/package-data.mk] Error 1

https://buildd.debian.org/status/fetch.php?pkg=ghc&arch=mipsel&ver=7.4.0.20111219-2&stamp=1324731190

Thanks, Joachim on behalf of the Debian Haskell Team

Attachments (1)

0002-Add-Architecture-names-from-aclocal.mk-missing-in-Pl.patch (1.1 KB) - added by nomeata 8 years ago.
Patch in git format, needs patch from #5735 applied first to apply cleanly.

Download all attachments as: .zip

Change History (14)

comment:2 Changed 8 years ago by nomeata

#5732 of course, sorry.

comment:3 Changed 8 years ago by igloo

difficulty: Unknown
Milestone: 7.4.1
Priority: normalhighest

Changed 8 years ago by nomeata

Patch in git format, needs patch from #5735 applied first to apply cleanly.

comment:4 Changed 8 years ago by simonmar

Owner: set to simonmar

comment:5 Changed 8 years ago by marlowsd@…

commit 3fc68b5c356b39b2b52a86d953367d0021c13262

Author: Simon Marlow <marlowsd@gmail.com>
Date:   Wed Jan 4 11:44:02 2012 +0000

    Remove missing archs (mipseb, mipsel, alpha) (#5734)
    
    It doesn't hurt to map these to ArchUnknown since we don't need to
    know anything specific about them, and adding them would be a pain
    (there are a bunch of places where we have to case-match on all the
    arches to avoid warnings).

 aclocal.m4 |    9 ---------
 1 files changed, 0 insertions(+), 9 deletions(-)

comment:6 Changed 8 years ago by simonmar

Status: newmerge

comment:7 Changed 8 years ago by igloo

Owner: simonmar deleted
Status: mergenew

Merged as 7f58606283d859d64be89715479819e1cf00534d, but I think we need to add them to the ArchUnknown list still.

comment:8 Changed 8 years ago by igloo

Owner: set to igloo

comment:9 Changed 8 years ago by igloo

Resolution: fixed
Status: newclosed

Done in HEAD:

commit af8cb680f41b0fdc0e22b1508d869e4ad15486c4

Author: Ian Lynagh <igloo@earth.li>
Date:   Sat Jan 7 22:16:41 2012 +0000

    Add alpha, mips, mipseb, mipsel to the list of ArchUnknown arches

and 7.4:

commit 370f755e118bfb2fabd34334f6b6f7bce20f5c77

Author: Ian Lynagh <igloo@earth.li>
Date:   Sat Jan 7 22:16:41 2012 +0000

    Add alpha, mips, mipseb, mipsel to the list of ArchUnknown arches

comment:10 Changed 8 years ago by nomeata

Thanks for fixing this in the 7.4 branch as well. Can we have the fixes for #5735, #5732 and #5733 also there? Then I’d package another snapshot in Debian and test them across all architectures, before the 7.4.1-release.

comment:11 Changed 8 years ago by simonmar

I think all those fixes are now merged into the 7.4 branch. Do you see any missing?

comment:12 Changed 8 years ago by nomeata

Ah, my bad, I was reading the git log, thought it was chronologically and stopped too early.

comment:13 Changed 7 years ago by ian@…

commit 9756690fe7aa26aee6955d0b720377d53170c542

Author: Ian Lynagh <ian@well-typed.com>
Date:   Tue Nov 13 17:58:52 2012 +0000

    Tell the compiler about alpha, mipseb and mipsel again; fixes #7339
    
    This reverts the compiler parts of
    
        commit 7b594a5d7ac29972db39228e9c8b7f384313f39b
        Author: David Terei <davidterei@gmail.com>
        Date:   Mon Nov 21 12:05:18 2011 -0800
    
        Remove registerised code for dead architectures: mips, ia64, alpha,
        hppa1, m68k
    
    In particular, we want to know whether bewareLoadStoreAlignment should
    return True or False for them.
    
    It also reverts
    
        commit 3fc68b5c356b39b2b52a86d953367d0021c13262
        Author: Simon Marlow <marlowsd@gmail.com>
        Date:   Wed Jan 4 11:44:02 2012 +0000
    
        Remove missing archs (mipseb, mipsel, alpha) (#5734)
    
        It doesn't hurt to map these to ArchUnknown since we don't need to
        know anything specific about them, and adding them would be a pain
        (there are a bunch of places where we have to case-match on all the
        arches to avoid warnings).

 aclocal.m4                                         |   11 ++++++++++-
 compiler/cmm/PprC.hs                               |    3 +++
 compiler/nativeGen/AsmCodeGen.lhs                  |    6 ++++++
 compiler/nativeGen/RegAlloc/Graph/TrivColorable.hs |   12 ++++++++++++
 compiler/nativeGen/RegAlloc/Linear/FreeRegs.hs     |    3 +++
 compiler/nativeGen/RegAlloc/Linear/Main.hs         |    3 +++
 compiler/nativeGen/TargetReg.hs                    |   15 +++++++++++++++
 compiler/utils/Platform.hs                         |    3 +++
 8 files changed, 55 insertions(+), 1 deletions(-)
Note: See TracTickets for help on using tickets.