Bug 254211

Summary: Review Request: guile-gnome-platform - Guile wrapper collection for the GNOME library stack
Product: [Fedora] Fedora Reporter: Xavier Lamien <lxtnow>
Component: Package ReviewAssignee: Chitlesh GOORAH <chitlesh>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: chitlesh, fedora-package-review, kevin, mtasaka, notting
Target Milestone: ---Flags: chitlesh: fedora-review+
kevin: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-10-03 22:40:31 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 226223, 254209    
Bug Blocks: 256481    

Description Xavier Lamien 2007-08-24 19:29:28 UTC
Spec : http://laxathom.fedorapeople.org/RPMS/guile-gnome-platform/guile-gnome-platform.spec
SRPM : http://laxathom.fedorapeople.org/RPMS/guile-gnome-platform/guile-gnome-platform-2.15.93-2.fc8.src.rpm

Description: 
The guile-gnome project brings the power of Scheme to your graphical
application. guile-gnome modules support the entire GNOME library
stack: from Pango to GnomeCanvas, GTK+ to GStreamer, Glade to
GtkSourceView, you will find in guile-gnome a comprehensive
environment for developing modern applications.

Comment 1 Chitlesh GOORAH 2007-08-26 23:52:58 UTC
#001: guile-cairo-devel already requires guile-devel
[build@goorah ~]$ rpm -qR guile-cairo-devel
cairo-devel
guile-cairo = 1.4.0-4.fc7
guile-devel

Remove guile-devel from the BR

#002: Requires should entail %{release}
Requires:        %{name} = %{version}-%{release}

#003: ORBit2-devel owns /usr/lib//orbit-2.0
remove %dir %{_libdir}/orbit-2.0 and add ORBit2-devel as requires

#004: (devel package) %{_libdir}/pkgconfig/* is owned by pkgconfig. Add 
pkconfig as requires

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

#006: typo Group:		Development/Librarie
missing "s"

Comment 2 Chitlesh GOORAH 2007-08-27 14:00:08 UTC
#007: Missing g-wrap-devel as Requires for %package devel

Requires:       g-wrap-devel


Comment 4 Chitlesh GOORAH 2007-08-27 19:49:45 UTC
I'm missing the following while installing :
[root@goorah ~]# 
rpm -ivh /home/chitlesh/rpmbuild/RPMS/i386/guile-gnome-platform-devel-2.15.93-3.fc7.i386.rpm /home/chitlesh/rpmbuild/RPMS/i386/guile-gnome-platform-2.15.93-3.fc7.i386.rpm
Preparing...                ########################################### [100%]
   1:guile-gnome-platform   ########################################### [ 50%]
install-info: No such file or directory 
for /usr/share/info/quile-gnome-tutorial.info.gz
   2:guile-gnome-platform-de########################################### [100%]

Notice the "q" on /usr/share/info/quile-gnome-tutorial.info.gz

Comment 6 Chitlesh GOORAH 2007-08-31 20:57:54 UTC
#001: Texinfo:
missing:
Requires(post): /sbin/install-info
Requires(preun): /sbin/install-info

#002: -devel package
Requires:       guile-devel, g-wrap-devel

g-wrap-devel already requires guile-devel
remove guile-devel from requires of -devel

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

#004:rpmlint /home/chitlesh/rpmbuild/SRPMS/guile-gnome-platform-2.15.93-3.fc7.src.rpm
W: guile-gnome-platform strange-permission guile-gnome-platform-2.15.93.tar.gz 
0755
W: guile-gnome-platform strange-permission guile-gnome-platform.spec 0755
W: guile-gnome-platform mixed-use-of-spaces-and-tabs (spaces: line 1, tab: 
line 14)

Comment 8 Chitlesh GOORAH 2007-09-09 10:35:58 UTC
For the record, Xavier has excluded PPC64, because it can't be built under 
that arch. Upstream has been notified and they are working on it.

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

