Skip to content

change sdk path to /opt/grisp - #6

Open
bwegh wants to merge 1 commit into
grisp:mainfrom
bwegh:bwegh/feature_sdk_path
Open

change sdk path to /opt/grisp#6
bwegh wants to merge 1 commit into
grisp:mainfrom
bwegh:bwegh/feature_sdk_path

Conversation

@bwegh

@bwegh bwegh commented Sep 24, 2025

Copy link
Copy Markdown

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

@ziopio

ziopio commented Sep 24, 2025

Copy link
Copy Markdown
Member

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

bwegh commented Sep 27, 2025

Copy link
Copy Markdown
Author

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

@sylane

sylane commented Sep 29, 2025

Copy link
Copy Markdown
Contributor

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

sylane commented Sep 30, 2025

Copy link
Copy Markdown
Contributor

@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

bwegh commented Oct 2, 2025

Copy link
Copy Markdown
Author

what if you also use another grisp flavor?

@sylane

sylane commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

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

@bwegh

bwegh commented Oct 6, 2025

Copy link
Copy Markdown
Author

@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

sylane commented Oct 6, 2025

Copy link
Copy Markdown
Contributor

@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