Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

New CI build scripts for flexible directory strucutre #240

Merged
merged 7 commits into from
Dec 1, 2021

Conversation

stephenhensley
Copy link
Collaborator

@stephenhensley stephenhensley commented Nov 30, 2021

We're planning on using more directories to better organize the existing (and future) examples provided in the repo.

The root level of the repo isn't planned to change too much, but a bunch of the library, and utility folders are filtered out of the compile-search in the new script.

Functionally, the new build_examples.py works the same as the build_examples.sh it is replacing, but it will support examples nested in other directories (i.e. seed/dsp/*, etc.).

This will be necessary for supporting new structures like requested in #239


The new build examples script has been tested to work with the existing structure and tested for recursive path-searching.

The build_dist.py script still needs to be updated to properly remap the "platform" to be a combo of folders (i.e. seed/dsp will end up showing up in the web programmer as "seed_dsp" under platform).

While making a few changes I'd also like to address #129 since it's very old, and causes annoying merge conflicts for pretty much no reason.

@stephenhensley
Copy link
Collaborator Author

Looks like everything (including the nested-folder stuff) is working correctly (patch_sm_GettingStarted bins are in place).

I'm not going to merge it now incase it breaks the web-programmer over night, but it should be good to go.

@stephenhensley stephenhensley merged commit 0e93f02 into master Dec 1, 2021
@stephenhensley stephenhensley deleted the new_examples_building branch December 1, 2021 16:44
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