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

[nomeata]

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

[nomeata]

For this problem, the patch at  http://anonscm.debian.org/darcs/pkg-haskell/experimental/ghc/patches/ArchMips seems to suffice, but then again this is hit by #5731:  https://buildd.debian.org/status/fetch.php?pkg=ghc&arch=mips&ver=7.4.0.20111219-3&stamp=1325289445

[nomeata]

#5732 of course, sorry.

[nomeata]

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

[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(-)

[igloo]

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

[igloo]

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

[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.

[simonmar]

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

[nomeata]

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

[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(-)