search

pine64 / bl_iot_sdk

BL602 SDK (Pine64 fork)
https://pine64.github.io/bl602-docs/
Apache License 2.0
134 stars 58 forks source link

Added .gitignore for build folders under customer_app #50

Closed n0toose closed 3 years ago

n0toose commented 3 years ago

I had to come up with a small one-liner in bash in order to do this, I'm leaving this here for future reference;

for i in $(ls); do; touch $i/.gitignore; (echo build_out > $i/.gitignore); done;

I believe that this small change will make development significantly less frustrating. All of the Makefile files inherit from include $(BL60X_SDK_PATH)/make_scripts_riscv/project.mk, so although I only tested whether the output directory was the same for bl602_boot2, bl602_boot2_mini, bl602_demo_at and bl602_demo_event (as well as sdk_app_helloworld, just to be sure), the output folder should always be build_out for every app under customer_app.

Avamander commented 3 years ago

Added the folder to the global one, there's absolutely no need to place it each and every directory.

n0toose commented 3 years ago

Understood.

n0toose commented 3 years ago

I just thought that it was more appropriate to do that with every single individual folder, since they concern different codebases and because I didn't want to assume that the output folders are always build_out across the repository, which seemed volatile from my perspective as far as this thing is concerned.

Avamander commented 3 years ago

It might be a bit more volatile, but it's a single place to keep track of, updating ten .gitignores on makefile change or when a new example is added is a bit more error-prone and fragile.

gamelaster commented 3 years ago

@panos Thank you for your pull request. Even the PR was closed, it was still applied, but bit different, so you can apply for the FREE EVB. For receiving the free PineCone, please sign up at this link. (If there will be any issues with signing up, please let me know here).

n0toose commented 3 years ago

Fair enough, I'll push a few more pull requests and request for it in a bit. @gamelaster

Thanks a lot! :D