Skip to content

Conversation

@Swoffa
Copy link
Owner

@Swoffa Swoffa commented Nov 8, 2018

Created after looking at PR #36 .
This hopefully does the same thing, but in a broader sense, since the kickass_startup_root_path setting is also used for pre/post/make scripts.

Used for startup-file, make and pre/post build
@BroneCk
Copy link
Contributor

BroneCk commented Nov 8, 2018

@Swoffa it's about a parameter. c64debugger requires "-prg" parameter before prg name to launch, and this yours root_path just an absolute path param
So the debugCommand var should be like
""${kickass_debug_path}" ${kickass_debug_args} -logfile "${kickass_output_path}/${build_file_base_name}_ViceLog.txt" -moncommands "${kickass_output_path}/${build_file_base_name}_MonCommands.mon" -prg "${kickass_output_path}/${kickass_compiled_filename}""

Simplest thing is to move ${kickass_debug_args} just right before "${kickass_output_path}/${kickass_compiled_filename}""
and then when you type in kickass_debug_args in config "-prg " it will run like a charm :)

@Swoffa
Copy link
Owner Author

Swoffa commented Nov 8, 2018

Referenced wrong PR in my comment, corrected,

@Swoffa
Copy link
Owner Author

Swoffa commented Nov 8, 2018

@BroneCk

@Swoffa it's about a parameter. c64debugger requires "-prg" parameter before prg name to launch, and this yours root_path just an absolute path param

Yes, sorry, i referenced the wrong thing in my comment.

@daleicious
Copy link
Collaborator

Oh nice @Swoffa - sounds better :) I won't be able to properly review/test for a at least a few days maybe a week (I'm in the middle of a huge project at work). If you don't wish to wait then might as well merge - I trust your code more than mine! :)

@Swoffa Swoffa requested a review from daleicious November 9, 2018 12:32
@Swoffa
Copy link
Owner Author

Swoffa commented Nov 9, 2018

Oh nice @Swoffa - sounds better :) I won't be able to properly review/test for a at least a few days maybe a week (I'm in the middle of a huge project at work). If you don't wish to wait then might as well merge - I trust your code more than mine! :)

No hurry, no hurry at all, do it whenever you get time, if you want.

@Swoffa Swoffa force-pushed the master branch 2 times, most recently from c573ace to 7a55fc2 Compare March 8, 2021 09:44
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.

4 participants