search

PromyLOPh / pianobar

Console-based pandora.com player
http://6xq.net/pianobar/
Other
1.73k stars 323 forks source link

Feature request: --env for environment #728

Closed tyleradams closed 3 months ago

tyleradams commented 2 years ago

Can you add a --env flag which returns the environment info you need to debug pianobar?

PromyLOPh commented 2 years ago

What do you mean by “environment info”?

tyleradams commented 2 years ago

This part of the bug template image

PromyLOPh commented 2 years ago

I feel the template doesn’t ask for information impossible to obtain without an --env flag, does it?

tyleradams commented 2 years ago

If it's automatable, why not automate it? More people will do it (people are lazy) and there will be no mistakes and omissions.

Tyler

On Tue, Jan 4, 2022 at 12:41 PM Lars @.***> wrote:

I feel the template https://github.com/PromyLOPh/pianobar/blob/master/.github/issue_template.md doesn’t ask for information impossible to obtain without an --env flag, does it?

— Reply to this email directly, view it on GitHub https://github.com/PromyLOPh/pianobar/issues/728#issuecomment-1005033355, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABDXQYKVOD2RR7GAKDUSF7LUUMWOJANCNFSM5LGIBZ4Q . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

PromyLOPh commented 2 years ago

I’m not aware of any mechanism to reliably determine the distribution. Also pianobar currently does not have an argument parser, so that’d have to be added too. I don’t see the value in adding this feature right now.

minecraftchest1 commented 9 months ago

I’m not aware of any mechanism to reliably determine the distribution. Also pianobar currently does not have an argument parser, so that’d have to be added too. I don’t see the value in adding this feature right now.

Take a look at https://www.man7.org/linux/man-pages/man5/os-release.5.html