This package is approved by ME.

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

Comment 9 Xavier Lamien 2007-09-09 10:41:35 UTC
New Package CVS Request
=======================
Package Name: guile-gnome-platform
Short Description: Guile wrapper collection for the GNOME library stack
Owners: lxtnow
Branches: FC-6 F-7 EL-4 EL-5 Devel
InitialCC:
cvsextras commit: no




Comment 10 Mamoru TASAKA 2007-09-09 14:19:58 UTC
Well,

* First I cannot install this rpm.
----------------------------------------------------------------
[root@localhost guile-gnome-platform]# LANG=C rpm -ivh --test
guile-gnome-platform-2.15.93-4.fc8.i386.rpm 
Preparing...                ########################################### [100%]
        file /usr/share/info/dir from install of
guile-gnome-platform-2.15.93-4.fc8 conflicts with file from package info-4.9-2.fc8
---------------------------------------------------------------

* Would you explain why guile-gnome-platform should have
  "Requires: orbit2-devel"? (although the ownership of
  %_libdir/orbit-2.0 is apparently wrong..)

* build.log shows that fedora specific compilation flags are not
  honored under
  BUILD/guile-gnome-platform-2.15.93/gtk/gnome/gw

* Check the dependency for -devel package.
  For example,  guile-gnome-gtk-0.pc contains the line:
-------------------------------------------------
Requires: gobject-2.0 g-wrap-2.0-guile
-------------------------------------------------
  This means guile-gnome-platform-devel should have
  "Requires: g-wrap-devel 

Comment 11 Kevin Fenzi 2007-09-09 22:58:59 UTC
Xavier: Can you address the items from comment #10 and then reset the fedora-cvs
flag to ? when the package is ready? 

