Bug 27681 - radiotray-ng is not fonctionnal
Summary: radiotray-ng is not fonctionnal
Status: NEW
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 7
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard:
Keywords: feedback
Depends on:
Blocks:
 
Reported: 2020-11-29 09:03 CET by Daniel BEZIVIN
Modified: 2021-02-19 10:34 CET (History)
3 users (show)

See Also:
Source RPM: radiotray-ng.0.2.5-1.mga7.rpm
CVE:
Status comment:


Attachments
file ~/.config/radiotray-ng/bookmarks.json (7.66 KB, application/json)
2020-11-29 09:05 CET, Daniel BEZIVIN
Details

Description Daniel BEZIVIN 2020-11-29 09:03:22 CET
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
Comment 1 Daniel BEZIVIN 2020-11-29 09:05:05 CET
Created attachment 12028 [details]
file ~/.config/radiotray-ng/bookmarks.json
Comment 2 Daniel BEZIVIN 2020-11-29 09:06:44 CET
I mean a comma is missing
Comment 3 Nicolas Nicolas 2020-11-29 10:18:57 CET
Hello,

It affect also Cauldron version.
Comment 4 Jani Välimaa 2020-11-29 10:50:27 CET
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.
Jani Välimaa 2020-11-29 10:51:06 CET

CC: (none) => jani.valimaa
Assignee: bugsquad => qa-bugs

Comment 5 Nicolas Nicolas 2020-11-29 14:23:45 CET
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.
Comment 6 Daniel BEZIVIN 2020-11-29 19:18:21 CET
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
Comment 7 Aurelien Oudelet 2020-12-01 09:34:46 CET
(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

Comment 8 Aurelien Oudelet 2020-12-07 10:56:16 CET
@David Hodgins,
Does this adv is satisfying? Does the fix seems OK?

CC: (none) => davidwhodgins

Comment 9 Dave Hodgins 2020-12-07 23:14:10 CET
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.
Dave Hodgins 2020-12-10 21:29:26 CET

Keywords: (none) => feedback

Comment 10 Jani Välimaa 2020-12-13 19:01:04 CET
I can add README.update.urpmi if someone tells what to write to it.
Comment 11 Aurelien Oudelet 2020-12-14 09:03:58 CET
(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.
Comment 12 Aurelien Oudelet 2021-02-04 18:34:39 CET
Ping?
Comment 13 Aurelien Oudelet 2021-02-19 10:34:40 CET
Re ping. We should fix this.
@Packager can you take a look?

Note You need to log in before you can comment on or make changes to this bug.