-
Notifications
You must be signed in to change notification settings - Fork 0
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
explain in the README differences to lwilletts/mpvc #3
Comments
I've tried to answer some of the questions above in the docs & wiki: Regarding the rest of questions, the nature of the fork is personal need for a working mpvc with the functionality described above. The name mpvc-tui is just the name of one of the disctintive tools found under the extras/ directory. As this is a personal project, the long term view of this repo is limited my own personal use. Anyway I've been maintaining this fork for a year, and It suits my needs, so I plan to keep using and maintaining this in the long term. Would adding a link to the above pages from at top of the README clarify the fork status? |
@m040601 Report back if the changes are ok, So I can close the issue. Thanks! |
Yes, thanks, the changes are OK. Issue can be closed. The needed information is there now. So package maintainers can decide better which version or fork to use, since you choose to keep the same name "mpvc". |
Thanks, closing the issue. |
If interested here's the latest version bump v1.4: https://github.com/lwilletts/mpvc/releases/tag/1.4 |
Changes have been merged to https://github.com/lwilletts/mpvc/ so the AUR mpvc-git now builds and installs. Is there any action to be performed to clear the "Out of Date" flag at https://aur.archlinux.org/packages/mpvc-git? Thanks. |
I've tested again today the PKGBUILD https://aur.archlinux.org/packages/mpvc-git. Seems to be working now. But it is still missing the README, docs and license. Read the comments there. Remember that PKGBUILD is still pulling from https://github.com/lwilletts/mpvc/ and not https://github.com/gmt4/mpvc/. So what you write or do on your (gmt4) README etc is not what the Archuser gets. Unless of course you merge your changes into lwillets repo. Upstream URL: https://github.com/lwilletts/mpvc
That is up to the AUR maintainer "lwillets" . If logged in to the AUR you should see a "button" or something to update and unmark it. Nothing much you can do here on github. Doesnt seem "lwillets" is paying much attention to the PKGBUILD though. You could eventually ask him to transfer it to you. If you are an Archlinux user yourself and willing to learn how to properly maintain it, https://wiki.archlinux.org/title/PKGBUILD And they you can decide which repo it pulls from (gmt4 or lwillets) etc. Or even best. Have 2 PKGBUILD's one "mpvc-git" for the rolling versions and one "mpvc" for the releases version. |
You made good points, I'll take some time to process them. (PS: regarding the out-of-date in mpvc-git package, it seems that only the one that did flag the package, can unflag it, at least I can't) |
I've reopened the issue, to address the issues related to the mpvc-git AUR package that can be fixed from here. I'm not the maintainer of the mpvc-git AUR package, nor I plan to be in the near future, so I can't correct the PKGBUILD. I've pushed lwilletts#63 that addresses the missing README and LICENSE in mpvc-git, I still have to think about how to ship docs/ in a proper way. Report back, if you see anything more missing, if it's all good, then I think only the docs/ is missing. Thanks. |
I've pushed lwilletts#64 that addresses the missing docs/mpv.conf and docs/logbook.html in mpvc-git. Thanks |
Hi, first of all thank you for your work and ongoing commitment to this very interesting tool.
I'm trying to figure out if I should use it, in Archlinux, instead of "lwillets/mpvc".
It seems to be more polished, maintained, adding extra functionality. But I'm not 100% sure of your future plans.
I found your repo through https://github.com/stax76/awesome-mpv.
Dont know why it is called "mpv-tui" there. Just confusing.
I'm on Archlinux, and we do have a PKGBUILD for "mpvc" . But it is currently pulling from "lwilletts/mpvc". Not your "gmt4/mpvc"
But because you also copied the README when you forked "lwillets/mpvc", you now have on your own "gmt4/mpvc" README,
And it is linking to, https://aur.archlinux.org/packages/mpvc-git . Which pulls from "lwilletts/mpvc". Not your "gmt4/mpvc"
So, because I was considering if the Archlinux PKGBUILD should switch to use yours instead of lwillets, I need more information.
Could you add one or two lines to the README explaining:
Of course everyone can diff download and compare your fork and the original repo to see the differences. But is much nicer if you just this briefly in the README.
Just a suggestion.Maybe if you plan to keep this long term, you should consider renaming it ? Choose your own distinct name ? Just in case lwillets starts updating theirs again ? Or maybe he closes his and contributes to this one ?
Thanks in advance.
The text was updated successfully, but these errors were encountered: