Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 168189 - Review Request: gtk+extra - A library of gtk+ widgets
Summary: Review Request: gtk+extra - A library of gtk+ widgets
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jose Pedro Oliveira
QA Contact: David Lawrence
URL: http://www.dattalo.com/gnupic/gpsim.html
Whiteboard:
Depends On:
Blocks: FE-ACCEPT 168190
TreeView+ depends on / blocked
 
Reported: 2005-09-13 11:54 UTC by Alain Portal
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2005-09-22 10:23:04 UTC
Type: ---
Embargoed:
petersen: fedora-cvs+


Attachments (Terms of Use)
Specfile patch to correct the problems mentioned in comment #1 (deleted)
2005-09-15 02:01 UTC, Jose Pedro Oliveira
no flags Details | Diff

Description Alain Portal 2005-09-13 11:54:59 UTC
Spec Name or Url: http://linuxelectronique.free.fr/download/fedora/4/SPECS/gtk+extra.spec
SRPM Name or Url: http://linuxelectronique.free.fr/download/fedora/4/SRPMS/gtk+extra-1.1.0-2.src.rpm
Description: A library of dynamically linked gtk+ widgets including:
GtkSheet, GtkPlot, and GtkIconList

This package is needed by gpsim.

Comment 1 Jose Pedro Oliveira 2005-09-15 01:53:03 UTC
Alain,

A couple of things that need to be fixed:

* the BR is wrong. the BR should be gtk2-devel. 

  output of configure
  -------------------
  ...
  checking for GTK+ - version >= 1.3.13... yes (version 2.4.14)
  checking for pkg-config... (cached) /usr/bin/pkg-config
  checking for GLIB - version >= 1.3.13... yes (version 2.4.8)
  checking for gtk+-2.0 >= 1.3.13... yes
  ...


  and the following mail
  ----------------------
http://www.linuxhacker.org/cgi-bin/ezmlm-cgi?1:mss:3641:200412:lecfjpaammlnkeekfboj

  Note: glib2-devel gets pulled by gtk2-devel

* the gtk+extras-devel package should require the gtk2-devel
  (includes <gtk/gtk.h>)

* exclude the .la file

* lots of documentation missing
 
/jpo

Comment 2 Jose Pedro Oliveira 2005-09-15 02:01:31 UTC
Created attachment 118831 [details]
Specfile patch to correct the problems mentioned in comment #1

Changelog:
* corrects the problems mentioned in comment #1
* most of the documentation is in DOS format
  (converts docs with line terminations "\r\n" to "\n")
* also moved a couple of ChangeLog files from the main packaged
  to the devel one (appears to make more sense)
* reformats a couple of lines (non-critical)

Comment 3 Alain Portal 2005-09-15 08:07:00 UTC
Jose,    
Thanks for reviewing, and of course for the patch (fully applied).   
   
%changelog   
* Thu Sep 15 2005 Alain Portal <aportal[AT]univ-montp2[DOT]fr> 1.1.0-3   
  - BuildRequires is gtk2-devel, not gtk+-devel   
  - Add Requires gtk2-devel for package devel   
  - Exclude .la files   
  - Add a lot of documentation   
  - Move gtk*.ChangeLog in devel package   
  - Convert DOS format end-of-line to Unix-like format   
  - Contributions of Jose Pedro Oliveira <jpo[AT]di[DOT]uminho[DOT]pt>   
    Thanks to him.   
   
Spec Name or Url:  
http://linuxelectronique.free.fr/download/fedora/4/SPECS/gtk+extra.spec  
SRPM Name or Url: 
http://linuxelectronique.free.fr/download/fedora/4/SRPMS/gtk+extra-1.1.0-3.src.rpm 

Comment 4 Jose Pedro Oliveira 2005-09-15 11:35:31 UTC
APPROVED

MD5SUMS:
f326a2710e09247104de18da1d0a207d  gtk+extra-1.1.0-3.src.rpm

886bbd0c54d59a9159df097c125f54cd  gtk+extra-1.1.0.tar.gz
8009a2731dc7511a95e30b78abe555f2  gtk+extra.spec


Good:
* Tarball MD5 digest verified against upstream copy
* URL and Source url are valid
* License text file included
* Builds without problems in FC-3
* Builds without problems in mock (FC-4)
* no rpmlint output


Comment 5 Jose Pedro Oliveira 2005-09-15 11:36:49 UTC
Alain,

Do you still have CVS access problems? Do you want me to import and build it?

/jpo

Comment 6 Alain Portal 2005-09-15 12:35:27 UTC
I can't access CVS, but I think this is an account problem, I can't change my 
password!!! 
 
So please, import and build. 
 
Your sincerely. 
Alain 

Comment 7 Jose Pedro Oliveira 2005-09-22 10:23:04 UTC
Devel and FC-3 RPMS are already available in the mirrors.
FC-4 RPMS just built.

Comment 8 Alain Portal 2007-07-20 18:34:02 UTC
Package Change Request
======================
Package Name: gtk+extras
Updated Fedora Owners: alain.portal

Please, add my home email in comps because I'm on vacation for 6 weeks.

Comment 9 Jens Petersen 2007-07-24 13:59:34 UTC
Added as comaintainer - might be better just to make it initialcc but anyway.


Note You need to log in before you can comment on or make changes to this bug.