Skip to content
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

Fix usage Yasat no root #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

D3vil0p3r
Copy link

yasat.advice will be written in the HOME folder as occurs for all other files. In this way, the user can use Yasat also with no root privileges.

Furthermore, fixed some / typo.

yasat.advice will be written in the HOME folder as occurs for all other files. In this way, the user can use Yasat also with no root privileges.

Furthermore, fixed some `/` typo.
@D3vil0p3r D3vil0p3r marked this pull request as ready for review October 5, 2024 16:47
@D3vil0p3r
Copy link
Author

@montjoie can you give a quick look and merge it please? Then, can you create also a new release of it and upload on its SourceForge page or Release section of this GitHub repository?

Please, just be sure that man, plugins and tests directories have 755 permissions because in the latest old release they have 750 permission and can generate some priv errors.

@montjoie
Copy link
Owner

Hello
Thanks for your PR, I am happy to see YASAT has still users in 2024.

You generate yasat.advices in YASAT_HOME, but common will still seek it in YASAT_ROOT
In the mean time, I will add some CI to detect this, so you will need to rebase on top of master
Thanks

@D3vil0p3r
Copy link
Author

Thank you. I fixed in common as well.

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