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

couple bugs #154

Closed
fawaf opened this issue Aug 31, 2011 · 4 comments
Closed

couple bugs #154

fawaf opened this issue Aug 31, 2011 · 4 comments

Comments

@fawaf
Copy link
Contributor

fawaf commented Aug 31, 2011

  1. when you select a station after doing showing upcoming songs, the select stations menu also prints the upcoming songs and then the stations themselves.

  2. when you do a man of pianobar, the format_msg section shows:
    format_msg_none = %s
    format_msg_info = (i) %s format_msg_nowplaying = |> %s format_msg_time = # %s format_msg_err = /!\ %s format_msg_question = [?] %s format_msg_list = %s Message format
    strings. %s is replaced with the actual message.
    I assume there should be new lines or other formatting?

@PromyLOPh
Copy link
Owner

  1. when you select a station after doing showing upcoming songs, the
    select stations menu also prints the upcoming songs and then the
    stations themselves.
    Screenshot? I doubt the upcoming songs list is printed twice.

  2. when you do a man of pianobar, the format_msg section shows:
    format_msg_none = %s
    format_msg_info = (i) %s format_msg_nowplaying = |> %s format_msg_time = # %s format_msg_err = /!\ %s format_msg_question = [?] %s format_msg_list = %s Message format
    strings. %s is replaced with the actual message.
    I assume there should be new lines or other formatting?
    Yes, newlines. Works with “man, Version 1.6f”. Which version are you
    using?

@fawaf
Copy link
Contributor Author

fawaf commented Aug 31, 2011

  1. actually, my bad, there is just no separator between whatever was printed before and the stations list. I must have never noticed it before.

  2. yeah, i have version 1.6c

@PromyLOPh
Copy link
Owner

  1. yeah, i have version 1.6c
    Ah, I think I see the problem:
.TQ

This macro is not defined on legacy Unix systems running classic
troff.   To be  certain  your page will be portable to those
systems, copy its definition from the an-ext.tmac file of a groff
installation.

Can you try applying this patch: https://gist.github.com/1186340

@fawaf
Copy link
Contributor Author

fawaf commented Sep 3, 2011

yep, the patch works.

thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants