Skip to content

Fix premissions#1

Open
roybarel8 wants to merge 7 commits intomainfrom
fix-premissions
Open

Fix premissions#1
roybarel8 wants to merge 7 commits intomainfrom
fix-premissions

Conversation

@roybarel8
Copy link
Collaborator

@roybarel8 roybarel8 commented Jun 19, 2025

Sh Script for permission denied issue.
There were minor changes, because after I ran the script on my computer I saw that it couldn't log in because a password was required in SSH.
This has been fixed and now after running it I enter the password for Gary and it works.

Stages:

  1. SSHs into GARYXXX_VPN

  2. Goes into ~/dev_workspaces

  3. Changes permission of roy/ to gary:gary

  4. Asks for gary’s password via sudo

  5. Prints success message

@roybarel8 roybarel8 requested a review from IdanCohh June 19, 2025 14:00
@roybarel8 roybarel8 self-assigned this Jun 19, 2025
@roybarel8
Copy link
Collaborator Author

  • Added default to the USER_FOLDER.

Copy link

@IdanCohh IdanCohh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's no reason to let the user know that there is another option that is not devel.
In this repository, all users should be under devel as Camilo defined.
The other option should not exist for our public users.
If you want to keep it, I prefer you keep it silent by not letting the user know the option exists (thus only our developers will know).

@roybarel8
Copy link
Collaborator Author

  • The USER_FOLDER is always devel .

Copy link

@IdanCohh IdanCohh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

To test it, you can create that folder on the robot (devel)
Then, you can try to run it.
After you see all the expected prints from you script, ssh to the robot and run this command inside it:
ls -ld ~/dev_workspaces/devel
The expected output should be drwxrwxr-x 5 gary gary 4096 <date> /home/gary/dev_workspaces/devel
The most important there is the gary gary, for owner/group permissions definition

@roybarel8
Copy link
Collaborator Author

I did it and i get the expected output with gary gary.
You can see i did an update also its just in the massage the user is getting.

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.

2 participants