Skip to content

Only buld changes#1369

Merged
rowleya merged 4 commits into
masterfrom
only_build_changes
Jun 15, 2026
Merged

Only buld changes#1369
rowleya merged 4 commits into
masterfrom
only_build_changes

Conversation

@rowleya

@rowleya rowleya commented Jun 8, 2026

Copy link
Copy Markdown
Member

This, along with the matching branch of SpiNNUtils, means that when you change a single file, and call make, it will only make the changes you have made, rather than everything. This speeds up the development process. Note that make clean will still clean everything, though I suppose you might want to add the modified_src folder to the clean process to be certain.

This part of the change keeps the object files around after the build. This means that a binary that needs multiple object files can just build the object files that have a dependency on a changed file somewhere.

Copilot AI review requested due to automatic review settings June 8, 2026 13:29

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR adjusts the C build makefile to retain object files between builds so incremental make runs only rebuild what changed, improving developer iteration speed.

Changes:

  • Mark built object files as secondary targets so make doesn’t delete them as intermediates.
  • Preserve object outputs to enable faster relinks when only a subset of sources change.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread c_common/front_end_common_lib/make/fec.mk
@Christian-B Christian-B self-requested a review June 15, 2026 12:17
@rowleya rowleya merged commit 9c738b7 into master Jun 15, 2026
13 checks passed
@rowleya rowleya deleted the only_build_changes branch June 15, 2026 14:00
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.

4 participants