search

lwilletts / mpvc

An mpc-like control interface for mpv.
MIT License
139 stars 18 forks source link

Fix $SEQCOMMAND #33

Closed woorst closed 7 years ago

woorst commented 7 years ago

seq "$REPS" "$FIRST" "$LAST" doesn't work even if $REPS variable is unset

heckendorfc commented 7 years ago

Thanks for catching this but there are a lot of unrelated changes in the commit stream.

I think the real problem is a regression introduced in daa53ba4a413101c54700c6fbb878700597400d4. $REPS should not have been quoted on line 216. Could you make a fresh branch on mpvc's master with a commit just removing the quotes around that variable (also confirm that the change I'm suggesting fixes the issue for you)?