Skip to content

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

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

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

Conversation

@mlauke
Copy link
Copy Markdown
Contributor

@mlauke mlauke commented Feb 19, 2026

+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
Copy link
Copy Markdown
Contributor Author

mlauke commented Feb 20, 2026

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
Copy link
Copy Markdown
Contributor Author

mlauke commented Mar 26, 2026

@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
@mlauke mlauke requested a review from smokku March 28, 2026 15:57
Copy link
Copy Markdown
Member

@smokku smokku left a comment

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
Copy link
Copy Markdown
Member

smokku commented Apr 10, 2026

@mlauke Thank you for your contribution. 😺

@mlauke
Copy link
Copy Markdown
Contributor Author

mlauke commented Apr 10, 2026

@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
Copy link
Copy Markdown
Member

smokku commented Apr 10, 2026

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