Comment 12 Xavier Lamien 2007-09-10 10:01:38 UTC
(In reply to comment #10)
> Well,
> 
> * First I cannot install this rpm.
> ----------------------------------------------------------------
> [root@localhost guile-gnome-platform]# LANG=C rpm -ivh --test
> guile-gnome-platform-2.15.93-4.fc8.i386.rpm 
> Preparing...                ########################################### [100%]
>         file /usr/share/info/dir from install of
> guile-gnome-platform-2.15.93-4.fc8 conflicts with file from package info-4.9-2.fc8
> ---------------------------------------------------------------

Thanks for pointed me this, not happened on mine.

> 
> * Would you explain why guile-gnome-platform should have
>   "Requires: orbit2-devel"? (although the ownership of
>   %_libdir/orbit-2.0 is apparently wrong..)
> 
_libdir/orbit-2.0 is currently owned by ORBit2-devel, so this package have to be
installed 

> * build.log shows that fedora specific compilation flags are not
>   honored under
>   BUILD/guile-gnome-platform-2.15.93/gtk/gnome/gw
> 

Will check this, thanks

> * Check the dependency for -devel package.
>   For example,  guile-gnome-gtk-0.pc contains the line:
> -------------------------------------------------
> Requires: gobject-2.0 g-wrap-2.0-guile
> -------------------------------------------------
>   This means guile-gnome-platform-devel should have
>   "Requires: g-wrap-devel 

this has already been fixed for release *-3

Comment 13 Mamoru TASAKA 2007-09-10 10:13:43 UTC
(In reply to comment #12)

> > * Would you explain why guile-gnome-platform should have
> >   "Requires: orbit2-devel"? (although the ownership of
> >   %_libdir/orbit-2.0 is apparently wrong..)
> > 
> _libdir/orbit-2.0 is currently owned by ORBit2-devel, 
> so this package have to be installed 

  - So I am now reviewing Merge review of ORBit2 (bug 226223).
    Anyway main package should not require ORBit2-devel.


> > * Check the dependency for -devel package.
> >   For example,  guile-gnome-gtk-0.pc contains the line:
> > -------------------------------------------------
> > Requires: gobject-2.0 g-wrap-2.0-guile
> > -------------------------------------------------
> >   This means guile-gnome-platform-devel should have
> >   "Requires: g-wrap-devel 
> 
> this has already been fixed for release *-3
  - Please also check other .pc file.
    For example, guile-gnome-libgnomeui-0.pc says -devel
    subpackage should require guile-cairo-devel



Comment 14 Xavier Lamien 2007-09-10 10:48:11 UTC
ok

Comment 16 Mamoru TASAKA 2007-09-11 05:16:08 UTC
Umm.. rebuild failed..
http://koji.fedoraproject.org/koji/taskinfo?taskID=155082

This is due to bug 285361

Comment 17 Mamoru TASAKA 2007-09-11 16:08:42 UTC
For me almost okay, except for 2 issues.

* For the line (in %prep)
-----------------------------------------------------
export CFLAGS="%{optflags} -I%{_libdir}/gcc/*-redhat-linux-gnu/include/"
sed -i "s|@GUILE_CAIRO_CFLAGS@|@GUILE_CAIRO_CFLAGS@
-I%{_libdir}/gcc/*-redhat-linux-gnu/include/|" gtk/gnome/gw/Makefile.in
-----------------------------------------------------
  - Well, my system does not have any directory which matches
    %_libdir/gcc/*-redhat-linux-gnu so these two lines can be
    removed

* rpmlint
  - rpmlint for installed -devel package complains about
    undefined-non-weak-symbol.
--------------------------------------------------------
guile-gnome-platform.i386: W: undefined-non-weak-symbol
/usr/lib/libguile-gnome-corba-0.so.0.0.0 TC_CORBA_sequence_CORBA_any_struct
guile-gnome-platform.i386: W: undefined-non-weak-symbol
/usr/lib/libguile-gnome-corba-0.so.0.0.0 TC_null_struct
(and so many lines)
--------------------------------------------------------
    Linking of libguile-gnome-corba-0.so against libORBit-2.so
    and libbonobo-2.so is missing. For a brief solution,
    adding
-------------------------------------------------------- 
export LDFLAGS="-lORBit-2 -lbonobo-2"
--------------------------------------------------------
    before %configure fixes this.



Comment 19 Mamoru TASAKA 2007-09-11 18:46:04 UTC
Also okay from me.

Comment 20 Xavier Lamien 2007-09-11 19:21:41 UTC
Thanks Mamoru.

Note: ownership on ORBit-2 need to be fix before importing.


New Package CVS Request
=======================
Package Name: guile-gnome-platform
Short Description: Guile wrapper collection for the GNOME library stack
Owners: lxtnow
Branches: FC-6 F-7 EL-4 EL-5 Devel
InitialCC:
cvsextras commit: no

Comment 21 Kevin Fenzi 2007-09-12 15:52:21 UTC
cvs done.

Comment 22 Mamoru TASAKA 2007-09-24 13:32:32 UTC
Xavier, ORBit2 directory ownership issue is not fixed.

So would you have this package explitly own the directory in issue
and import this package anyway?

Comment 23 Xavier Lamien 2007-09-28 12:26:33 UTC
Ownership on ORBit-2 has already been fixed but not imported yet. 
I'm still waitting that the bug #226223 is fixed.

Comment 24 Mamoru TASAKA 2007-09-28 12:40:14 UTC
(In reply to comment #23)
> Ownership on ORBit-2 has already been fixed but not imported yet. 
> I'm still waitting that the bug #226223 is fixed.

So, for now please make %_libdir/orbit-2.0 owned by 
guile-gnome-platform explicitly 
(i.e. add %dir %_libdir/orbit-2.0 until bug 226223 is solved)
and import this package.

(My message:
(In reply to comment #22)
> Xavier, ORBit2 directory ownership issue is not fixed.
 was for ORBit2 package)

Comment 25 Xavier Lamien 2007-10-03 22:39:25 UTC
imported and built.
thx all