Description of problem: Impossible to start it, When clicking to the icon in systray, no way to open the bookmarks. Version-Release number of selected component (if applicable): radiotray-ng.0.2.5-1.mga7.rpm How reproducible: Install it Start it in a terminal . Clic on the icon in systray, Preferences/Bookmarks editor Then you get this message in console : "[2020-11-29 08:54:55.485976] [0x00007f01d5951e40] [error] (bookmarks.cpp:43) - Failed to parse: /~/.config/radiotray-ng/bookmarks.json : * Line 238, Column 4 Missing ',' or ']' in array declaration A " ' " is missing after " } " in line 246, col 5 just above the group "France" in the file bookmarks.json
Created attachment 12028 [details] file ~/.config/radiotray-ng/bookmarks.json
I mean a comma is missing
Hello, It affect also Cauldron version.
Please test radiotray-ng-0.2.5-1.1.mga7 in core/updates_testing. You might need to remove old bookmarks file ~/.config/radiotray-ng/bookmarks.json first.
CC: (none) => jani.valimaaAssignee: bugsquad => qa-bugs
Hello, I installed the testing package, but i had to rename ~/.config/radiotray-ng/ folder for radiotray-ng to regenerate the bookmarks.json. So i suppose it will work on a fresh profile, but not on an already installed/launched one.
Hello radiotray-ng-0.2.5-1.1.mga7 tested and works fine but it is necessary to delete the whole folder ~/.config/radiotray-ng before installing, not only bookmarks.json Thanks
(In reply to Daniel BEZIVIN from comment #6) > Hello > radiotray-ng-0.2.5-1.1.mga7 tested and works fine but it is necessary to > delete the whole folder ~/.config/radiotray-ng before installing, not only > bookmarks.json > Thanks Based on this as I had never installed this before, Suggested advisory: ======================== Updated radiotray-ng package fixes bugs The updated package fixes bugs in bookmarks file. Users are advised that they must delete ~./config/radiotray-ng folder before updating. ======================== Updated packages in core/updates_testing: ======================== radiotray-ng-0.2.5-1.1.mga7 from SRPM: radiotray-ng-0.2.5-1.1.mga7.src.rpm
CC: (none) => ouaurelien
@David Hodgins, Does this adv is satisfying? Does the fix seems OK?
CC: (none) => davidwhodgins
Not in my opinion. Most people never read advisories, especially before installing updates. I'm assuming it's safe to delete the directory ~./config/radiotray-ng after installing, as long as radiotray-ng is not running. I think the package should be modified to include a README.urpmi warning users that old versions of that directory must be deleted with radiotray-ng not running.
Keywords: (none) => feedback
I can add README.update.urpmi if someone tells what to write to it.
(In reply to Jani Välimaa from comment #10) > I can add README.update.urpmi if someone tells what to write to it. Proposed README.update.urpmi radiotray-ng package was updated. Old version of files in that directory must be removed. We strongly advice you to make a backup of ~./config/radiotray-ng directory and remove it. As user in a Terminal, please do: "mv ~/.config/radiotray-ng/ ~/.config/radiotray-ng.old/" Normally, according to our policies, we do not want to remove/rename/touch files in user directory, but sometimes, some config files can be problematic and make software to have some misbehaviour.
Ping?
Re ping. We should fix this. @Packager can you take a look?