[Bug 483360] Review Request: pdfshuffler - PDF file merging, rearranging, and spliting

2009-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|0.3.1-3.fc9 |0.3.1-3.fc10




-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483360] Review Request: pdfshuffler - PDF file merging, rearranging, and spliting

2009-03-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Fedora Update System   
2009-03-09 19:09:30 EDT ---
pdfshuffler-0.3.1-3.fc10 has been pushed to the Fedora 10 stable repository. 
If problems still persist, please make note of it in this bug report.

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483360] Review Request: pdfshuffler - PDF file merging, rearranging, and spliting

2009-03-05 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||0.3.1-3.fc9
 Resolution||NEXTRELEASE




-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483360] Review Request: pdfshuffler - PDF file merging, rearranging, and spliting

2009-03-05 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #8 from Fedora Update System   
2009-03-05 11:27:54 EDT ---
pdfshuffler-0.3.1-3.fc9 has been pushed to the Fedora 9 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483360] Review Request: pdfshuffler - PDF file merging, rearranging, and spliting

2009-02-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #7 from Fabian Affolter   2009-02-23 
10:53:48 EDT ---
https://admin.fedoraproject.org/updates/F9/FEDORA-2009-1756
https://admin.fedoraproject.org/updates/F10/FEDORA-2009-1746

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483360] Review Request: pdfshuffler - PDF file merging, rearranging, and spliting

2009-02-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|ON_QA




--- Comment #5 from Fedora Update System   
2009-02-04 21:12:52 EDT ---
pdfshuffler-0.3.1-2.fc10 has been pushed to the Fedora 10 testing repository. 
If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update pdfshuffler'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/F10/FEDORA-2009-1258

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483360] Review Request: pdfshuffler - PDF file merging, rearranging, and spliting

2009-02-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Fedora Update System   
2009-02-04 21:13:32 EDT ---
pdfshuffler-0.3.1-2.fc9 has been pushed to the Fedora 9 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing-newkey update pdfshuffler'.  You can
provide feedback for this update here:
http://admin.fedoraproject.org/updates/F9/FEDORA-2009-1271

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483360] Review Request: pdfshuffler - PDF file merging, rearranging, and spliting

2009-02-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #4 from Kevin Fenzi   2009-02-01 14:01:40 EDT ---
cvs done.

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483360] Review Request: pdfshuffler - PDF file merging, rearranging, and spliting

2009-01-31 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Fabian Affolter   2009-01-31 
04:34:11 EDT ---
Thanks for the review and finding the typo.

(In reply to comment #1)
> - packaging guidelines
>   - "The summary should be a short and concise description of the package" 
> ? "A desktop diary" -- left from different spec file ? :)

fixed

> OTHERS:
> - if you already insist on proper English grammar when patching (which is
> great!), do it proper;)
>   from the desktop file patch:
>   "Comment=PDF Merging, Rearranging, and Spliting"
>  ^

fixed in summary and .desktop files

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483360] Review Request: pdfshuffler - PDF file merging, rearranging, and spliting

2009-01-31 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fabian Affolter  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #3 from Fabian Affolter   2009-01-31 
04:35:09 EDT ---
New Package CVS Request
===
Package Name: pdfshuffler
Short Description: PDF file merging, rearranging, and splitting
Owners: fab
Branches: F-9 F-10
InitialCC:

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483360] Review Request: pdfshuffler - PDF file merging, rearranging, and spliting

2009-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Milos Jakubicek  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag|fedora-review?  |fedora-review+




--- Comment #1 from Milos Jakubicek   2009-01-30 20:09:45 
EDT ---
MUST:
+ rpmlint is silent
+ package name
+ spec file name
- packaging guidelines
  - "The summary should be a short and concise description of the package" 
? "A desktop diary" -- left from different spec file ? :)
+ Python guidelines:
  + defines %{python_sitelib}
  + builds egg from source
  + egg_info included
  + no unnecessary pyc/pyo
+ license (GPLv2+) ok, in COPYING as well in source files
+ spec file legible, English
+ source matches upstream (md5sum c59f3efcbc448c39fabbaac70c4237d4)
+ architectures ok (noarch)
+ builds in mock, no unnecessary BR
+ no need to ldconfig
+ locales ok (uses find_lang)
+ no library, hence no devel, static etc.
+ not relocatable
+ directories ownership ok
+ no dups in %files, proper %clean, %install
+ macros consistent
+ code and perm. content
+ %docs present
+ UTF-8

SHOULD:
+ license file
+ it works!:)

OTHERS:
- if you already insist on proper English grammar when patching (which is
great!), do it proper;)
  from the desktop file patch:
  "Comment=PDF Merging, Rearranging, and Spliting"
 ^
Provided that you fix the summary (and the patch typo:) when importing, this
package is APPROVED.

-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 483360] Review Request: pdfshuffler - PDF file merging, rearranging, and spliting

2009-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Milos Jakubicek  changed:

   What|Removed |Added

 CC||xja...@fi.muni.cz
 AssignedTo|nob...@fedoraproject.org|xja...@fi.muni.cz
   Flag||fedora-review?




-- 
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review