-
Notifications
You must be signed in to change notification settings - Fork 204
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
Include sd, fhd, and uhd in picon.py for Release 9.0 #4053
base: release-9.0
Are you sure you want to change the base?
Conversation
Fallback trimming of sd, 4k, uhd and fhd from SNPicon names As suggested by @kiddac amd @AbuBaniaz but change via a discussion on the forum https://forums.openpli.org/topic/16424-some-questions-bugs/page-321 from post 6408
@littlesat ok to merge? |
I don't get it. The is already a proper fix in the develop branch. If needed in 9.0 branch that should be picked. |
I added Littlesat's two commits as they resolve issue 3830. Your commit which is after Littlesat's is about unicode names and not explicitly to do with issue 3830. I'm sure your commit will be added later on. Now we start the discussion about backports are for bugfixes and not new features. The same reason why the steamrelay code still isn't in 9.0. |
And removes the endless "if name" horrible code. |
I think it is OK to Merge... but somehow I get a message Merging is Blocked... In the commit there is also a typo Releae... we miss an S there ;) |
If develop has new features that should be in a release image, it is an indication that a new release is overdue... |
Some picons with new 4k and uhd names don't show... Not too important... Stream relay stuff really is an important missing feature though. |
The issue is no new release!!! |
Appearently if I don't take action, nobody does. I tried twice, zero response or feedback. |
I cannot merge in Release... please add a new merge request for Develop. Then we can 'port' it back to Release. That is how it should be done. For Release I get "merging is blocked".... |
This is a cherry pick of your two commits already in develop. |
I don't understand wasting time on this. It is not a bug fix, but a new feature in a later version. People can either wait for Release 9.1, or switch to Develop, which is undated daily. And even with the 2 commits the code still won't be in sync with the new version. |
An issue was raised to add a feature for users who are mostly on release. The feature was added in two commits by Littlesat into develop image. The issue was closed a long time after citing it had been added ( yes It is in develop). This pull request is to add the two commits by Littlesat into release image. Wanwizard asked Littlesat if it is okay to merge them, Littlesat has finally agreed after a long time. But it seems he can't merge his own commits into release I hope the PLi team can see through the spiteful/vindictiveness behind the objection. |
It is not an objection. It is an observation. Resources are limited so we should be concentrating on the future. |
As far I can see this is not a cherry pick from develop but a kind of stand allone merge request on release. And indeed it is also a feature improvement and not a fix for a bug. The issue is more people who want to stuck on release and want this improvement… |
It is a merge request into Release 9.0 of two cherry picks of following commits in Developer image: The commit hashes if you want to cherry pick them yourself are I branched of 9.0, cherry picked those two commits and submitted a pull request when this issue #3830 was closed You can clearly see those two commits in the branch from which the pull request was sent. https://github.com/AbuBaniaz/enigma2_openpli/commits/releae-9.0_picon/ |
I have no objections, only accounts with write access to a repo can do direct cherry-picks. The only downside of doing it with a merge is that you may loose original commit information, so it might be better if someone with the correct access cherry picks these two commits. |
Like I already said it is not just 2 commits. |
I count two commits. Which other commits are there to fulfill Kiddac's request? |
Instead of trolling, why don't you just say,
Your commit was 4 months after Littlesat's and is nothing to do with the change request by Kiddac. It makes code better. You should submit it to ATV too. |
Why don't you just commit what is there? |
Actually don't bother because it is not going to run on a python 2 image. |
No description provided.