> On Friday 09 September 2011 20:38:35 Timothy Pearson wrote: > >> >> In light of this, I would consider this patch a temporary workaround to >> a >> far more serious problem, > > Currently there are two mediamanager backends: the hal backend and the > fstab/mtab backend. > The patch only repairs and reenables the previously disabled parts of the > fstab/mtab backend? > and makes it compatible with modern environments. > > A proper solution maybe would require writing a third backend for dealing > with udisks. > I hope you'll share the third backend to mediamanager once you write it. Of course. Just look at all the new code I have added to Trinity over the past few years and you should be able to infer that when I write something for Trinity it gets released. :-) > >> and would reject the patch for inclusion in the >> upstream TDE sources. > >> In fact, as a matter of policy I am going to reject all piecemeal HAL >> workarounds unless they implement portions of the libraries (TUComputer, >> TUPower) that I have mentioned earlier. > > The patch only solves the problem of mediamanager's detection of new disk > devices. It does not deal with other hal functionality. Understood. Tim