[Bug 1046842] Review Request: python-ofxparse - Python library for working with the Open Financial Exchange file format

2014-01-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046842



--- Comment #10 from Fedora Update System  ---
python-ofxparse-0.14-2.fc19 has been pushed to the Fedora 19 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1046842] Review Request: python-ofxparse - Python library for working with the Open Financial Exchange file format

2014-01-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046842

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||python-ofxparse-0.14-2.fc20
 Resolution|--- |ERRATA
Last Closed||2014-01-17 00:46:53



--- Comment #9 from Fedora Update System  ---
python-ofxparse-0.14-2.fc20 has been pushed to the Fedora 20 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1046842] Review Request: python-ofxparse - Python library for working with the Open Financial Exchange file format

2014-01-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046842

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #8 from Fedora Update System  ---
python-ofxparse-0.14-2.fc20 has been pushed to the Fedora 20 testing
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1046842] Review Request: python-ofxparse - Python library for working with the Open Financial Exchange file format

2014-01-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046842



--- Comment #7 from Dan Callaghan  ---
EPEL6 build is coming soon, it needs patches to fix a build failure but I'm
going to wait for upstream's opinion first:

https://github.com/jseutter/ofxparse/pull/65

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1046842] Review Request: python-ofxparse - Python library for working with the Open Financial Exchange file format

2014-01-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046842



--- Comment #5 from Fedora Update System  ---
python-ofxparse-0.14-2.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/python-ofxparse-0.14-2.fc20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1046842] Review Request: python-ofxparse - Python library for working with the Open Financial Exchange file format

2014-01-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046842



--- Comment #6 from Fedora Update System  ---
python-ofxparse-0.14-2.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/python-ofxparse-0.14-2.fc19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1046842] Review Request: python-ofxparse - Python library for working with the Open Financial Exchange file format

2014-01-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046842

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1046842] Review Request: python-ofxparse - Python library for working with the Open Financial Exchange file format

2014-01-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046842

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1046842] Review Request: python-ofxparse - Python library for working with the Open Financial Exchange file format

2014-01-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046842



--- Comment #4 from Jon Ciesla  ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1046842] Review Request: python-ofxparse - Python library for working with the Open Financial Exchange file format

2014-01-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046842

Dan Callaghan  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #3 from Dan Callaghan  ---
New Package SCM Request
===
Package Name: python-ofxparse
Short Description: Python library for working with the Open Financial Exchange
file format
Owners: dcallagh
Branches: f19 f20 el6
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1046842] Review Request: python-ofxparse - Python library for working with the Open Financial Exchange file format

2014-01-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046842



--- Comment #2 from Dan Callaghan  ---
(In reply to Parag AN(पराग) from comment #1)
> Suggestions:
> 1) I will not block this review for %{python_sitelib} which should have been
> written as %{python2_sitelib} and same for %{__python} to %{__python2}

Thanks for the tip, I missed this change in the guidelines. I was going to
target EPEL6 for this package so I will change it to use %{__python2} with some
suitable conditionals when I import the package.

And thanks for the review :-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1046842] Review Request: python-ofxparse - Python library for working with the Open Financial Exchange file format

2013-12-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046842

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||panem...@gmail.com
   Assignee|nob...@fedoraproject.org|panem...@gmail.com
  Flags||fedora-review+



--- Comment #1 from Parag AN(पराग)  ---
Review:

+ Mock build successful for rawhide x86_64

+ rpmlint on generated rpms is
python3-ofxparse.noarch: W: spelling-error %description -l en_US ofx -> fox,
of, ox
python-ofxparse.noarch: W: spelling-error %description -l en_US ofx -> fox, of,
ox
python-ofxparse.src: W: spelling-error %description -l en_US ofx -> fox, of, ox
3 packages and 0 specfiles checked; 0 errors, 3 warnings.

+ Source verified with upstream as (sha256sum)
upstream tarball:
d8c486126a94d912442d040121db44fbc4a646ea70fa935df33b5b4dbfbbe42a
srpm tarball: d8c486126a94d912442d040121db44fbc4a646ea70fa935df33b5b4dbfbbe42a

+ License is valid and included in LICENSE file

Suggestions:
1) I will not block this review for %{python_sitelib} which should have been
written as %{python2_sitelib} and same for %{__python} to %{__python2}

you can check these python2 macros are already on the system
/etc/rpm/macros.python2

but you should use python2 macros actually.


APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review