Page MenuHomeSolus

Update to 0.5.0.1 and fix dependencies
ClosedPublic

Authored by Jacalz on Feb 3 2019, 8:20 PM.
Tags
None
Referenced Files
F11071290: D5256.id12825.diff
Sat, Aug 12, 1:21 AM
F11071289: D5256.id12786.diff
Sat, Aug 12, 1:20 AM
F11071288: D5256.id12789.diff
Sat, Aug 12, 1:20 AM
F11071287: D5256.id.diff
Sat, Aug 12, 1:20 AM
F11071285: D5256.id12787.diff
Sat, Aug 12, 1:20 AM
F11071283: D5256.id12822.diff
Sat, Aug 12, 1:20 AM
F11071282: D5256.id12824.diff
Sat, Aug 12, 1:20 AM
F11053001: D5256.diff
Thu, Aug 10, 10:28 PM

Details

Summary

Summarized Changelog:

  • Initialize playtime attribute when invalid playtime found
  • Add application attribute on GenericPanel
  • Avoid crashing on weird GPU config
  • Remove get_config_id

Packaging:

  • Fix dependencies to make the program actually starts on Plasma and fix notifications

Fixes T7564

Test Plan

Diff Detail

Repository
R1992 lutris
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Jacalz requested review of this revision.Feb 3 2019, 8:20 PM
Jacalz updated this revision to Diff 12787.
  • Fix more strange shit produced by the build

The "weird shit" in pspec is due to the fact you didn't do +1 on the release number ?

package.yml
18–19

Since you're here, please also fix the order of the rundeps.

  • Fix some stuff because I need sleep
Jacalz edited the summary of this revision. (Show Details)

Ok, I managed to run tests. Well, I technically didn't as tests for lutris are supposed to be manual, nevertheless python3 setup.py test imported all the Python code and that helped me to figure out what lutris needs to work properly. Here are the rundeps:

- libgnome-desktop
- libgnome-keyring
- libwebkit-gtk
- notify-python
- python-evdev
- python-pillow
- python-requests
- python3-gobject

None of them is needed to build lutris, so the builddeps block can be removed.

Reordered runtime dependencies in alphabetical order

This should not be in the summary because the summary is meant for the end user, they don't care about packaging details.

Ah right, my test plan on Plasma:

  • lutris started successfully
  • Browsed in lutris windows, like preferences and advanced Steam settings
  • Launched the Steam client

You can look in the setup.py and eventually in requirements.txt (if it exists) to find out about dependecies

I know that @kyrios123, but they're not entirely listed. For example, libwebkit-gtk is completely missing and, oddly, pyyaml is not used, at least in tests.
We should probably take the time to manually run those tests as lutris devs intended.

JoshStrobl added a subscriber: JoshStrobl.

Let's get this updated to 0.5.0.1 while we're at it.

This revision now requires changes to proceed.Feb 4 2019, 11:58 AM
  • Add change rundeps, remove all builddeps and update 0.5.0.1
Jacalz retitled this revision from Add libwebkit-gtk as dependency to Update to 0.5.0.1 and fix dependencies.Feb 4 2019, 4:40 PM
Jacalz edited the summary of this revision. (Show Details)
Jacalz edited the test plan for this revision. (Show Details)
JoshStrobl added inline comments.
package.yml
30

Why was this removed? They have YAML functionality for configuration and their installer bits. You can easily see it in lutris/util/yaml.py, lutris/config.py, lutris/installer/interpreter.py, etc.

This revision now requires changes to proceed.Feb 4 2019, 5:13 PM
package.yml
30

It was removed by mistake because it wasn't on the test list from @livingsilver94, I see that it is needed and I will add it back ?

  • Add back pyyaml that I accidentally removed
This revision is now accepted and ready to land.Feb 4 2019, 5:19 PM

LGTM now. Thanks for the patch ?

You are welcome ? Also many thanks to @livingsilver94 for the testing ?

This revision was automatically updated to reflect the committed changes.

You're welcome too @Jacalz! But you didn't write my test plan, nor yours ?

Haha, you are sure right about that ? I did basically the same that you did but on MATE and without the Python tests ?