[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2011-06-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=612796

--- Comment #34 from Bug Zapper  2011-06-01 
10:22:04 EDT ---

This message is a reminder that Fedora 13 is nearing its end of life.
Approximately 30 (thirty) days from now Fedora will stop maintaining
and issuing updates for Fedora 13.  It is Fedora's policy to close all
bug reports from releases that are no longer maintained.  At that time
this bug will be closed as WONTFIX if it remains open with a Fedora 
'version' of '13'.

Package Maintainer: If you wish for this bug to remain open because you
plan to fix it in a currently maintained version, simply change the 'version' 
to a later Fedora version prior to Fedora 13's end of life.

Bug Reporter: Thank you for reporting this issue and we are sorry that 
we may not be able to fix it before Fedora 13 is end of life.  If you 
would still like to see this bug fixed and are able to reproduce it 
against a later version of Fedora please change the 'version' of this 
bug to the applicable version.  If you are unable to change the version, 
please add a comment here and someone will do it for you.

Although we aim to fix as many bugs as possible during every release's 
lifetime, sometimes those efforts are overtaken by events.  Often a 
more recent Fedora release includes newer upstream software that fixes 
bugs or makes them obsolete.

The process we are following is described here: 
http://fedoraproject.org/wiki/BugZappers/HouseKeeping

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2011-06-29 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=612796

Bug Zapper  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX
   Flag|needinfo?(and...@schomin.co |
   |m)  |
Last Closed||2011-06-29 09:47:00

--- Comment #35 from Bug Zapper  2011-06-29 
09:47:00 EDT ---

Fedora 13 changed to end-of-life (EOL) status on 2011-06-25. Fedora 13 is 
no longer maintained, which means that it will not receive any further 
security or bug fix updates. As a result we are closing this bug.

If you can reproduce this bug against a currently maintained version of 
Fedora please feel free to reopen this bug against that version.

Thank you for reporting this bug and we are sorry it could not be fixed.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-12-27 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=612796

--- Comment #29 from Fabian Affolter  2010-12-27 
05:32:13 EST ---
python-ping 0.2 was released in the meantime.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2011-01-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=612796

--- Comment #30 from Fabian Affolter  2011-01-04 
08:31:34 EST ---
Can you please update your package?

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2011-01-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=612796

--- Comment #31 from Andrew Schomin  2011-01-04 10:26:40 
EST ---
I've been kind of busy lately. Give me a day and I'll get the package updated.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2011-01-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=612796

--- Comment #32 from Fabian Affolter  2011-01-05 
04:06:19 EST ---
Thanks

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2011-01-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=612796

Fabian Affolter  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2011-02-08 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=612796

Fabian Affolter  changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
 AssignedTo|fab...@bernewireless.net|nob...@fedoraproject.org
   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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2011-02-08 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=612796

Jason Tibbitts  changed:

   What|Removed |Added

  Status Whiteboard||StalledSubmitter

--- Comment #33 from Jason Tibbitts  2011-02-08 15:21:45 EST 
---
Not sure why this was returned to the review queue instead of just being
closed, but I've marked it stalled.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-08 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=612796

Andrew Schomin  changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-08 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=612796

Andrew Schomin  changed:

   What|Removed |Added

Version|rawhide |13

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-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=612796

Athmane Madjoudj  changed:

   What|Removed |Added

 CC||athma...@gmail.com

--- Comment #1 from Athmane Madjoudj  2010-07-09 13:49:32 
EDT ---
rpmlint shows the following:

$ rpmlint python-ping-0.1-1.fc13.src.rpm 
python-ping.src: I: enchant-dictionary-not-found en_US
python-ping.src: W: summary-ended-with-dot C An implementation of the standard
ICMP ping in pure Python.
python-ping.src: E: no-changelogname-tag
python-ping.src:3: W: macro-in-comment %global
python-ping.src:3: W: macro-in-comment %{__python}
python-ping.src:44: W: macro-in-comment %{python_sitearch}
1 packages and 0 specfiles checked; 1 errors, 4 warnings.


$ rpmlint python-ping-0.1-1.fc13.src.rpm 
python-ping.src: I: enchant-dictionary-not-found en_US
python-ping.src: W: summary-ended-with-dot C An implementation of the standard
ICMP ping in pure Python.
python-ping.src: E: no-changelogname-tag
python-ping.src:3: W: macro-in-comment %global
python-ping.src:3: W: macro-in-comment %{__python}
python-ping.src:44: W: macro-in-comment %{python_sitearch}
1 packages and 0 specfiles checked; 1 errors, 4 warnings.


Proposed fixes:

[-] Try to cleanup all comments containing macros.
[-] Remove dot in Summary.
[-] Add the first Changelogs entry line, some thing like: - Initial package

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-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=612796

--- Comment #3 from Athmane Madjoudj  2010-07-09 13:57:35 
EDT ---
if this is pure python it should be platform/hardware independent ie:
BuildArch: noarch

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-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=612796

--- Comment #2 from Athmane Madjoudj  2010-07-09 13:54:15 
EDT ---
Sorry I forgot:

$ rpmlint python-ping-0.1-1.fc13.i686.rpm 
python-ping.i686: I: enchant-dictionary-not-found en_US
python-ping.i686: W: summary-ended-with-dot C An implementation of the standard
ICMP ping in pure Python.
python-ping.i686: E: no-changelogname-tag
python-ping.i686: E: no-binary
python-ping.i686: W: no-documentation
python-ping.i686: E: non-executable-script
/usr/lib/python2.6/site-packages/ping.py 0644L /usr/bin/env
1 packages and 0 specfiles checked; 3 errors, 2 warnings.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-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=612796

--- Comment #4 from Andrew Schomin  2010-07-09 14:56:14 EDT 
---
[make...@andrew SRPMS]$ rpmlint python-ping-0.1-1.fc13.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.


Made changes heres the new output. Sorry I'm a new noob forgot to run rpmlint.
Re-uploaded the packages to the URL above. They are the same 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-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=612796

--- Comment #5 from Athmane Madjoudj  2010-07-09 15:56:42 
EDT ---
rpmlint on SPEC and SRPM is OK.

but 

$ rpmlint python-ping-0.1-1.fc13.noarch.rpm 
python-ping.noarch: I: enchant-dictionary-not-found en_US
python-ping.noarch: W: no-documentation
python-ping.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/ping.py 0644L /usr/bin/env
1 packages and 0 specfiles checked; 1 errors, 1 warnings.


You should either remove shebang or make executable: ping.py

Tip to remove shebang:
%{__sed} -i '1d' ping.py

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-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=612796

--- Comment #6 from Athmane Madjoudj  2010-07-09 16:03:03 
EDT ---
i have looked at the source of ping.py it seems to be a pure module so removing
shebang is more appropriate.

[...SNIP...]
if __name__ == '__main__':
verbose_ping("heise.de")
verbose_ping("google.com")
verbose_ping("a-test-url-taht-is-not-available.com")
verbose_ping("192.168.1.1")

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-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=612796

--- Comment #7 from Athmane Madjoudj  2010-07-09 16:11:13 
EDT ---
you should add the following files to the %doc section:

AUTHORS
COPYING
README

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-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=612796

--- Comment #8 from Andrew Schomin  2010-07-09 16:20:42 EDT 
---
Ok added %{__sed} -i '1d' ping.py
and the stuff in %doc.

Only warning I'm getting is invalid URL on the source location but it was
working a minute ago must be down right now.

Uploaded updated 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-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=612796

--- Comment #9 from Andrew Schomin  2010-07-09 16:21:36 EDT 
---
never mind URL is working now

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-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=612796

--- Comment #10 from Athmane Madjoudj  2010-07-09 16:41:39 
EDT ---
rpmlint is OK now.

$ rpmlint python-ping-0.1-1.fc13.noarch.rpm 
python-ping.noarch: I: enchant-dictionary-not-found en_US
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint python-ping.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

$ rpmlint python-ping-0.1-1.fc13.src.rpm 
python-ping.src: I: enchant-dictionary-not-found en_US
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

PS.
Other fedora contributor should review this package, I'm a new package
reviewer.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-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=612796

--- Comment #11 from Andrew Schomin  2010-07-09 16:48:02 
EDT ---
Alright thank you. Do I just need to wait until someone else looks at it then?

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-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=612796

Fabian Affolter  changed:

   What|Removed |Added

 CC||fab...@bernewireless.net

--- Comment #12 from Fabian Affolter  2010-07-09 
17:26:52 EDT ---
Just some more comment:

- The license is GPLv2+, see 'version 2 of the License, or (at your option) any
later version.'
- This is a 'noarch' package the CFLAGS are not needed.

Everytime you make changes to your spec file, you must bump the release.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-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=612796

--- Comment #13 from Andrew Schomin  2010-07-09 17:42:37 
EDT ---
Thank you for the comments I made the changes you recommended and uploaded the
files:

SPEC: http://www.schomin.com/fedora/python-ping.spec
SRPM: http://www.schomin.com/fedora/python-ping-0.1-4.fc13.src.rpm

I bumped the release to 4 as this is the 4th time I've made changes to the spec
file and built the package now.

Let me know if anything else needs to be 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-10 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=612796

--- Comment #14 from Fabian Affolter  2010-07-10 
10:47:00 EDT ---
There are still some issues:

- The %{python_sitelib} macros should be defined as mentioned in the guidelines
  https://fedoraproject.org/wiki/Packaging:Python#Macros
- The license is still wrong
- Every release bump must have a corresponding changelog entry.
  https://fedoraproject.org/wiki/Packaging:Guidelines#Changelogs
- Isn't the build command missing?

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-10 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=612796

--- Comment #15 from Andrew Schomin  2010-07-10 14:51:45 
EDT ---
Ok I change the  %{python_sitelib} macro definition to have the if statement
for Fedora 12 or bellow and RHEL 5 or bellow. I assume thats what you were
refering to since I had already defined %{python_sitelib}.

I added GPLv2+ to the liscense.

Added changes to change log

I thought when you said CFLAGS wasn't needed to remove the line. The build
command is back now.

Here are the new files.

SPEC: http://www.schomin.com/fedora/python-ping.spec
SRPM: http://www.schomin.com/fedora/python-ping-0.1-6.fc13.src.rpm

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-10 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=612796

--- Comment #16 from Athmane Madjoudj  2010-07-10 15:00:45 
EDT ---
AFAIK you should keep history of changes in changelog section,
since this is -6 they should be 6 entries, correct me if I'm wrong.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-10 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=612796

--- Comment #17 from Andrew Schomin  2010-07-10 15:03:34 
EDT ---
So I should be keeping all previous changes in the log for 1, 2, 3, etc...?

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-10 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=612796

--- Comment #18 from Andrew Schomin  2010-07-10 15:34:25 
EDT ---
Anyway I assume thats the case and have made the change there as well here are
the updated for files.

Sorry for all of the changes guys this stuff is probably pretty trivial. Just
trying to get the hang of things.

SPEC: http://www.schomin.com/fedora/python-ping.spec
SRPM: http://www.schomin.com/fedora/python-ping-0.1-7.fc13.src.rpm

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-11 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=612796

--- Comment #19 from Andrew Schomin  2010-07-11 20:19:12 
EDT ---
Looks like the links above were down today, but they should be working again
now.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-13 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=612796

--- Comment #20 from Andrew Schomin  2010-07-13 13:35:44 
EDT ---
Does the package look like its good to go? or does it still need work?

Also not positive of the procedure for getting a sponsor since it my first
time, but http://fedoraproject.org/wiki/PackageMaintainers/Join doesn't really
give specifics. I have a Fedora account already.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-13 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=612796

--- Comment #21 from Andrew Schomin  2010-07-13 13:37:27 
EDT ---
Never mind found
http://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-16 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=612796

Pierre Bourdon  changed:

   What|Removed |Added

 CC||delr...@gmail.com

--- Comment #22 from Pierre Bourdon  2010-07-16 16:17:51 EDT 
---
Hi,

For starter, I'm the current python-ping upstream maintainer. I would like to
point that Fabian is not exactly right about the license : while setup.py is
GPLv2+, ping.py is GPLv2 only, so this package should be advertised as using a
GPLv2 license instead of GPLv2+.

When I'll release 0.2 (probably not soon as the code is quite mature and is
around since 1997), I'll probably change the license of setup.py from v2+ to v2
to avoid any confusion like this.

