Skip to content

build: gate debug prefix map to GitHub Actions#8215

Closed
scarf005 wants to merge 1 commit intocataclysmbn:mainfrom
scarf005:fix/debug-source-paths
Closed

build: gate debug prefix map to GitHub Actions#8215
scarf005 wants to merge 1 commit intocataclysmbn:mainfrom
scarf005:fix/debug-source-paths

Conversation

@scarf005
Copy link
Member

Purpose of change (The Why)

Regression from #7678 remapped local debug file paths to build-dir-relative paths, which breaks IDE/GDB breakpoints in local builds.

Describe the solution (The How)

Apply -ffile-prefix-map=${REL_SRC_PATH}=. only when running on GitHub Actions (GITHUB_ACTIONS is set), while keeping local debug symbols mapped to real source paths.

Describe alternatives you've considered

  • Remove -ffile-prefix-map entirely (keeps local debugging but drops CI/worktree ccache path normalization).

Testing

  • cmake --preset linux-full -DLUA_DOCS_ON_BUILD=OFF
  • cmake --build --preset linux-full --target cataclysm-bn-tiles cata_test-tiles
  • ./out/build/linux-full/tests/cata_test-tiles "time_duration_to_string"
  • Confirmed no ffile-prefix-map in out/build/linux-full/compile_commands.json in local env.

Additional context

LUA_DOCS_ON_BUILD was disabled for local verification because docs generation in this environment expects a root-level ./cataclysm-bn-tiles path and fails post-link.

Checklist

Mandatory

Optional

  • This is a PR that modifies build process or code organization.
  • Please document the changes in the appropriate location in the docs/ folder.
  • If documentation for this feature or process does not exist, please write it.
  • If the change alters versions of software required to build or work with the game, please document it.

@scarf005 scarf005 changed the title fix: gate debug prefix map to GitHub Actions build: gate debug prefix map to GitHub Actions Feb 20, 2026
@scarf005
Copy link
Member Author

on second thoughts, reverting this changes cause LLDB in local linux to not work

@scarf005 scarf005 closed this Feb 20, 2026
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.

1 participant