Skip to content
This repository was archived by the owner on May 4, 2026. It is now read-only.

fix(platformio): 🐛 Use relative paths instead of absolute paths - #157

Merged
ZZ-Cat merged 13 commits into
Version-1.1.0-Developmentfrom
is-my-QC-and-QL-still-broken
Aug 30, 2025
Merged

fix(platformio): 🐛 Use relative paths instead of absolute paths#157
ZZ-Cat merged 13 commits into
Version-1.1.0-Developmentfrom
is-my-QC-and-QL-still-broken

Conversation

@ZZ-Cat

@ZZ-Cat ZZ-Cat commented Aug 29, 2025

Copy link
Copy Markdown
Owner

When I initially containerised CFA, I had the hare-brained idea of using absolute paths in CFA's configuration files.
Turns out, this led to obscure permission denial errors whenever I tried to run both PlatformIO commands or CFA's build.py script as a part of the QC and QL workflows.

This resolves the broken QC and QL workflows. Everything should run as expected now.

This is the latest update for the Version-1.1.0-Development branch. Plus it makes the `is-my-QC-and-QL-still-broken` branch different enough to check whether-or-not my QC and Vulnerability Scanner is still broken.
@ZZ-Cat ZZ-Cat self-assigned this Aug 29, 2025
@ZZ-Cat ZZ-Cat added Bug Fix 👋💥🐞 Bug meet swatter! 🚨 Blocks Release 🚨 No release is to be made, until this is resolved. CI/CD 🚧 Continuous Integration/Continuous Deployment labels Aug 29, 2025
@ZZ-Cat

ZZ-Cat commented Aug 29, 2025

Copy link
Copy Markdown
Owner Author

Okay, that's wild. 🤔
I'm getting permission denied for CFA's workspace directory. That's why the CI is failing.

ZZ-Cat added 6 commits August 29, 2025 21:43
…litting the `cd` command from the same line that runs the script.

FAFO phase 3
It seems I am getting permission errors every time `build.py` is executed, so I'll see what happens here. FAFO phase 6
@ZZ-Cat

ZZ-Cat commented Aug 29, 2025

Copy link
Copy Markdown
Owner Author

Okay, so it seems I am running into permission errors on the /workspaces folder when both the QL and QC scripts try to run build.py or anything else PlatformIO and Python related... 🤔

ZZ-Cat added 4 commits August 29, 2025 22:42
This has shown to be unnecessary, as the permission error was not on this file itself, but with `/workspaces`
…r` with the part that installs PlatformIO

Would I still get the same permission error?
@ZZ-Cat

ZZ-Cat commented Aug 29, 2025

Copy link
Copy Markdown
Owner Author
Screenshot_20250830_113557

Awesome-sauce! =^/,..,^=
TIL: ALWAYS use relative paths!
Otherwise, I'll get obscure permission issues here, despite CFA building perfectly in my local environment.

That's one headache out of the way.
Now all I need to do is figure out what's going on with the build on compatible targets and why its status is not being reported. I may need to fix that in another branch or so... I don't know yet. But I'm glad I got the main headache that this Pull Request was attempting to fix sorted.

@ZZ-Cat ZZ-Cat changed the title ci(quality control): Is it still broken? ci(quality control): Use relative paths instead of absolute paths Aug 29, 2025
@ZZ-Cat ZZ-Cat changed the title ci(quality control): Use relative paths instead of absolute paths fix(platformio): 🐛 Use relative paths instead of absolute paths Aug 30, 2025
@ZZ-Cat
ZZ-Cat marked this pull request as ready for review August 30, 2025 00:02
@ZZ-Cat

ZZ-Cat commented Aug 30, 2025

Copy link
Copy Markdown
Owner Author

I also found out why the build_on_compatible_targets check is infinitely waiting for its status to be reported—it's leftovers from the old way I used to do it, and it has since been amalgamated into my Defect Detector.

So, I'll merge this as-is, because the important checks pass all a-okay. 👌🏼

@ZZ-Cat ZZ-Cat moved this to In Progress in Development Schedule Aug 30, 2025
@ZZ-Cat ZZ-Cat added this to the Version 1.1.0 milestone Aug 30, 2025
@ZZ-Cat ZZ-Cat removed the 🚨 Blocks Release 🚨 No release is to be made, until this is resolved. label Aug 30, 2025
@ZZ-Cat
ZZ-Cat merged commit 4d90014 into Version-1.1.0-Development Aug 30, 2025
4 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Development Schedule Aug 30, 2025
@ZZ-Cat
ZZ-Cat deleted the is-my-QC-and-QL-still-broken branch August 30, 2025 00:09
@ZZ-Cat ZZ-Cat mentioned this pull request Aug 30, 2025
34 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Bug Fix 👋💥🐞 Bug meet swatter! CI/CD 🚧 Continuous Integration/Continuous Deployment

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant