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.
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.
This decision is an effort to keep Trinity maintainable and stable, and continue with the philosophy of having dedicated system access libraries that then distribute information and control of system functions to higher level TDE applications.