Skip to content

Added kickass_startup_file_root_path #36

Closed
daleicious wants to merge 1 commit intoSwoffa:masterfrom
daleicious:master
Closed

Added kickass_startup_file_root_path #36
daleicious wants to merge 1 commit intoSwoffa:masterfrom
daleicious:master

Conversation

@daleicious
Copy link
Collaborator

I have been working on a project with a large number of asm files, organized into many folders. While working, it became onerous to make a change in a file in a subfolder, then hunt for the Startup.asm file in my main folder (or another file in the same root folder) and activate that file in Sublime, before hitting F5 to test my changes.

With this pull request, I've added a new setting, "kickass_startup_file_root_path", which can be set to a fully qualified path to the folder containing the "Startup.asm" to be compiled and run when F5/Shift-F5 are pressed.

Now we can have a robust file folder structure and have a single Startup.asm that will be built and run whenever we hit F5 from any asm file in our tree.

Note that this is they first time I've coded in Python, so I cannot vouch for the voracity of my code, and I may not have tested all the variations of settings in the Sublime package. I did test F5 vs F7 on Windows and both behaved as I would have expected.

Anyway I figured I'd fork the project and send you a PR in case you wanted to include this for others to use (tho please carefully review my code if you do merge the PR since I'm a n00b with Python and Sublime package development!)

…l path to the folder containing the Startup.asm file that is built via F5/etc. This allows us to work on a file in a subfolder of a project, and easily test via F5 without needing to select the main startup.asm file or another file in the main directory.
@daleicious
Copy link
Collaborator Author

Oh one important note: the path provided MUST end with "/" - ie, "c:/projects/mygame/src/".

@Swoffa
Copy link
Owner

Swoffa commented Nov 5, 2018

Thanks @daleicious!
Thats great. Will review and test soon.

@Swoffa
Copy link
Owner

Swoffa commented Nov 8, 2018

@daleicious
Inspired by the changes you suggested in this PR, I did the same thing but also with support for pre/post/make-scripts.
It is available in PR #38 and branch startup_root_path.

The setting name is kickass_startup_root_path, so thats a difference.

Can you please review and test if it does the same thing that you intended this PR to do.

Many thanks again!!

@Swoffa
Copy link
Owner

Swoffa commented Nov 21, 2018

Closing this. We continue with #38 instead.

@Swoffa Swoffa closed this Nov 21, 2018
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.

2 participants