add ncmpcpp pkg #547

Open
mitchweaver wants to merge 1 commits from unknown repository into master
mitchweaver commented 5 years ago (Migrated from github.com)
Owner

ncmpcpp a featureful ncurses based MPD client inspired by ncmpc.


Normally distros include the visualizer and clock in their package, and is one of the reasons people like ncmpcpp. So I added them in by default, with a comment block allowing people to edit it out if they don't want it.


Note on gnu-readline dep: it kept complaining about readline RL_TERMINAL_* messages, but after installing gnu-readline these errors went away. Maybe this dep isn't needed and can be patched, but I'm not that experienced with that sort of thing.


Note on libmpdclient dep: This is required to interface with mpd, see my other pull request for this pkg.

[ncmpcpp](https://github.com/arybczak/ncmpcpp) a featureful ncurses based MPD client inspired by ncmpc. -------- Normally distros include the visualizer and clock in their package, and is one of the reasons people like ncmpcpp. So I added them in by default, with a comment block allowing people to edit it out if they don't want it. ------- **Note on gnu-readline dep**: it kept complaining about readline RL_TERMINAL_* messages, but after installing gnu-readline these errors went away. Maybe this dep isn't needed and can be patched, but I'm not that experienced with that sort of thing. ------ **Note on libmpdclient dep:** This is required to interface with mpd, see my other pull request for this pkg.
rofl0r (Migrated from github.com) requested changes 5 years ago
rofl0r (Migrated from github.com) commented 5 years ago
Owner

automake is a host dep ( [deps.host] pls)

automake is a host dep ( [deps.host] pls)
rofl0r (Migrated from github.com) commented 5 years ago
Owner

does libmpdclient ship executables that are called at runtime, unlike its name suggests ? if so, this is the right section to put it. but if it is a library, which code is linked against at build time, it should be in [deps]

does libmpdclient ship executables that are called at runtime, unlike its name suggests ? if so, this is the right section to put it. but if it is a library, which code is linked against at build time, it should be in [deps]
rofl0r (Migrated from github.com) commented 5 years ago
Owner

please use a tarball directive too, so the tarball doesnt get saved as 324894038024.tar.gz, which nobody can figure out what it actually is

please use a tarball directive too, so the tarball doesnt get saved as 324894038024.tar.gz, which nobody can figure out what it actually is
# NOTE TO USERS: clock and visualizer are enabled by default
# on most distros, so they are to be expected. If you don't
# wish to use them, these flags can be deleted:
ADD_ONS="--enable-clock --enable-visualizer"
rofl0r (Migrated from github.com) commented 5 years ago
Owner

if you want to make this customizable, maybe you could just check if the environment var is already set, indicating that the user wants it changed, that way he doesnt need to edit the package itself

if you want to make this customizable, maybe you could just check if the environment var is already set, indicating that the user wants it changed, that way he doesnt need to edit the package itself
ADD_ONS="--enable-clock --enable-visualizer"
# --------------------------------------------------------
./autogen.sh
rofl0r (Migrated from github.com) commented 5 years ago
Owner

why do you run autogen.sh ? if it's possible, we rahter avoid that, since automake is a very heavy dependency. if it is because there's no configure script in the git checkout, maybe the author has tagged releases which include proper release tarballs with included configure script (related info: https://github.com/sabotage-linux/sabotage/wiki/Why-github-downloads-suck )

why do you run autogen.sh ? if it's possible, we rahter avoid that, since automake is a very heavy dependency. if it is because there's no configure script in the git checkout, maybe the author has tagged releases which include proper release tarballs with included configure script (related info: https://github.com/sabotage-linux/sabotage/wiki/Why-github-downloads-suck )
mitchweaver (Migrated from github.com) reviewed 5 years ago
ADD_ONS="--enable-clock --enable-visualizer"
# --------------------------------------------------------
./autogen.sh
mitchweaver (Migrated from github.com) commented 5 years ago
Owner

it does not ship with a configure script, and there isn't a "releases" page on his github.

The author has the configure script in his git-ignore. Looking back into commits of years ago you might be able to find a script but at that point its too old anyway.

Let me know what you want to do, and sorry if I misunderstood your comment

it does not ship with a configure script, and there isn't a "releases" page on his github. The author has the configure script in his git-ignore. Looking back into commits of years ago you might be able to find a script but at that point its too old anyway. Let me know what you want to do, and sorry if I misunderstood your comment
rofl0r (Migrated from github.com) reviewed 5 years ago
ADD_ONS="--enable-clock --enable-visualizer"
# --------------------------------------------------------
./autogen.sh
rofl0r (Migrated from github.com) commented 5 years ago
Owner

that's how i do it: i raise awareness of the issue https://github.com/arybczak/ncmpcpp/issues/263

is there a specific reason you want to have latest master, over the latest tagged release from 2 weeks ago ? as i already mentioned, releases are usually preferable over bleeding egde master downloads.

that's how i do it: i raise awareness of the issue https://github.com/arybczak/ncmpcpp/issues/263 is there a specific reason you want to have latest master, over the latest tagged release from 2 weeks ago ? as i already mentioned, releases are usually preferable over bleeding egde master downloads.
rofl0r (Migrated from github.com) reviewed 5 years ago
ADD_ONS="--enable-clock --enable-visualizer"
# --------------------------------------------------------
./autogen.sh
rofl0r (Migrated from github.com) commented 5 years ago
Owner

his releases page is here btw https://github.com/arybczak/ncmpcpp/releases
(click on code, then the releases button is visible)

his releases page is here btw https://github.com/arybczak/ncmpcpp/releases (click on code, then the releases button is visible)
rofl0r commented 5 years ago (Migrated from github.com)
Owner

the author pointed out that he has proper release tarballs available: by using this one http://rybczak.net/ncmpcpp/stable/ncmpcpp-0.8.1.tar.bz2 we dont need the automake dependency

the author pointed out that he has proper release tarballs available: by using this one http://rybczak.net/ncmpcpp/stable/ncmpcpp-0.8.1.tar.bz2 we dont need the automake dependency

Reviewers

This pull request is broken due to missing fork information.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sabotage-linux/sabotage#547
Loading…
There is no content yet.