Anyway, thanks for considering python-ping for inclusion in Fedora :) .

Regards,
Pierre.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-16 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=612796

--- Comment #23 from Andrew Schomin  2010-07-16 17:14:28 
EDT ---
Sorry about the licensing mix up. I'll get the spec changed back to GPLv2.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-07-16 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=612796

--- Comment #24 from Andrew Schomin  2010-07-16 17:24:40 
EDT ---
Updated spec and srpm:

SPEC: http://www.schomin.com/fedora/python-ping.spec
SRPM: http://www.schomin.com/fedora/python-ping-0.1-8.fc13.src.rpm

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-10-26 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=612796

--- Comment #25 from Fabian Affolter  2010-10-26 
07:37:49 EDT ---
The URLs for the spec file and the Source RPM are not right or the files are
gone.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-10-29 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=612796

--- Comment #26 from Andrew Schomin  2010-10-29 13:30:28 
EDT ---
The server was moved I'm uploading the files now. Links will be up soon.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-10-29 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=612796

--- Comment #27 from Andrew Schomin  2010-10-29 13:38:15 
EDT ---
spec and srpm from 2010-07-16 back up.

SPEC: http://www.schomin.com/fedora/python-ping.spec
SRPM: http://www.schomin.com/fedora/python-ping-0.1-8.fc13.src.rpm

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612796] Review Request: python-ping - An implementation of the standard ICMP ping in pure Python.

2010-11-17 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=612796

Fabian Affolter  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|fab...@bernewireless.net

--- Comment #28 from Fabian Affolter  2010-11-17 
06:44:52 EST ---
Thanks for the links.  I can do an informal review but according to the
guidelines a sponsor has to do the official 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review