On Wed, 2008-09-17 at 17:05 +0800, Rick Ju wrote:
> In your previous mail, you said, "?gnome-vfs is still needed".
> 
> By that, I meant its headers files. There is no 64bit gnome-vfs so no 
> chance libgsf 64bit could depend on that.

So it is BuildRequires SUNWgnome-vfs-devel, not Requires SUNWgnome-vfs.

But is still looks strange that you depend on a header file but not
their libs.

Harry

> 
> -rick
> 
> Harry Lu wrote:
> > On Wed, 2008-09-17 at 16:14 +0800, Rick Ju wrote:
> >   
> >> No, this option will not bring into libgnomevfs library.
> >>     
> >
> > In your previous mail, you said, "?gnome-vfs is still needed".
> >
> > So does SUNWlibgsf depends on SUNWgnome-vfs or not?
> >
> > If still depends, your patch is useless before SUNWgnome-vfs have 64 bit
> > libraries.
> >
> > If not depend, you should remove SUNWgnome-vfs from SUNWlibgsf's
> > requirement.
> >
> >   
> >> It is just how libgsf configure works. If there is only 
> >> "with-gnome-vfs", libgsf depends on gnome-vfs. If "with-gnome-vfs + 
> >> with-gio", libgsf depends on gio only.
> >>     
> >
> > What is the result of "without-gnome-vfs + with-gio"?
> >
> >
> > Thanks,
> > Harry
> >
> >
> >   
> >> -rick
> >>
> >> Ghee Teo wrote:
> >>     
> >>> Rick Ju wrote:
> >>>       
> >>>> No, it should be --with-gnome-vfs + --with-gio.
> >>>>
> >>>> gnome-vfs is still needed
> >>>>         
> >>> Can you elaborate on this? Does this option brought in libgnomevfs-2.so?
> >>> How is libgnomevfs-2.so required for use?
> >>> I thought gio would have provided at least most of gnomevfs functions.
> >>> If it has not, gvfs should.
> >>> I think we should avoid extending 64-bit libgnomevfs-2 for now:
> >>> - it is obsoleted no going to get on-going maintainance
> >>> - it is very close to FCS build for a large libraries
> >>>
> >>> -Ghee
> >>>
> >>>       
> >>>> Harry Lu wrote:
> >>>>         
> >>>>> On Wed, 2008-09-17 at 15:06 +0800, Rick Ju wrote:
> >>>>>  
> >>>>>           
> >>>>>> +./configure --prefix=%{_prefix}            \
> >>>>>> +      --bindir=%{_bindir}                  \
> >>>>>> +      --libdir=%{_libdir}                  \
> >>>>>> +      --includedir=%{_includedir}          \
> >>>>>> +      --sysconfdir=%{_sysconfdir}          \
> >>>>>> +      --mandir=%{_mandir}                  \
> >>>>>> +      --with-gnome                         \
> >>>>>> +      --with-bz2                           \
> >>>>>> +      --with-gnome-vfs                     \
> >>>>>>     
> >>>>>>             
> >>>>> Should not the above be --without-gnome-vfs?
> >>>>>
> >>>>>  
> >>>>>           
> >>>>>> +      --with-gio                           \
> >>>>>> +      %gtk_doc_option
> >>>>>>     
> >>>>>>             
> >>>>> And with the patch, I assume SUNWlibgsf doesn't depend on SUNWgnome-vfs
> >>>>> anymore.  So I think you should remove
> >>>>>
> >>>>> Requires: SUNWgnome-vfs
> >>>>> and BuildRequires: SUNWgnome-vfs-devel
> >>>>>
> >>>>> from SUNWlibgsf.spec.
> >>>>>
> >>>>>
> >>>>> Thanks,
> >>>>> Harry
> >>>>>
> >>>>>
> >>>>>   
> >>>>>           
> 
-- 
This is a test signature
?This is a test signature


Reply via email to