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