Bug 254209

#001: Requires should entail %{release} as well
Requires:        %{name} = %{version}-%{release}

#002: Please remove useless comments so as to remove confusions for others.

#003: as for

> #remove .la (kill the build on guile-gnome & gnucash not requires g-wrap 
> anymore)
> %exclude %{_libdir}/*.la

please replace %exclude %{_libdir}/*.la by:
%exclude %{_libdir}/libgwrap-core-runtime.la
%exclude %{_libdir}/libgwrap-guile-runtime.la

This might be very helpful for others reading the spec file.

#004 add the following comment on top of "make":
# Is not parallel-build-safe

#005: %setup -q -n %{name}-%{version}

you can remove  -n %{name}-%{version}

#006: The patches (%patch0 -p1)
Take as good example :
%patch0 -p1 -b .REASON

#007: (devel package) same for %{_libdir}/pkgconfig/*
/usr/lib/pkgconfig/ is required by pkgconfig

#008: as well for the (devel package) add the following Requires due to 
directory ownership:
ORBit2-devel, automake

#009 the directory /usr/share/guile/site/ is required by guile
Add guile as Requires:

#010:
You can replace all these
# remove .la files
rm -f %{buildroot}%{_libdir}/{*.a,libffi.la}
[..]
%{_libdir}/%{name}/modules/*.la
[..]
#remove .la (kill the build on guile-gnome & gnucash not requires g-wrap 
anymore)
%exclude %{_libdir}/*.la

 by:

find $RPM_BUILD_ROOT -name '*.la' -exec rm -f {} ';'

since both guile-gnome-platform and gwave build successfully with the *.la

(I've lost my comments on guile-gnome-platform during the shift to bugzilla's 
maintenance. I'll catch up later on) 

(In reply to comment #2)

> since both guile-gnome-platform and gwave build successfully with the *.la

TYPO: _WITHOUT_ the *.la

Ok, 
All issues have been fixed.

Spec : http://laxathom.fedorapeople.org/RPMS/g-wrap/g-wrap.spec
SRPM : http://laxathom.fedorapeople.org/RPMS/g-wrap/g-wrap-1.9.9-2.fc7.src.rpm

typo:
- Remoed all libtool archive.
missing "v"

I believe you can fix this before committing to cvs.

===============================

This package is approved by ME.

===============================

Please don't forget to close the bug #250792 once you have built g-wrap on koji.

New Package CVS Request
=======================
Package Name: g-wrap
Short Description: A tool for creating Scheme interfaces to C libraries
Owners: lxtnow
Branches: FC-6 F-7 EL-4 EL-5 Devel
InitialCC:

Well, some comments from me:

* Please check which package owns %{_libdir}/%{name}/modules/
  (currently no package)

* m4 files under %{_datadir}/aclocal/ usually should be in -devel
  package

* I don't know what software calls g-wrap modules, however usually
  modules are dlopen'ed with the name '*.so', not '*.so.X'.
  So I guess libgw-guile-*.so under %{_libdir}/%{name}/modules/
  should be in main package, not in -devel subpackage.

* Well, why should -devel subpackage have "Requires: ORBit2-devel"?

* And -devel package should have "Requires: guile-devel" because
  /usr/include/g-wrap/guile-compatibility.h (and one another header
  file) requires libguile.h

Please resubmit your CVS request with your FAS name instead of e-mail.

> * Please check which package owns %{_libdir}/%{name}/modules/
>  (currently no package)

hm...%{_libdir}/%{¬ame} is owned by current main package that imply its
subdirectory which's shipped & provided by main package to be owned to same one.


> * m4 files under %{_datadir}/aclocal/ usually should be in -devel
>  package

right, thanks for notice me this missing point.

> * I don't know what software calls g-wrap modules, however usually
>  modules are dlopen'ed with the name '*.so', not '*.so.X'.
>  So I guess libgw-guile-*.so under %{_libdir}/%{name}/modules/
>  should be in main package, not in -devel subpackage.

do you also check this:
$ file libgw-guile-*.so
libgw-guile-gw-glib.so:  symbolic link to `libgw-guile-gw-glib.so.0.0.0'
libgw-guile-standard.so: symbolic link to `libgw-guile-standard.so.0.0.0'


> * Well, why should -devel subpackage have "Requires: ORBit2-devel"?

it seem that we did some mixed up things between g-wrap & guile-gnome-platform
(which requires ORBit2-devel) cause, only guile-gnome-platform ships ORBit2's
stuffs.

(In reply to comment #9)
> > * Please check which package owns %{_libdir}/%{name}/modules/
> >  (currently no package)
> 
> hm...%{_libdir}/%{name} is owned by current main package
  Yes

> that imply its
> subdirectory which's shipped & provided by main package to be owned
> to same one.
  It may be that I don't understand correctly what you want to say, however
  if you write "%dir foo/", the subdirectories of foo/ are not _automatically_
  owned by any subpackages.


> > * I don't know what software calls g-wrap modules, however usually
> >  modules are dlopen'ed with the name '*.so', not '*.so.X'.
> >  So I guess libgw-guile-*.so under %{_libdir}/%{name}/modules/
> >  should be in main package, not in -devel subpackage.
> 
> do you also check this:
> $ file libgw-guile-*.so
> libgw-guile-gw-glib.so:  symbolic link to `libgw-guile-gw-glib.so.0.0.0'
> libgw-guile-standard.so: symbolic link to `libgw-guile-standard.so.0.0.0'
 Symlinks themselves are no problem, the problem is how these modules
 are dlopened (with what name).

Okay,
after checked the dlname, modules are dlopen'ed with '*.so.X'
dlname='libgw-guile-gw-glib.so.0'
dlname='libgw-guile-standard.so.0'

(In reply to comment #11)
> Okay,
> after checked the dlname, modules are dlopen'ed with '*.so.X'
> dlname='libgw-guile-gw-glib.so.0'
> dlname='libgw-guile-standard.so.0'

Would you tell me where these modules are dlopen'ed?

(In reply to comment #12)
> Would you tell me where these modules are dlopen'ed?

Mamoru, what do you mean? Is there something in your mind that you are trying to
point to us that I can't see ?
Please shed more light on this.

What I meant in the comment 12 is that I want to check
where Xavier found the actual code noted in comment 11.

re,
sorry for the delay on this post.
well, from lib-gw-*.La (where those libtool archives have been removed from the
package as well)
COuld you tell me if there something wrong with this ?

Well, now I assume that the dlopen'ed name is *.so.0..

Then would you fix the left issues I noted on the comment 7?

okay,

Spec : http://laxathom.fedorapeople.org/RPMS/g-wrap/g-wrap.spec
SRPM : http://laxathom.fedorapeople.org/RPMS/g-wrap/g-wrap-1.9.9-3.fc8.src.rpm

New Package CVS Request
=======================
Package Name: g-wrap
Short Description: A tool for creating Scheme interfaces to C libraries
Owners: laxathom
Branches: FC-6 F-7 EL-4 EL-5 Devel
InitialCC:

updated

(In reply to comment #17)
> okay,
> 
> Spec : http://laxathom.fedorapeople.org/RPMS/g-wrap/g-wrap.spec
> SRPM : http://laxathom.fedorapeople.org/RPMS/g-wrap/g-wrap-1.9.9-3.fc8.src.rpm
> 

looke okay.

The reason why guile-gnome-platform's mock is failing at your place is g-wrap 
hasn't yet been built for rawhide.

Please:
1. build g-wrap for rawhide
2. close the bug #250792
3. close this bug as well.

Okay, tag dist-F8 has been unblocked.

now, g-wrap has been imported and built (on working branch)