[PATCH] libvolume_id fixes for solaris

David Zeuthen david at fubar.dk
Tue Mar 7 08:41:45 PST 2006


On Tue, 2006-03-07 at 16:09 +0100, Kay Sievers wrote:
> > +#ifndef sun
> >  	id->type = "vfat";
> > +#else
> > +	id->type = "pcfs";
> > +#endif
> 
> > +#ifndef sun
> >  	id->type = "iso9660";
> > +#else
> > +	id->type = "hsfs";
> > +#endif
> 
> Hmm, the whole goal of HAL is to abstract that system specific information
> to applications. Are you really sure you want to do it that way and
> patch all the stuff in the upper stack too?

That's an interesting question.

>  (I'm sure no app will recognize
> an iPod formatted with "pcfs" :)) Anyway, you better do that "translation"
> in the Solaris code then, if you really want to do it. :)

Maybe the app is just broken if it looks for the file system name? It
should just look for volume.fsusage='filesystem' and rely on e.g.
gnome-mount etc. or whatever to do the right instead?

However, at least now gnome-mount currently has some hard coded default
mount options for what file system to use but that is subject to change
once it reads stuff from gconf, see

 http://mail.gnome.org/archives/utopia-list/2006-February/msg00000.html

for details.

I just wonder what will break if fs XYZ means something to Solaris and
something else to Linux. Then we get to have conditionals in
20-storage-methods.fdi file at least :-)

Maybe it would be better to just adopt the Linux file system names;
Artem, what do you think? Something to think about.

    David




More information about the hal mailing list