Skip to content

+fix: source file resolving if debugee is located in subdir - #2

Merged
smokku merged 10 commits into
X65:mainfrom
mlauke:feature/steckschwein_support
Apr 10, 2026
Merged

+fix: source file resolving if debugee is located in subdir#2
smokku merged 10 commits into
X65:mainfrom
mlauke:feature/steckschwein_support

Conversation

@mlauke

@mlauke mlauke commented Feb 19, 2026

Copy link
Copy Markdown
Contributor

+fix: error message handling
+feat: add steckschwein to list of emulators supporting cc65-dbg

+fix: error message handling
+feat: add steckschwein to list of emulators supporting cc65-dbg
@mlauke

mlauke commented Feb 20, 2026

Copy link
Copy Markdown
Contributor Author

mhm... little bit tricky. will do some regression tests for windows... and with a .dbg file with absolute file names.

…sub directory

+feat: support for additional src directories via srcDirs launch confifuration property - e.g. used for cc65 library source lookup path
@mlauke
mlauke marked this pull request as ready for review March 26, 2026 08:10
@mlauke
mlauke marked this pull request as draft March 26, 2026 08:11
@mlauke

mlauke commented Mar 26, 2026

Copy link
Copy Markdown
Contributor Author

@smokku hi, needed to change the source lookup a little bit, but now it works for various workspace setups. e.g. workspaces with sub directories. furthermore i've added an srcDirs configuration option to allow for configuration of additional paths to lookup source files. useful for cc65 module libraries. e.g. had to deal with .dbg files like

version	major=2,minor=0
info	csym=0,file=29,lib=1,line=2863,mod=9,scope=16,seg=7,span=1456,sym=2026,type=9
file	id=0,name="games/shmup/shmup.s",size=16686,mtime=0x69AC1294,mod=0
...
file	id=7,name="steckos/asminc/common.inc",size=7523,mtime=0x69A1795B,mod=0+0
file	id=20,name="../../steckos/asminc/zeropage.inc",size=2124,mtime=0x69A1795B,mod=8
file	id=21,name="../../steckos/asminc/vdp.inc",size=12927,mtime=0x69A1795B,mod=3+4+5+6+7
file	id=22,name="../../steckos/asminc/system.inc",size=2499,mtime=0x69A1795B,mod=3+4+5+6+7
...
file	id=23,name="t99xx/mode2.s",size=3118,mtime=0x69A1795B,mod=3
file	id=24,name="t99xx/t99x_memcpy.s",size=2349,mtime=0x69A1795B,mod=4
...
lib	id=0,name="steckos/lib/steckschwein.a"

where source paths of modules within a library also use relative paths which may not reside in the current workspace or different directory from the program which is debugged.

@mlauke
mlauke marked this pull request as ready for review March 26, 2026 10:12
Comment thread src/cc65Debug.ts Outdated
Comment thread src/cc65Debug.ts
Comment thread src/cc65Debug.ts Outdated
Comment thread src/cc65Debug.ts
Comment thread src/cc65Debug.ts Outdated
Comment thread src/cc65Debug.ts Outdated
Comment thread README.md
@mlauke
mlauke requested a review from smokku March 28, 2026 15:57

@smokku smokku left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@smokku
smokku merged commit c8dfd42 into X65:main Apr 10, 2026
1 check passed
@smokku

smokku commented Apr 10, 2026

Copy link
Copy Markdown
Member

@mlauke Thank you for your contribution. 😺

@mlauke

mlauke commented Apr 10, 2026

Copy link
Copy Markdown
Contributor Author

@smokku forgot to increase version number in package.json etc... or will you do it before shipping it to https://marketplace.visualstudio.com/items?itemName=X65-Project.cc65-dbg ?

@smokku

smokku commented Apr 10, 2026

Copy link
Copy Markdown
Member

or will you do it before shipping it marketplace.visualstudio.com

Bumping the version is the part of release procedure.

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