Bug 30916 - at desktop presentation, "openbox syntax error" popup
Summary: at desktop presentation, "openbox syntax error" popup
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: i586 Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Nicolas Salguero
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-09-30 10:13 CEST by Ben McMonagle
Modified: 2022-10-10 05:20 CEST (History)
1 user (show)

See Also:
Source RPM: lxde-common-0.99.2-3.git20190224.7.mga9
CVE:
Status comment:

Attachments
image of error (35.01 KB, image/png)
2022-09-30 10:17 CEST, Ben McMonagle 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Description Ben McMonagle 2022-09-30 10:13:57 CEST 
Description of problem: a new netinstall of LXDE. at desktop presentation, a popup notification; Openbox Syntax Error


Version-Release number of selected component (if applicable):


How reproducible: every time


Steps to Reproduce:
1.new net-install of LXDE desktop only
2.reboot to desktop
3.
Comment 1 Ben McMonagle 2022-09-30 10:17:24 CEST 
Created attachment 13405 [details]
image of error
Comment 2 Jani Välimaa 2022-09-30 18:24:06 CEST 
It's caused by latest change in multimedia-keys patch in lxde-common pkg.

https://svnweb.mageia.org/packages?view=revision&revision=1863847
Comment 3 Lewis Smith 2022-10-01 21:09:54 CEST 
Thanks for the screenshot, Ben.
Re previous comment:

https://svnweb.mageia.org/packages/cauldron/lxde-common/current/SOURCES/lxde-common-0.99.2-mga-config-multimedia-keys.patch?r1=1863847&r2=1863846&pathrev=1863847

 +      <action name='Execute'><command>sh -c "if [ -z \"$(pgrep lxmusic)\" ]; then (lxmusic &); fi; xmms2 toggle"</command></action>
Is it the unpaired " before 'if'?

CC: (none) => lewyssmith

Comment 4 Marja Van Waes 2022-10-03 15:18:09 CEST 
(In reply to Jani Välimaa from comment #2)
> It's caused by latest change in multimedia-keys patch in lxde-common pkg.
> 
> https://svnweb.mageia.org/packages?view=revision&revision=1863847

Thanks, Jani.

Assigning to the registered lxde-common maintainer.

CC: (none) => marja11
Assignee: bugsquad => nicolas.salguero
Source RPM: (none) => lxde-common-0.99.2-3.git20190224.7.mga9

Comment 5 Nicolas Salguero 2022-10-03 16:04:02 CEST 
Hi,

lxde-common-0.99.2-3.git20190224.8.mga9 will solve that issue, for new users at least.

For an existing user, at the line 237 of the file $HOME/.config/openbox/lxde-rc.xml, all " must be replaced by &quot; and & must be replaced by &amp;.

Best regards,

Nico.
Comment 6 Nicolas Salguero 2022-10-03 16:06:57 CEST 
Not at the line 237 but at the line 327 of the file.
Comment 7 Lewis Smith 2022-10-03 21:43:37 CEST 
(In reply to Lewis Smith from comment #3)
>  +      <action name='Execute'><command>sh -c "if [ -z \"$(pgrep lxmusic)\"
> ]; then (lxmusic &); fi; xmms2 toggle"</command></action>
> Is it the unpaired " before 'if'?
I see now it is not unpaired! I did look carefully - clearly not carefully enough.

CC: lewyssmith => (none)

Comment 8 Ben McMonagle 2022-10-10 05:20:32 CEST 
(In reply to Nicolas Salguero from comment #5)
> Hi,
> 
> lxde-common-0.99.2-3.git20190224.8.mga9 will solve that issue, for new users
> at least.

> 
> Best regards,
> 
> Nico.

thanks Nico.

new netinstall today does not exhibit issue.

marking as fixed

regards

Status: NEW => RESOLVED
Resolution: (none) => FIXED
Note You need to log in before you can comment on or make changes to this bug.