Closed GoogleCodeExporter closed 8 years ago
Some tasks for this enhancement:
1. Create a "Link to Phase Shift" menu function that queries for the path of
the Phase Shift executable and song folder, which will be stored in variables
the same way that eof_fof_songs_path and eof_fof_executable_path currently are.
2. Modify eof_menu_song_test() to take a parameter and launch either Phase
Shift or FoF based on that parameter.
3. Modify the eof_song_menu[] dialog to call a function
(eof_menu_song_test_fof) to call eof_menu_song_test(), since dialog menu
functions cannot have parameters. Add a menu item for "Test in Phase Shift",
ensuring that EOF_LINUX_DISABLE is used to conditionally disable the menu item
if the platform isn't Windows.
4. Update the documentation.
Original comment by raynebc
on 20 Feb 2011 at 7:15
[deleted comment]
r708 completes tasks 1, 2 and 3. This does need to be tested, but provided
that it works, all that is left is to update the documentation.
Original comment by raynebc
on 24 Feb 2011 at 12:32
r709 adjusts the command line, but it still doesn't seem to be working for some
reason.
Original comment by raynebc
on 24 Feb 2011 at 3:39
At this point, I'm convinced that there is a limitation in the method EOF uses
to launch the executable. The debug logic exports the command that is
launched, and it looks perfectly fine, but the command doesn't seem to launch
Phase Shift.
Original comment by raynebc
on 28 Feb 2011 at 6:39
NewCreature wasn't immediately sure why it isn't working either. It may be
best to just write a batch file that is launched. The "Test in Phase Shift"
debug text file output I'm currently creating can just be altered to write to a
.bat file and launched with eof_system() instead.
Original comment by raynebc
on 3 Mar 2011 at 8:38
r717 gets the feature working in Phase Shift. All that's left is to update the
documentation.
Original comment by raynebc
on 4 Mar 2011 at 3:44
r718 updates the documentation.
Original comment by raynebc
on 4 Mar 2011 at 6:57
Original issue reported on code.google.com by
raynebc
on 19 Feb 2011 at 12:33