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

"Install log2ram" does not work from the menu #213

Closed
fcarella opened this issue Dec 24, 2020 · 6 comments
Closed

"Install log2ram" does not work from the menu #213

fcarella opened this issue Dec 24, 2020 · 6 comments

Comments

@fcarella
Copy link

line 46 in misc_commands.py is
subprocess.call("bash disable_swap.sh", shell=True)

doesnt look right, I think should be
subprocess.call("./scripts/disable_swap.sh ...", shell=True)

with some argument ..., not sure what though

@Slyke
Copy link
Collaborator

Slyke commented Dec 24, 2020

This is probably from your other bug #212 Give it a try now, and let me know if you still have troubles with it.

@fcarella
Copy link
Author

fcarella commented Dec 24, 2020 via email

@fcarella
Copy link
Author

Still doesnt work.

line 46 in misc_commands.py is
subprocess.call("bash disable_swap.sh", shell=True)

I still think the syntax is wrong but even so, if you look at "disable_swap.sh", theres nothing in there about log2ram.
The old IOTstack code does this (see line 410 @ https://github.com/gcgarner/IOTstack/blob/master/menu.sh):
...
git clone https://github.com/azlux/log2ram.git ~/log2ram
chmod +x ~/log2ram/install.sh
...

Hope this helps...

@Slyke
Copy link
Collaborator

Slyke commented Dec 25, 2020

Hey @fcarella , Looks like I forgot to actually commit the file and rename the command. Copy paste mistake! I just pushed another fix, and tested that begins to install it (but have not yet tested if install completes since I don't have access to a RPi right now).

@Slyke
Copy link
Collaborator

Slyke commented Dec 25, 2020

You can view the changes in this file: 123f35c, I'll close this issue, but please recomment if you continue to have issues with it.

@Slyke Slyke closed this as completed Dec 25, 2020
@fcarella
Copy link
Author

fcarella commented Dec 25, 2020 via email

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

No branches or pull requests

2 participants