search

dukesrg / rop3ds

MIT License
8 stars 3 forks source link

Build problem (sed, windows) #8

Closed Syphurith closed 9 years ago

Syphurith commented 9 years ago

@dukesrg I heard yours is a busybox one. Mine is GNU sed 4.2.1 provided by MSYS2. While make clean && make.. it was like sed -e "/17498'/{rLoadCode.utf8" -e "N}" -i index.html.

generating SPIDER_4X ROP
/bin/sed: -e expression #1, char 50: unterminated `s' command
Makefile:31: recipe for target 'index.html' failed
make: *** [index.html] Error 1

If execute manually yes i could get sed working (with a f*cking backslash). If auto with \ still fail.

sed -e "\/17498'/{rLoadCode.utf8" -e "N}" -i index.html
/bin/sed: -e expression #1, char 50: unterminated `s' command
Makefile:32: recipe for target 'index.html' failed
make: *** [index.html] Error 1

Some other lines would fail. It seems not be compatible to be executed manually. make -s LoadCode.dat ASFLAGS=-DSPIDER_4X -DSPIDER_ARM_CODE_OFFSET=0 Getting latest GNU sed source from gnu ftp.. Oh my, there is 4.2.2-2 offered from MSYS2.

Syphurith commented 9 years ago

Orz the sed 4.2.1 from PortableGit.. 4.2.2-2 from MSYS2.. I hate different Mingw enviorments now..

Syphurith commented 9 years ago

@dukesrg Problem solved once i removed my portablegit path from System PATH and use MSYS2.

dukesrg commented 9 years ago

That's shy I recommend to use toolchain with devkitPro MSYS that was originally used :)

Syphurith commented 9 years ago 
In file included from rop.S:4:0:
rop.h:425:3: error: #error ROP version not defined
  #error ROP version not defined
   ^
Makefile:12: recipe for target 'rop.elf' failed
make: *** [rop.elf] Error 1

Maybe you could remove rop.dat LoadCode.dat LoadCodeMset.dat from make all.

Syphurith commented 9 years ago

@dukesrg Wait. The version devkitPro MSYS offered is a 4.2.1, which doesn't suite your makefile.

dukesrg commented 9 years ago

@Syphurith I'm building with that sed! yes rop 3ds all rule is not quite right, just becase there is no need to make it all :)

Syphurith commented 9 years ago

@dukesrg on windows that sed version failed for me. so i would use newer one.

dukesrg commented 9 years ago

@Syphurith If you setup toolchain like described in readme.md and remove all conflicting values from %path% it will work fine.

Syphurith commented 9 years ago

@dukesrg all right then since target index.html now build for me. trouble maker, windows

dukesrg commented 9 years ago

@Syphurith yes but you cant force all devs switch to linux - we'd rather have as many as possible since there are actually A LOT of things to fix and implement

Syphurith commented 9 years ago

@dukesrg Oh well. With devkitPro MSYS. I would have to run it in cmd, but turns out to be fine.