Tantric's Hexxagon example app fails to load - is this a bug in r43?

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Install Hexxagon app on an SD card as instructed at
http://wiibrew.org/wiki/Hexxagon
2. Launch app from Homebrew Channel, running on System Menu 4.0E
3. DOSBox appears to use sd:/DOSBox/dosbox.conf instead of
sd:/apps/hexxagon/dosbox.conf, sits at Z:\> prompt as a result instead of
launching the game

Tantric suggested in the Talk section [1] of the aforementioned Wiki page
that DOSBox might be trying to load "apps/hexxagon/dosbox.conf" instead of
"sd:/apps/hexxagon/dosbox.conf".

C++ isn't my first language, but I've put together a simple test case
(attached) for the CreateAppPath function added in r43 - when I feed it
"sd:/apps/hexxagon/", it gives me back "apps/hexxagon/" - is this the
expected input and output, and is it the cause of the problem?

[1] http://wiibrew.org/wiki/Talk:Hexxagon

Original issue reported on code.google.com by jonathan.buchanan on 15 May 2009 at 11:50

Attachments:

• test.cpp

[GoogleCodeExporter]

Yes, I knew it was doing this. The question is if "apps/hexxagon/dosbox.conf" 
works 
fine, and the answer is "sometimes". eg: for me, and some others, but not for 
you. 
So, probably adding sd:/ will fix this.

Original comment by dborth@gmail.com on 15 May 2009 at 3:46

• Changed state: Accepted

[GoogleCodeExporter]

May as well have a crack at it while I'm here - I've attached a diff which 
removes
the attempt to strip up to and including the first "/".

Built it, replaced sd:/apps/hexxagon/boot.dol and Hexxagon now works for me as 
expected.

Apologies for the "WTF?" in advance if I've committed any major C++ sins in 
doing so,
but it's been most educational :)

Original comment by jonathan.buchanan on 16 May 2009 at 2:21

Attachments:

• CreateAppPath.diff

[GoogleCodeExporter]

Thanks for trying this. I will actually do this differently and still strip the 
sd:/ 
from the start and re-add it later. Why? because older versions of HBC add 
fat:/ at 
the start (the old standard), which will fail for those people running an older 
HBC. 
But thanks for testing the fix out and confirming it works :D

Original comment by dborth@gmail.com on 16 May 2009 at 4:13

• Changed state: Fixed

[GoogleCodeExporter]

Sorry about that. This was a feature I asked Tantric to add at the last minute 
when 
he was just about to release it, so it was a bit rushed.

Original comment by Carl.Kenner@gmail.com on 18 May 2009 at 4:10

[GoogleCodeExporter]

Original comment by dborth@gmail.com on 4 Jun 2009 at 5:30

• Changed state: Completed