Skip to content

Conversation

@bwegh
Copy link

@bwegh bwegh commented Sep 24, 2025

this allows for the user to adjust permissions on the grisp sub-path and does not force root permissions for building

@ziopio
Copy link
Member

ziopio commented Sep 24, 2025

Good point, I am experiencing the same on linux.
Only annoyng thing is that the user needs to manually create and change permisisons on said directory.

@bwegh
Copy link
Author

bwegh commented Sep 27, 2025

totally agree, yet it is better than building as root .
alternative would be to store all data in $HOME

@sylane
Copy link
Contributor

sylane commented Sep 29, 2025

Instead of adding an extra directory depth, wouldn't manually creating /opt/grisp_alloy_sdk changing to not fail if the directory already exists be good enough ? Did you try creating /opt/grisp_alloy_sdk manually ?

@sylane
Copy link
Contributor

sylane commented Sep 30, 2025

@bwegh So it seems to me that if you create the directory /opt/grisp_alloy_sdk before running build-sdk.sh with sudo mkdir /opt/grisp_alloy_sdk you are basically doing exactly the same thing without having to add an extra depth... Am I missing something ?

@bwegh
Copy link
Author

bwegh commented Oct 2, 2025

what if you also use another grisp flavor?

@sylane
Copy link
Contributor

sylane commented Oct 3, 2025

@bwegh The flavor is already in the path: /opt/grisp_alloy_sdk/0.2.0/kontron-albl-imx8mm

@bwegh
Copy link
Author

bwegh commented Oct 6, 2025

@sylane yes, I agree, so if I will use a different flavor I need to manually create that folder again
/opt/grisp_alloy_sdk/0.2.0/kontron-albl-imx8mm or /opt/grisp_forge_sdk/0.1.0/kontron-albl-imx8mm both folders must be created and the user adjusted with /opt/grisp/... this is only needed once

@sylane
Copy link
Contributor

sylane commented Oct 6, 2025

@bwegh What is /opt/grisp_forge_sdk ? I think you only need to create /opt/grisp_alloy_sdkas you did /opt/grisp.

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.

3 participants