Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove installed #191

Merged
merged 5 commits into from
Apr 14, 2023
Merged

Remove installed #191

merged 5 commits into from
Apr 14, 2023

Conversation

Christian-B
Copy link
Member

@Christian-B Christian-B commented Apr 13, 2023

This is an extra check of the install

It also avoid duplicate aplx files to finds

All files that should have been copied to site packages by the install are removed.

All c code that should have been built is removed

PDP2 is not removed due to:
SpiNNakerManchester/SpiNNaker_PDP2#71

This exposed various small issues which are fixed below:
These are all independent and can be merged before this one:

Problems included:

test cfg checker which was using up and down to find cfg files are than package import

Not installing aplx files that the need to be built. Build first then install them

board_test_configuration.py and the cfg it uses being outside of the spinnman package yet used inside

  • The cfg includes some Machine options which have been moved to spinnman.cfg

spinnaker_testbase/test_no_job_destroy has to be run as a module as it is now only in site_packages

@Christian-B Christian-B merged commit 1ac8618 into main Apr 14, 2023
@Christian-B Christian-B deleted the remove_installed branch April 14, 2023 08:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants