search

IgnorantGuru / spacefm

SpaceFM File Manager
http://ignorantguru.github.com/spacefm/
GNU General Public License v3.0
489 stars 73 forks source link

0.8.1 hal build fails #177

Closed mckaygerhard closed 12 years ago

mckaygerhard commented 12 years ago

In new work made by blackninja team are good in #146, but hal support (need by lenny and olders desktops with backported libs) are broken :

This its broken inclusive using recent squeeze/sid

Results of configuring in lenny or squeeze using hal:

SpaceFM...................................... : Version 0.8.1

Prefix....................................... : /usr
Executable................................... : /usr/bin/spacefm
Icon Dir..................................... : /usr/share/pixmaps/
Documentation Dir............................ : /usr/share/doc/spacefm/
Linux device support......................... : hal
Linux file change monitor support............ : inotify
64-bit Large file support.................... : yes
Desktop icon integration..................... : yes
Startup notification......................... : yes

results compiling in squeze/lenhny using hal:

est -f 'vfs/vfs-volume-hal.c' || echo './'`vfs/vfs-volume-hal.c
vfs/vfs-volume-hal.c: In function ‘vfs_volume_mount_by_udi_as_root’:
vfs/vfs-volume-hal.c:2888: warning: assignment discards qualifiers from pointer target type
vfs/vfs-volume-hal.c:2892: error: implicit declaration of function ‘vfs_sudo_cmd_sync’
make[1]: *** [spacefm-vfs-volume-hal.o] Error 1
make[1]: se sale del directorio `/home/intranet/devel/spacefm/spacefm/src'
make: *** [all-recursive] Error 1
mckaygerhard commented 12 years ago

seems due no set sudo value for prefered commen line at configure (i do not set any)

IgnorantGuru commented 12 years ago

It appears the deprecated hal support is broken in 0.8.1 - I likely removed a function required by hal.

IgnorantGuru commented 12 years ago

Until 0.8.2 is released, hal users can build from this modified tarball which should correct the hal build issues in 0.8.1. spacefm-hal in the PPA has been temporarily reverted to 0.7.5 (apparently the last version that worked with hal). Although hal support is deprecated (no longer actively developed), all SpaceFM releases should still build with hal, so users should continue to report issues if it doesn't.

mckaygerhard commented 12 years ago

status:

testing right now! give me few minutes..

mckaygerhard commented 12 years ago

F5 over paner side crash but its not related and i dont care about it, works for me..

some issues with umounting devices but its not relevant to spacefm due when are cmpiled with hal, udisk related apps mount devices and space dont able to umount.. its normal..

so then, closed