mpv-player / mpv

🎥 Command line video player
https://mpv.io
Other
28.39k stars 2.91k forks source link

AUR package #916

Closed qmega closed 10 years ago

qmega commented 10 years ago

This is mainly @divVerent, who maintains the mpv-git AUR package (I think).

I left a comment there a while ago requesting zsh completion be enabled, and there hasn't been any update or response. Just now there was another comment regarding an out of date dependency. (It's gotten pretty out of sync with the official one in general, though I'm not actually sure which one is more correct.) So I'm hoping I can get your attention here. (Sorry if this comes across as rude; that is not intended.)

Also, please install the completion to site-functions instead of vendor-completions. The latter is just a debian thing, AFAICT, and isn't in the default fpath on Arch. (Actually, site-functions might be a better default location in the first place, but I guess that's a different issue.)

Another minor thing: the source url should be changed to git+https, I think.

If you don't want to maintain the package anymore, I'd be willing to.

ricardomv commented 10 years ago

fixing the output of git describe would be a good improvement, the current output is libav_0.8.x-3824-g1e8b98a

divVerent commented 10 years ago

Am 08.07.2014 21:37 schrieb "Philip Sequeira" notifications@github.com:

This is mainly @divVerent, who maintains the mpv-git AUR package (I think).

I left a comment there a while ago requesting zsh completion be enabled, and there hasn't been any update or response. Just now there was another comment regarding an out of date dependency. (It's gotten pretty out of sync with the official one in general, though I'm not actually sure which one is more correct.) So I'm hoping I can get your attention here. (Sorry if this comes across as rude; that is not intended.)

Also, please install the completion to site-functions instead of vendor-completions. The latter is just a debian thing, AFAICT, and isn't in the default fpath on Arch. (Actually, site-functions might be a better default location in the first place, but I guess that's a different issue.)

Another minor thing: the source url should be changed to git+https, I think.

If you don't want to maintain the package anymore, I'd be willing to.

Sorry for not having the time for it recently - you're right, I probably should let you take over the package.

I'll disown it in a few minutes - feel free to take it. Previously I regularly merged with the binary package, but didn't get to it recently - whenever I got to maintaining the packages at all, it was ffmpeg-full that needed upgrading again.

Best regards,

Rudolf Polzer

qmega commented 10 years ago

Ok, I've adopted it and updated. I'll try and clean up the dependencies some more soon, but for now I think I've addressed the immediate issues, at least.

@ricardomv I'm not sure that's related to the AUR package... If you're saying that it should show a release number, I believe that doesn't happen because the release tags don't include all commits to master.