> 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