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

Confusing "How to use?" #10

Closed
BoostCookie opened this issue Oct 14, 2023 · 9 comments
Closed

Confusing "How to use?" #10

BoostCookie opened this issue Oct 14, 2023 · 9 comments

Comments

@BoostCookie
Copy link

I'm talking about the lines

3. `status_display_program.sh` for `cmus` should invoke the program.
```bash
cmus_status_scrobbler.py "$@" &
```

Why does it talk about a status_display_program.sh? Where is this shell script? In my opinion it should say something like

Tell cmus to use the scrobbler

:set status_display_program=path/to/cmus_status_scrobbler.py
@vjeranc
Copy link
Owner

vjeranc commented Oct 14, 2023

Usually people use cmus with a variety of display programs, one of them being cmus_status_scrobbler.py.

But yeah, setting cmus_status_scrobbler.py as your only status processing program is also possible.

@vjeranc
Copy link
Owner

vjeranc commented Oct 14, 2023

But yeah, I added this as the default option now in the README

@BoostCookie
Copy link
Author

Thanks. And also thanks for this nice script.

@BoostCookie
Copy link
Author

I've added it to the AUR, btw.
https://aur.archlinux.org/packages/cmus-status-scrobbler-git

@GUUser91
Copy link

I want to give a heads up for those using cmus-status-scrobbler in the termux app and have android 14.
This command below no longer works in android 14. Or at least it doesn't on OnePlus Pad.

:set status_display_program=path/to/cmus_status_scrobbler.py

So create a bash script called status_display_program.sh. Add these lines

#!/bin/sh

/data/data/com.termux/files/home/.config/cmus/cmus_status_scrobbler.py "$@" &

Execute both the .py and .sh files

chmod +rx status_display_program.sh

chmod +rx cmus_status_scrobbler.py

Invoke

:set status_display_program=/data/data/com.termux/files/home/.config/cmus/status_display_program.sh

in cmus

@vjeranc
Copy link
Owner

vjeranc commented Feb 29, 2024

@GUUser91 Thanks, I've added the Termux section under "How to use?".

@GUUser91
Copy link

I want to give another heads up. You need to change the location of the tmp folder in order for cmus-status-scrobbler to work in termux. Edit the cmus-status-scrobbler.py file with a text editor like micro and vim.

micro cmus_status_scrobbler.py

Change

/tmp/cmus_scrobbler.log

to

/data/data/com.termux/files/usr/tmp/cmus_scrobbler.log

@vjeranc
Copy link
Owner

vjeranc commented Feb 29, 2024

This last suggestion I should fix in cmus_status_scrobbler.py. I should use the environment variable TMPDIR if it exists.

@vjeranc
Copy link
Owner

vjeranc commented Feb 29, 2024

@GUUser91 I've fixed it here 2f33fe6 . It should work with termux now without that edit.

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

3 participants