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

Build System: Add project.mk as a Watched Dependency File #591

Merged
merged 4 commits into from
May 30, 2023

Conversation

Jake-Carter
Copy link
Contributor

This will add project.mk as a watched dependency file for the build system, which will address a lot of the issues we've been having lately, especially with Cordio library build options.

Any time project.mk changes a complete rebuild will be triggered including all library files.

These changes have been applied to the MAX32655 BLE_dats project for review/testing but are representative of what will go out to all micros. Please test this on your machine and confirm it works.

@Jake-Carter Jake-Carter added the build system This issue or pull request is related to the MSDK build system label May 18, 2023
@Jake-Carter
Copy link
Contributor Author

@EricB-ADI @AbbyWolf-ADI @EdwinFairchild @yc-adi please have a look, thanks

@EdwinFairchild
Copy link
Contributor

works on my linux end

- exporting PROJECTMK caused a regression for projects like BLE_otac/BLE_otas.  The "source" project would call into the other project's makefile, but exporting PROJECTMK would use the host project.mk file for everything
- Add support for PROJECTMK variable
@EricB-ADI
Copy link
Contributor

good for me

@EdwinFairchild
Copy link
Contributor

The time has come.

@Jake-Carter Jake-Carter merged commit 3b79077 into analogdevicesinc:main May 30, 2023
@Jake-Carter Jake-Carter deleted the fix/watch-projectmk branch May 30, 2023 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build system This issue or pull request is related to the MSDK build system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants