Menu

#503 New nasm 2.09 cause build problems on i386/i686 machines

closed
nobody
Assembler (85)
5
2010-09-18
2010-09-15
No

I think this might be a reappearance of this bug:

http://sourceforge.net/tracker/?func=detail&aid=3026808&group_id=6208&atid=106208

Basically when building xbmc on f14 in mock or in the RPM Fusion build system, versions of nasm >= 2.09 causes problems on i686 machines:

make[1]: Entering directory
`/builddir/build/BUILD/xbmc-Dharma_beta1/xbmc/cores/paplayer/SPCCodec/SNES/SNESAPU'
nasm -Xvc -DLINUX -O1 -f elf -o SPC700.o SPC700.Asm
SPC700.Asm(328) : error: `%assign' expects a macro identifier
SPC700.Asm(328) : error: `%assign' expects a macro identifier
SPC700.Asm(377) : error: `%assign' expects a macro identifier
SPC700.Asm(377) : error: `%assign' expects a macro identifier
SPC700.Asm(377) : error: `%assign' expects a macro identifier
SPC700.Asm(440) : error: `%assign' expects a macro identifier
SPC700.Asm(481) : error: interminable macro recursion
SPC700.Asm(481) : error: `%xdefine' expects a macro identifier
SPC700.Asm(481) : error: interminable macro recursion
SPC700.Asm(481) : error: `%xdefine' expects a macro identifier
SPC700.Asm(806) : error: `%assign' expects a macro identifier

More details also here on upstream Fedora bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=633646

Discussion

  • Nobody/Anonymous

    I'm looking into it. Thanks for report.

     
  • Cyrill Gorcunov

    Cyrill Gorcunov - 2010-09-15

    ok, I managed to reproduce the issue and seems fixed it. so the patch will be discussed and reviewed first before hit nasm code.

    n.b. please next time push example of problematic code together with report, it's not that easy to understand what is wrong when you have no example.

     
  • Cyrill Gorcunov

    Cyrill Gorcunov - 2010-09-15

    test-case: compile as "nasm -felf SPC700.Asm"

     
  • Cyrill Gorcunov

    Cyrill Gorcunov - 2010-09-15

    test case attached

     
  • Nobody/Anonymous

    it seems we've resolved this issue though need some time for testing before merging

     
  • Cyrill Gorcunov

    Cyrill Gorcunov - 2010-09-18
    • status: open --> closed
     
  • Cyrill Gorcunov

    Cyrill Gorcunov - 2010-09-18

    Should be fixed in 2.09.02. Thanks.

     

Log in to post a comment.