Thanks for review!
None
https://github.com/fengsp/color-thief-py/blob/master/LICENSE
python-colorthief
https://github.com/fengsp/color-thief-py
No
Some screenshot links for AppData broken, 404. We need to update some of them. This prevent to update runtime on Flathub currently.
builddir/files/share/appdata/org.openrgb.OpenRGB.appdata.xml: FAILED:
• file-invalid : <screenshot> failed to load [https://gitlab.com/CalcProgrammer1/OpenRGB/-/wikis/uploads/e1d8d4603ecdd04f1acbcf6b2314fc66/OpenRGB_0.31_Device_View.PNG]https://buildbot.flathub.org/#/builders/6/builds/73052
Thanks for review. Maybe we could package just source code of opentyrian and document how to install data files? This question probably out of scope of this issue, but don't know where to discuss this.
Tyrian was released as Freeware in 2004.
From http://www.freewebs.com/worldtreegames/ :
Feel free to play it all you want and share it with friends, but for the moment I'd like to just have a central backup place (here) for the game. In the future I might not mind other sites sharing it, but I do enjoy amusing fans. :)
From http://www.camanis.net/ :
Tyrian has been released as freeware by Jason Emery.
These terms overide the one from the license.doc present in tyrian21.zip.
opentyrianData files, assets - no.
Artem (02d72bb4) at 15 Mar 20:34
Artem (02d72bb4) at 15 Mar 19:55
Fix building with GCC 13
I thought this could due uint8_t in ../Controllers/SteelSeriesController/SteelSeriesMouseController.cpp:58:5 perhaps?
In file included from ../Controllers/SteelSeriesController/SteelSeriesMouseController.cpp:10:
../Controllers/SteelSeriesController/SteelSeriesMouseController.h:40:17: error: 'uint8_t' was not declared in this scope
40 | std::vector<uint8_t> modes;
| ^~~~~~~
../Controllers/SteelSeriesController/SteelSeriesMouseController.h:13:1: note: 'uint8_t' is defined in header '<cstdint>'; did you forget to '#include <cstdint>'?
12 | #include "SteelSeriesGeneric.h"
+++ |+#include <cstdint>
13 |
../Controllers/SteelSeriesController/SteelSeriesMouseController.h:40:24: error: template argument 1 is invalid
40 | std::vector<uint8_t> modes;
| ^
../Controllers/SteelSeriesController/SteelSeriesMouseController.h:40:24: error: template argument 2 is invalid
../Controllers/SteelSeriesController/SteelSeriesMouseController.h:62:36: error: 'uint8_t' has not been declared
62 | virtual void SetLightEffectAll(uint8_t effect) = 0;
| ^~~~~~~
../Controllers/SteelSeriesController/SteelSeriesMouseController.cpp: In member function 'virtual void SteelSeriesMouseController::Save()':
../Controllers/SteelSeriesController/SteelSeriesMouseController.cpp:57:11: error: 'uint8_t' does not name a type
57 | const uint8_t SAVE_BUFFER_SIZE = 10;
| ^~~~~~~
../Controllers/SteelSeriesController/SteelSeriesMouseController.cpp:11:1: note: 'uint8_t' is defined in header '<cstdint>'; did you forget to '#include <cstdint>'?
10 | #include "SteelSeriesMouseController.h"
+++ |+#include <cstdint>
11 |
../Controllers/SteelSeriesController/SteelSeriesMouseController.cpp:58:5: error: 'uint8_t' was not declared in this scope
58 | uint8_t usb_buf[SAVE_BUFFER_SIZE] = { 0x00, 0x09 };
| ^~~~~~~
../Controllers/SteelSeriesController/SteelSeriesMouseController.cpp:58:5: note: 'uint8_t' is defined in header '<cstdint>'; did you forget to '#include <cstdint>'?
../Controllers/SteelSeriesController/SteelSeriesMouseController.cpp:60:20: error: 'usb_buf' was not declared in this scope
60 | hid_write(dev, usb_buf, SAVE_BUFFER_SIZE);
| ^~~~~~~
../Controllers/SteelSeriesController/SteelSeriesMouseController.cpp:60:29: error: 'SAVE_BUFFER_SIZE' was not declared in this scope
60 | hid_write(dev, usb_buf, SAVE_BUFFER_SIZE);
| ^~~~~~~~~~~~~~~~
make: *** [Makefile:8629: SteelSeriesMouseController.o] Error 1Missed types? Maybe similar to this fix in other project?
Sure, great change! New version with unbundled libs already built for all current Fedora branches. Need a proper testing though. Fedora users please welcome https://bodhi.fedoraproject.org/updates/?packages=corectrl
Added patch for F37+. Build available for testing. Thanks!
OpenWeather only says "Loading ..." without ever showing the weather in new upcoming GNOME v43. Works fine in GNOME v42.
118
Fedora 37 (Pre-release)
v43.RC
Thanks. Just now arrived my device which should supported by OpenRGB and i finally could test myself app and everything related to udev rules and such...
Folks who interesting in this app please help with testing flatpak build.
I'll rebase this ASAP. And sorry for some delay here - i am seeking for testers at this moment. We tried to test this and original AppImage build without this MR and for some reasons Thermaltake-Riing didn't found by app, but it mentioned in supported devices on the wiki. I'll also ask to test RPM version these folks. And also would be nice if people from original issue help with testing...
Artem (81b58705) at 23 Dec 03:50
Great, update submitted in Fedora.
@Dr_No i have some doubts here because didn't tested on other distros, but is should work. Also after some talks with Debian folks and they said Debian already have packages which installs udev rules in such location:
# apt-file search /usr/lib/udev/rules.d/
light: /usr/lib/udev/rules.d/90-backlight.rules
ltunify: /usr/lib/udev/rules.d/42-logitech-unify-permissions.rules
mobile-tweaks-common: /usr/lib/udev/rules.d/90-nm-gadget-managed.rules
util-linux: /usr/lib/udev/rules.d/85-hwclock.rules
xpra: /usr/lib/udev/rules.d/71-xpra-virtual-pointer.rulesAnd some background.