[Bug 276961] Review Request: straw - Desktop news aggregator

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: straw - Desktop news aggregator


https://bugzilla.redhat.com/show_bug.cgi?id=276961





------- Additional Comments From mtasaka at ioa.s.u-tokyo.ac.jp  2007-09-09 04:37 EST -------
Created an attachment (id=190961)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=190961&action=view)
gdb log of straw on rawhide

Well, even if you are a ambassador, you still have to
be sponsored to get the proper right to access Fedora CVS.

For 0.27-1:

* BuildRequires
  - At least "BuildRequires: python-devel" is missing
    (Note: I first fixed the typo "%{__python)" to try rebuilding)
    http://koji.fedoraproject.org/koji/taskinfo?taskID=152954
  - Even after I add "BuildRequires: python-devel" the rebuild
    still fails like:
    http://koji.fedoraproject.org/koji/taskinfo?taskID=152956
    Please check the BuildRequires so as to be rebuilt in
    mock.

    Note: check setup.py
-------------------------------------------------------------
    33	if sys.version < strawenv['PYTHON']:
    47	    try:
    48		import pygtk
    49		pygtk.require('2.0')
    50		imp.find_module('gtk')
    51		import gtk
    52		if gtk.pygtk_version < strawenv['PYGTK']:
    72	    mod_list = [
    73		('gnome', 'python-gnome2', 0),
    74		('gtkhtml2', "python-gnome2-extras", 0),
    75		('gconf', "python-gnome2", 1),
    76		('bsddb.db', 'python-bsddb', 0),
    77		('dbus', 'python-dbus', 0)]
-----------------------------------------------------------

* Requires
  - Please check the dependency for straw correctly.
    * For example, straw/strawdbus.py comtails the line
      "import dbus", which means this package should have
      "Requires: dbus-python"
    * Some other dependencies are also missing
    * Here you can again check what "setup.py" requires.
  - Why does this need "Requires: hicolor-icon-theme"?

* desktop vendor prefix
  - This is not rpmforge... Change to fedora.

* GConf scriptlets
  - Please refer to the section "GConf" of
    http://fedoraproject.org/wiki/Packaging/ScriptletSnippets

* License
  - License is actually GPLv2+ as the source codes actually
    say so.

* Runtime problem
  - This package cannot be launched on rawhide. gdb log attached
    (Maybe this is due to python-adns)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.