Bug 30916

Summary:	at desktop presentation, "openbox syntax error" popup
Product:	Mageia	Reporter:	Ben McMonagle <westel>
Component:	RPM Packages	Assignee:	Nicolas Salguero <nicolas.salguero>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	Normal	CC:	marja11
Version:	Cauldron
Target Milestone:	---
Hardware:	i586
OS:	Linux
Whiteboard:
Source RPM:	lxde-common-0.99.2-3.git20190224.7.mga9	CVE:
Status comment:
Attachments:	image of error

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