Bug#593195: udisks detect chrooted patch for debian package.

2010-08-30 Thread Andreas Henriksson
On mån, 2010-08-30 at 19:05 +0200, Martin Pitt wrote: > Michael Biebl [2010-08-30 12:14 +0200]: > > Martin, what do you think? > > I like the D-BUS query approach. It avoids doing extra stuff at each > startup with PID files (which would involve writing to the disk), and > also avoids having to cl

Bug#593195: udisks detect chrooted patch for debian package.

2010-08-30 Thread Martin Pitt
Michael Biebl [2010-08-30 12:14 +0200]: > Martin, what do you think? I like the D-BUS query approach. It avoids doing extra stuff at each startup with PID files (which would involve writing to the disk), and also avoids having to clean up stale pid files, etc. Martin -- Martin Pitt

Bug#593195: udisks detect chrooted patch for debian package.

2010-08-30 Thread Andreas Henriksson
On Mon, Aug 30, 2010 at 12:14:02PM +0200, Michael Biebl wrote: [...] > Given this, patching upower/udisks to write a pid file is probably the > cleanest > but also most invasive solution. I fail to see why it has to be invasive. Please point out why a simple hack like the one attached won't work.

Bug#593195: [Pkg-utopia-maintainers] Bug#593195: udisks detect chrooted patch for debian package.

2010-08-30 Thread Julien Cristau
On Mon, Aug 30, 2010 at 13:10:01 +0200, Michael Biebl wrote: > if [ -f /var/run/dbus/system_bus_socket ] ; then > pid=$(dbus-send --system --dest=org.freedesktop.DBus --print-reply \ >/org/freedesktop/DBus org.freedesktop.DBus.GetConnectionUnixProcessID \ >string:'org.freedesktop.UDisks'

Bug#593195: [Pkg-utopia-maintainers] Bug#593195: udisks detect chrooted patch for debian package.

2010-08-30 Thread Michael Biebl
On 30.08.2010 13:10, Michael Biebl wrote: > if [ -f /var/run/dbus/system_bus_socket ] ; then That better be if [ -S /var/run/dbus/system_bus_socket ] ; then Michael -- Why is it that all of the instruments seeking intelligent life in the universe are pointed away from Earth? signature.asc

Bug#593195: [Pkg-utopia-maintainers] Bug#593195: udisks detect chrooted patch for debian package.

2010-08-30 Thread Michael Biebl
On 30.08.2010 12:14, Michael Biebl wrote: > Another idea, I'd prefer over your proposed patch, is to test if the D-Bus > names > for org.freedesktop.UDisks resp. org.freedesktop.UPower are taken (which > should > not be the case in a chroot) and skip the restart if not. So, s-s-d --exec *without

Bug#593195: udisks detect chrooted patch for debian package.

2010-08-30 Thread Michael Biebl
clone 593195 -1 reassign -1 upower retitle -1 installing upower in a chroot kills upowerd in the host thanks On 30.08.2010 11:52, Andreas Henriksson wrote: > Hello! > > I've created a (completely untested) patch with the solution > you seemed to have agreed on (see attachment). > > It would be a

Bug#593195: udisks detect chrooted patch for debian package.

2010-08-30 Thread Andreas Henriksson
Hello! I've created a (completely untested) patch with the solution you seemed to have agreed on (see attachment). It would be appreciated if someone was willing to test the patch. If it happens to solve the problem, it would be nice to know if maintainers would appreciate a NMU or not. -- Rega