-
Notifications
You must be signed in to change notification settings - Fork 0
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
Consider supporting SBC-GS image #15
Comments
I personally do not use that image, created the ticket just as a suggestion. So it is hard for me to tell if the support is good or not. Did I get it correctly that your proposal is to make the paths to the config files and scripts configurable? |
yes, correct |
Well, for me personally it sounds a bit strange to let the end user edit the file locations. I rater expect the ground station developers to define those locations in some config files when they assemble their OS images 🤔 . So for the end user it just works out of the box |
You have a good point. That kind of access is potential security issue. I always think what do I want, as a developer/hacker and necessarilty about security.
…On Fri, Nov 8, 2024 at 2:15 PM Sergey Prokhorov ***@***.***> wrote:
Well, for me personally it sounds a bit strange to let the end user edit
the file locations. I rater expect the ground station developers to define
those locations in some config files when they assemble their OS images 🤔 .
—
Reply to this email directly, view it on GitHub
<#15 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAPBBJTUCYUIM36QQ6NPLLTZ7UZWTAVCNFSM6AAAAABP2SGP36VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRVHAZTGNZZGU>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
SBC GS of mine is still on active dev, when it stable I'll try to build improver in. |
I removed settings page for now but we can add a config with the correct file locations |
There is an alternative image for Radxa with much more features: https://github.com/zhouruixi/SBC-GS
Might be cool to have a look and decide if it can be supported as well.
The text was updated successfully, but these errors were encountered: