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

Assembled file isn't loaded ... #33

Closed
BacchusFLT opened this Issue Oct 29, 2018 · 7 comments

Comments

Projects
None yet
2 participants
@BacchusFLT
Copy link

BacchusFLT commented Oct 29, 2018

One want that pressing F7, the project should assemble, launch VICE and load the recently assembled file. However introducing segments, this doesn't work.

When I assemble the below WITH the ".segment" row, address 1000 has the labels
in the monitor, but not the actual code. The binary was clearly never loaded. Commenting out the .segment it's loaded as normal.

	.segment MyCode[]

	*=$1000
start:	inc $d020
	jmp start
@Swoffa

This comment has been minimized.

Copy link
Owner

Swoffa commented Oct 29, 2018

Thanks for reporting, will look into it.

@Swoffa

This comment has been minimized.

Copy link
Owner

Swoffa commented Dec 10, 2018

@BacchusFLT
My suggested solution is to put annotations in the file being built, something like this:

// @kickass-build: {"file-to-run": "file.prg", "disk-to-run": "disk.d64"} 
	.segment MyCode[]

	*=$1000
start:	inc $d020
	jmp start

The file-to-run-annotation sets which prg-file to run in vice (or C64Debugger) and the disk-to-run-annotation sets which d64 to load into Vice. If disk-to-run is set, file-to-run is loaded from that disk, otherwise from the file system.

Would this solution give you what you need?

@BacchusFLT

This comment has been minimized.

Copy link
Author

BacchusFLT commented Dec 10, 2018

@Swoffa

This comment has been minimized.

Copy link
Owner

Swoffa commented Dec 10, 2018

I wasn't aware of this option. Thx! /Pontus Berg

@BacchusFLT
No, no, it is not implemented yet! It is my suggestion, but I wanted to ensure it actually helps you before I implement it. From your answer I guess its "thumbs up". :)

@BacchusFLT

This comment has been minimized.

Copy link
Author

BacchusFLT commented Dec 10, 2018

@Swoffa

This comment has been minimized.

Copy link
Owner

Swoffa commented Jan 14, 2019

Fixed in PR #50. Will be in release 1.3.1.

@Swoffa

This comment has been minimized.

Copy link
Owner

Swoffa commented Jan 14, 2019

Version 1.3.1 available thru Package control now

@Swoffa Swoffa closed this Jan 14, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment