search

dietervanhoof / polybar-spotify-controls

MIT License
87 stars 8 forks source link

Move IPC logic to polybar-msg #13

Closed nileshp87 closed 5 years ago

nileshp87 commented 5 years ago

Quick fix to move the messaging from the /tmp/bottombar fifo into the new polybar-msg. Makes installation easier, and should fix the multimonitor issues.

nileshp87 commented 5 years ago

Should fix #5 and #6

dietervanhoof commented 5 years ago

Thanks for contributing. I won't easily be able to test this as I won't have access to my Linux machine. Can I assume this works on your system?

nileshp87 commented 5 years ago

I can confirm it works on my system. I didn't have my second monitor to confirm that. But, nothing should break.

On Sun, Sep 1, 2019, 10:31 PM Dieter notifications@github.com wrote:

Thanks for contributing. I won't easily be able to test this as I won't have access to my Linux machine. Can I assume this works on your system?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/dietervanhoof/polybar-spotify-controls/pull/13?email_source=notifications&email_token=AADHMPIMSAGDZDD6F6XU7RLQHSQLBA5CNFSM4ISR67YKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5UX3IA#issuecomment-527007136, or mute the thread https://github.com/notifications/unsubscribe-auth/AADHMPM7OVHVRLIHDMSAIW3QHSQLBANCNFSM4ISR67YA .

dietervanhoof commented 5 years ago

Sweet. Thanks again! Merging this now.