Skip to content

buildsystem: drop source __pycache__ dirs in mrproper#1803

Open
nicolassanchez02 wants to merge 1 commit into
SFTtech:masterfrom
nicolassanchez02:fix/mrproper-pycache
Open

buildsystem: drop source __pycache__ dirs in mrproper#1803
nicolassanchez02 wants to merge 1 commit into
SFTtech:masterfrom
nicolassanchez02:fix/mrproper-pycache

Conversation

@nicolassanchez02
Copy link
Copy Markdown
Contributor

Merge Checklist

make mrproper left every __pycache__/ in the source tree behind. The clean targets only run against the out-of-source build dir, but running python from the repo (tests, codegen, the converter) scatters bytecode caches through openage/ and buildsystem/ and nothing swept those up. cleaninsourcebuild now removes them too, outside .bin, alongside the other in-source cleanups it already does.

Checked on a built tree: 46 __pycache__ dirs before, git clean -ndx clean after make mrproper.

Closes #442.

make mrproper left every __pycache__/ in the source tree behind: the
clean targets only run against the out-of-source build dir, but running
python from the repo (tests, codegen, the converter) scatters bytecode
caches through openage/ and buildsystem/. cleaninsourcebuild now removes
those too, outside .bin, matching the other in-source cleanups already
there.

Closes SFTtech#442.
@nicolassanchez02
Copy link
Copy Markdown
Contributor Author

The macOS-CI failure here isn't anything in this PR: configure aborts at find_package(Eigen3 3.3) rejecting the Eigen 5 the runner ships, before the build runs. That's the issue #1801 fixes. Once #1801 lands I'll rebase this onto master and the check goes green.

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.

mrproper doesn't remove some generated files

1 participant