Bug 448531 - Review Request: pyrenamer - A mass file renamer written in PyGTK
Summary: Review Request: pyrenamer - A mass file renamer written in PyGTK
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 340011 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-05-27 14:01 UTC by Jean-François Martin
Modified: 2008-06-13 02:22 UTC (History)
3 users (show)

Fixed In Version: 0.5.0-4.fc9
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-06-13 02:22:37 UTC
Type: ---
Embargoed:
mtasaka: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Jean-François Martin 2008-05-27 14:01:47 UTC
Spec URL: http://lokthare.fedorapeople.org/temp/pyrenamer.spec
SRPM URL: http://lokthare.fedorapeople.org/temp/pyrenamer-0.5.0-1.fc9.src.rpm
Description:  With pyRenamer you can change the name of several files at the same time easily.

Comment 1 Jean-François Martin 2008-05-27 14:03:10 UTC
*** Bug 340011 has been marked as a duplicate of this bug. ***

Comment 2 Mamoru TASAKA 2008-05-27 16:41:52 UTC
Do you in need of a sponsor?

By the way your package does not build on dist-f10:
http://koji.fedoraproject.org/koji/taskinfo?taskID=630509

Comment 3 Jean-François Martin 2008-05-27 19:08:51 UTC
No, i don't to be sponsored (cvsextras approved)

Spec URL: http://lokthare.fedorapeople.org/temp/pyrenamer.spec
SRPM URL: http://lokthare.fedorapeople.org/temp/pyrenamer-0.5.0-2.fc9.src.rpm
* Fix BR: Add perl(XML::Parser).

Comment 4 Mamoru TASAKA 2008-05-28 15:00:04 UTC
For 0.5.0-2

* BuildRequires
  - "python-eyed3" seems needed also for BuildRequires.
  ! build.log shows (from following)
    http://koji.fedoraproject.org/koji/taskinfo?taskID=632906
--------------------------------------------------------
   177  checking for eyeD3 module... 
   178  no
   179  configure: WARNING: eyeD3 not found, rename music disabled!
--------------------------------------------------------
     This affects HAVE_EYED3 macro in configure{,.ac} then
     src/pyrenamer_globals.py.in. Actually the installed
     pyrenamer/pyrenamer_globals.py has:
--------------------------------------------------------
    26  have_eyed3=False
--------------------------------------------------------

* spec file convention
  - Please write in the order:
--------------------------------------------------------
%prep
%build
%install
%clean

%pre (%post, etc)

%files
--------------------------------------------------------
    i.e. Please don't write scriptlets between %prep and %build
         so that we can read your spec file easily.

Comment 5 Jean-François Martin 2008-05-28 20:58:47 UTC
Spec URL: http://lokthare.fedorapeople.org/temp/pyrenamer.spec
SRPM URL: http://lokthare.fedorapeople.org/temp/pyrenamer-0.5.0-3.fc9.src.rpm
- Fix BR: Add python-eyed3
- Reorder to match spec file convention

Comment 6 Mamoru TASAKA 2008-05-29 07:42:50 UTC
Okay, perhaps this is the final check.

For 0.5.0-3:
* python module dependency
----------------------------------------------------------
# grep -h 'import ' `rpm -ql pyrenamer 2>/dev/null | grep -v /usr/shre/doc` |
sed -e 's|^[ \t][ \t]*||' | sort | uniq
from gettext import gettext as _
from gettext import gettext as _  
from os import path as ospath
from os.path import isfile
from pyrenamer import pyrenamer
from pyrenamer import treefilebrowser
import EXIF
import TreeViewTooltips as tvt
import dircache
import eyeD3
import gconf
import getopt
import gettext
import glob
import gobject
import gtk
import gtk.gdk
import gtk.glade
import locale
import os
import pygtk
import pyrenamer_globals
import pyrenamer_globals as pyrenamerglob
import pyrenamerfilefuncs as renamerfilefuncs
import re
import sys
import threading
import time
import tooltips
import treefilebrowser
import user
import warnings
import webbrowser
----------------------------------------------------------
  - It seems the following is missing from Requires (please check!!)
----------------------------------------------------------
gnome-python2-gconf
pygtk2-libglade
----------------------------------------------------------

Comment 7 Jean-François Martin 2008-05-30 12:09:43 UTC
Spec URL: http://lokthare.fedorapeople.org/temp/pyrenamer.spec
SRPM URL: http://lokthare.fedorapeople.org/temp/pyrenamer-0.5.0-4.fc9.src.rpm
- Fix Requires : Add gnome-python2-gconf and pygtk2-libglade

Comment 8 Mamoru TASAKA 2008-05-30 13:21:16 UTC
Okay.

--------------------------------------------------------
   This package (pyrenamer) is APPROVED by me
--------------------------------------------------------

Please follow
http://fedoraproject.org/wiki/PackageMaintainers/NewPackageProcess
from step 7.

Comment 9 Jean-François Martin 2008-05-30 14:39:34 UTC
New Package CVS Request
=======================
Package Name: pyrenamer
Short Description: A mass file renamer written in PyGTK
Owners: lokthare
Branches: F-8 F-9
InitialCC: 
Cvsextras Commits: yes


Comment 10 Kevin Fenzi 2008-05-30 20:13:15 UTC
cvs done.

Comment 11 Fedora Update System 2008-05-30 21:56:33 UTC
pyrenamer-0.5.0-4.fc9 has been submitted as an update for Fedora 9

Comment 12 Fedora Update System 2008-06-13 02:22:35 UTC
pyrenamer-0.5.0-4.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.


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