Re: [patch] remove /system/storage settings and use new hal api

On Wed, 2006-03-01 at 20:08 -0500, David Zeuthen wrote:
> Hey,
> Attached patch does two things
>  1. It uses new hal API to figure out details about crypto between
>     crypted and clear volumes - this way we won't need to rely on
>     a magic hal property; this also saves a single roundtrip to the
>     hal daemon for every volume added - that's always good!
>  2. Removes the /system/storage settings - only the hal backend used
>     these and there was never any UI to modify these. In retrospect
>     these settings are kind of useless
> OK to commit? There are no string additions or changes; only removals.
> I've also tested this patch with a few different pieces of storage
> devices and it looks good to me.

Yes, please commit.

 Alexander Larsson                                            Red Hat, Inc 
                   alexl redhat com    alla lysator liu se 
He's a fiendish bohemian gangster in a wheelchair. She's a wealthy 
thirtysomething vampire with a knack for trouble. They fight crime! 

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]