Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changing the devkitPro path from absolute to environment defined #31

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

john-peterson
Copy link
Member

Changing the devkitPro path from absolute to environment defined

Problem

The user sometimes want to place devkitPro in another path than "/c/devkitpro", f.e. "/c/file/source/lib/devkitpro", because

  • it's better organisation because other lib files are in that path

Solution

Changing "C:\devkitPro" to "$(DEVKITPRO_)" in the .vcxproj files because

  • that allow VS to locate declarations when devkitPro isn't in "C:\devkitPro"

MSYS path conversion

The environment variable is "DEVKITPRO_" instead of "DEVKITPRO" because

  • devkitPro is designed to build from MSYS where a Windows style path is unhandled as in the example in this topic
  • i.o.w. the MSYS automatic path conversion is inadequate for devkitPro make
# start devkitPro MSYS environment
call c:\file\source\lib\devkitpro\msys\msys.bat -mintty MSYS

# environment information
uname -a; echo $MSYSTEM
MSYS_NT-6.1 PC 1.0.17(0.48/3/2) 2011-04-24 23:39 i686 Msys
MSYS

# set Windows style devkitPro path
export DEVKITPRO=c:\\file\\source\\lib\\devkitpro
export DEVKITPPC=$DEVKITPRO\\devkitppc

# build WPAD
cd /c/file/source/dolphin/repo/Source/TestSuite/WPAD
make -r
      0 [main] make 22152 open_stackdumpfile: Dumping stack trace to make.exe.stackdump

# set Unix style devkitPro path
export DEVKITPRO_=$DEVKITPRO
export DEVKITPPC_=$DEVKITPPC
export DEVKITPRO=`cygpath DEVKITPRO_`
export DEVKITPPC=`cygpath DEVKITPPC_`
# status
echo $DEVKITPRO
/c/file/source/lib/devkitpro

# build WPAD
make -r
dolphintest_wpad.cpp
linking ... WPAD.elf
output ... WPAD.dol

@john-peterson
Copy link
Member Author

Adding visual stick position to pad test

Message

This message is at #31 (comment)

Commit

The commit is in this branch with the same title as this post

Commit message

because

  • it's useful for changing the stick radius

Problem

A visual compared to text stick position is useful for changing the stick position because

  • it's easier to determine when the stick is at the active range

Build

The .elf files are in this folder because

  • that allow testing this commit without a devkitPro environment

because

* a build test reduce commits with build errors
@john-peterson
Copy link
Member Author

Changing ARAM test header file path

Commit

And the commits

  • Changing the SD card reader object
  • Changing the GU vector struct name

Commit message

Changing ARAM test header file path

because

  • it's useful for building the program

Problem

The devkitPro tests don't build because of changes in devkitPro

This is a problem because

  • the tests are important to analyse emulation

because

* that allow placing the files in another path
because

* it's useful for changing the stick radius
because

* it's useful for building the program
in the ASM test

because

* it's useful to build the program
in the GX test

because

* it's useful to build the program
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant