LibreELEC.tv icon indicating copy to clipboard operation
LibreELEC.tv copied to clipboard

chrome: improvements to the python launch script

Open pohutukawa opened this issue 3 years ago • 7 comments

chrome: improvements to the python launch script

  • improved exception reporting to the logger
  • improved subprocess handling to be more pythonic
  • fixed compatibility with configuration for custom audio devices (ALSA) to be compatible with previous custom settings
  • fixed launchability of Chrome in presence of custom audio device config (that were working previously)
  • slight simplifications for more pythonic code in some functions

pohutukawa avatar Dec 16 '21 02:12 pohutukawa

Also created a PR for the backport to the libreelec-10.0 branch in PR #6009

pohutukawa avatar Dec 23 '21 03:12 pohutukawa

Sorry for the delay, but I've been on a camping holiday and recently got back. It's hot and summery here ... :-)

I've incorporated the suggested change from above.

pohutukawa avatar Jan 26 '22 08:01 pohutukawa

Thanks. Unfortunately this PR needs a rebase to current master and revision 112 already exist.

Edit: please squash your commits.

mglae avatar Jan 26 '22 22:01 mglae

What package version should it be, then? 112 or 113? (As the libreelec-10.0 doesn't have a 112, yet, so my worry is that they won't be in sync). Dang, I wish I had gotten this sorted out before the holiday :-)

pohutukawa avatar Jan 26 '22 23:01 pohutukawa

The master and libreelec-10.0 branches are not in-sync to start with (each contains different overall content) so there is no need for the PRs to be in-sync. Just update the master branch PR to the next available number.

chewitt avatar Jan 26 '22 23:01 chewitt

done. I just don't know how/what those two conflicts above are coming from, as I guess they should be part of the PR on the change. Sorry for being such a nuisance ... :-(

pohutukawa avatar Jan 27 '22 01:01 pohutukawa

You have directly edited and pushed the master branch. It cannot updated any more following the usual workflow. Fortunately the local and remote branch name can be different.

To clean up your local master branch environment use (untested):

# Create a backup. Just in case.
git branch -c master le11_chrome_backup_220127

# Create the new working branch
git branch -c master le11_chrome_improvements

# Reset and update local master branch to LE master
git checkout master
git fetch upstream
git reset --hard upstream/master

# Rebase working branch
git checkout le11_chrome_improvements
git rebase -i master

# When all done force push the working branch to the existing PR
git push -f origin le11_chrome_improvements:master

Similar has to be done for the backport but let's start with master.

mglae avatar Jan 27 '22 16:01 mglae

Work incorporated into

  • #7101
  • #7310

heitbaum avatar Jan 12 '23 19:01 heitbaum