FS#21466 - [pcmanfm] doesn't completely function in LXDE; segfaults

Attached to Project: Arch Linux
Opened by Jeremy M. (jskier) - Monday, 25 October 2010, 14:34 GMT
Last edited by Angel Velasquez (angvp) - Wednesday, 10 November 2010, 17:16 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Jürgen Hötzel (juergen)
Angel Velasquez (angvp)
Architecture All
Severity High
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 7
Private No

Details

Description:

Latest pcmanfm 0.9.8-1 package will load desktop fine but cannot open a file manager instance, a segfault occurs.
A forum poster mentioned that root does not have this problem (noted below), perhaps a permission issue?

Additional info:
* pcmanfm 0.9.8-1, libfm 0.1.14-1

Briefly discussed at the bottom of this forum posting as well:
https://bbs.archlinux.org/viewtopic.php?id=104409


Steps to reproduce:
Install latest pcmanfm package, and while in LXDE try to run a file manager instance of pcmanfm as a regular user.
This task depends upon

Closed by  Angel Velasquez (angvp)
Wednesday, 10 November 2010, 17:16 GMT
Reason for closing:  Upstream
Additional comments about closing:  Thanks for the patch, but let's wait until 0.9.9 release
Comment by Van Nguyen (kaizoku) - Tuesday, 26 October 2010, 08:02 GMT
I don't get a segfault, but I get pcmanfm: symbol lookup error: pcmanfm: undefined symbol: fm_path_new_for_str
Comment by Angel Velasquez (angvp) - Wednesday, 03 November 2010, 14:26 GMT
Try with the new release of pcmanfm but I don't had this issue with the 0.9.8-1
Comment by Jeremy M. (jskier) - Wednesday, 03 November 2010, 17:17 GMT
More details on the segfault after trying to open pcmanfm 3 times (thus 3 instances). This is with 0.9.8-2 as well.
   shot.jpg (29.4 KiB)
Comment by Jeremy M. (jskier) - Wednesday, 03 November 2010, 17:19 GMT
Also, not sure if it matters, but I'm running x86_64 on all the machines I use.
Comment by Matteo Bernardini (ponce) - Thursday, 04 November 2010, 05:41 GMT
FYI, I wrote a patch that should apply to latest git trying to revert two commits (they can't be simply reverted directly, code in git changed) related to the addition of a new IPC
http://pcmanfm.git.sourceforge.net/git/gitweb.cgi?p=pcmanfm/pcmanfm;a=commit;h=cc448d7fc14155927423373a91f5ccf1c2254557
http://pcmanfm.git.sourceforge.net/git/gitweb.cgi?p=pcmanfm/pcmanfm;a=commit;h=0c3f71f2360a9878198ecada1e12185de1738c4f

applying the attached patch fixes multiinstance here on slackware64-current

refeer to http://sourceforge.net/tracker/?func=detail&aid=3079666&group_id=156956&atid=801864
   revert-new-IPC.patch (12.3 KiB)
Comment by Richard Jackson (richjn) - Saturday, 06 November 2010, 00:58 GMT
Just to confirm the patch provided by ponce (thanks) restores all functionality.
I haven't spotted any regressions yet.
My system Arch x86_64 no Hal
Comment by Angel Velasquez (angvp) - Sunday, 07 November 2010, 15:47 GMT
Applied on [testing] please would you like to test the package before move it to [extra].

Thanks for the patch.
Comment by Nick (nicoseb) - Sunday, 07 November 2010, 16:25 GMT
Works like a charm!!!
I can finally keep my wallpaper and open multiple sessions of pcmanfm!
Thanks guys :)
Comment by Jeremy M. (jskier) - Sunday, 07 November 2010, 17:18 GMT
Testing works good on my end as well, no issues.
Comment by Nick (nicoseb) - Sunday, 07 November 2010, 17:30 GMT
Works like a charm!!!
I can finally keep my wallpaper and open multiple sessions of pcmanfm!
Thanks guys :)
Comment by Nick (nicoseb) - Sunday, 07 November 2010, 20:50 GMT
Comment by Richard Jackson (richjn) - Monday, 08 November 2010, 08:36 GMT
It looks as if the IPC issue has been fixed upstream.
Latest git (without patch) compiles and works well.
Master is now at 0.9.9
Just keeping us all on our toes I guess
Comment by Matteo Bernardini (ponce) - Monday, 08 November 2010, 18:40 GMT
good news, btw! just FYI, working fine (without patch) also here on slackware64-current :)

Loading...