[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-05-01 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2013-05-01 15:23:36

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=MPeObCUiTGa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-05-01 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

--- Comment #11 from Fedora Update System upda...@fedoraproject.org ---
libuv-0.10.4-1.el6, nodejs-0.10.4-1.el6,
http-parser-2.0-4.20121128gitcd01361.el6, c-ares19-1.9.1-4.el6.3 has been
pushed to the Fedora EPEL 6 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=w5CpYQsxD1a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-16 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
libuv-0.10.4-1.el6, nodejs-0.10.4-1.el6,
http-parser-2.0-4.20121128gitcd01361.el6, c-ares19-1.9.1-4.el6.3 has been
pushed to the Fedora EPEL 6 testing repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=BS1JWS0VZha=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-16 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=cX3PLhWGtwa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-15 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |MODIFIED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=NbDSfNrfqga=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-12 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

--- Comment #9 from Fedora Update System upda...@fedoraproject.org ---
nodejs-0.10.3-2.el6.1, libuv-0.10.3-2.el6,
http-parser-2.0-4.20121128gitcd01361.el6, c-ares19-1.9.1-4.el6.3 has been
pushed to the Fedora EPEL 6 testing repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=aFp4Phtzsha=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-12 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=XuUFiGyDSNa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-11 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

Jakub Hrozek jhro...@redhat.com changed:

   What|Removed |Added

  Flags|needinfo?(jhro...@redhat.co |
   |m)  |

--- Comment #3 from Jakub Hrozek jhro...@redhat.com ---
Thank you for the autoconf patch, I'll start using it in Fedora c-ares package.

Here is the correct link to the new package:
http://sgallagh.fedorapeople.org/packagereview/c-ares19/c-ares19-1.9.1-4.fc18.2.src.rpm

Koji Build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=5242026

rpmlint output:
c-ares19.ppc64: E: changelog-time-in-future 2013-04-12
c-ares19-debuginfo.ppc64: E: changelog-time-in-future 2013-04-12
c-ares19-devel.ppc64: E: changelog-time-in-future 2013-04-12
c-ares19-devel.ppc64: W: no-documentation
3 packages and 0 specfiles checked; 3 errors, 1 warnings.

The changelog in future is a simple specfile change that would be nice to get
done.
The no-documentation warning is OK, the man pages are removed on purpse.

* The package is named correctly given it's the 1.9 version
* Builds on all supported architectures (see the koji link)
* Smoke testing (a simple C program linked against c-ares19) worked fine
* Devel files are in the -devel package, file ownership seems sane
* Source tarballs' MD5 sums match:
  ** RPM  - 389db4917a3b58c4ce4ebfe961fd84c4
  ** upstream - 389db4917a3b58c4ce4ebfe961fd84c4

Fix the date in the changelog and I'll approve :-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=W6C8QtS7DHa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-11 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

--- Comment #4 from Stephen Gallagher sgall...@redhat.com ---
Oof, sorry about the bad links. Copy-paste errors.


SPEC: http://sgallagh.fedorapeople.org/packagereview/c-ares19/c-ares19.spec
SRPM:
http://sgallagh.fedorapeople.org/packagereview/c-ares19/c-ares19-1.9.1-4.fc18.3.src.rpm
KOJI: http://koji.fedoraproject.org/koji/taskinfo?taskID=5242142

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=dnGx28cqeda=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-11 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

Jakub Hrozek jhro...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-review+

--- Comment #5 from Jakub Hrozek jhro...@redhat.com ---
rpmlint output:
c-ares19-devel.x86_64: W: no-documentation
4 packages and 0 specfiles checked; 0 errors, 1 warnings.

My only comment was fixed, so this package is APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=vOnrmKNWisa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-11 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

Stephen Gallagher sgall...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #6 from Stephen Gallagher sgall...@redhat.com ---
New Package SCM Request
===
Package Name: c-ares19
Short Description: A library that performs asysnchronous DNS operations
Owners: sgallagh
Branches: el6
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=45kyK6VlEoa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-11 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |
  Flags||fedora-cvs+

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=cJigr1Szqta=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-11 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

--- Comment #7 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=0x2OypAAyPa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-11 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=PgP3SnYnKIa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-11 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
nodejs-0.10.3-2.el6.1,libuv-0.10.3-2.el6,http-parser-2.0-4.20121128gitcd01361.el6,c-ares19-1.9.1-4.el6.3
has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/nodejs-0.10.3-2.el6.1,libuv-0.10.3-2.el6,http-parser-2.0-4.20121128gitcd01361.el6,c-ares19-1.9.1-4.el6.3

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=5crYomEPGfa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-10 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

Jakub Hrozek jhro...@redhat.com changed:

   What|Removed |Added

 CC||jhro...@redhat.com
   Assignee|nob...@fedoraproject.org|jhro...@redhat.com

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=3oiD4RRY4va=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-10 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

Jakub Hrozek jhro...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

--- Comment #1 from Jakub Hrozek jhro...@redhat.com ---
koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=5238142

rpmlint output:
c-ares19-debuginfo.x86_64: E: debuginfo-without-sources
c-ares19-devel.x86_64: W: no-documentation

the debuginfo-without-sources sounds wrong to me, on my F-18 the debuginfo
contains the sources correctly..

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=jVSCstMJYYa=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 950639] Review Request: c-ares19 - A library that performs asynchronous DNS operations

2013-04-10 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=950639

Stephen Gallagher sgall...@redhat.com changed:

   What|Removed |Added

  Flags||needinfo?(jhro...@redhat.co
   ||m)

--- Comment #2 from Stephen Gallagher sgall...@redhat.com ---
SPEC: http://sgallagh.fedorapeople.org/packagereview/c-ares19/c-ares19.spec
SRPM:
http://sgallagh.fedorapeople.org/packagereview/c-ares19/c-ares19-1.9.1-4.fc18.1.src.rpm
KOJI:
http://sgallagh.fedorapeople.org/packagereview/c-ares19/c-ares19-1.9.1-4.fc18.1.src.rpm

So the reason for the debuginfo issue was that when I added autoreconf -if (to
process my changes to Makefile.am), it was overwriting the changes made by the
original patch c-ares-1.7.5-optflags.patch (which were written directly into
the configure script instead of configure.ac or its dependencies).

I added a patch that makes an equivalent modification to the
m4/cares-compilers.m4 and now it's producing appropriate debuginfo output (as
well as building with all the correct flags).

Jakub, you may want to convert to this approach in your Fedora package as well,
so you can run autoreconf in it.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=DDqq54ldaya=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review