[Bug 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-09-25 12:33:28 --- Comment #14 from Fedora Update System upda...@fedoraproject.org --- php-pear-Net-DNS2-1.2.4-1.fc18 has been pushed to the Fedora 18 stable repository. -- 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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #13 from Fedora Update System upda...@fedoraproject.org --- php-pear-Net-DNS2-1.2.3-1.fc18 has been pushed to the Fedora 18 testing repository. -- 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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 --- Comment #5 from Shawn Iwinski shawn.iwin...@gmail.com --- (In reply to comment #4) (In reply to comment #3) In %check, I'm assuming ping -c 1 dns.google.com is to check whether the build host is connected to a public network and will then run all tests? Yes. But it also check that resolver works. So, as still prefer a name rather that an ip (and a server allowing ping) But your ping test would check that the build machine's DNS is set up and resolving correctly. As far as I understand the package, it could work regardless of whether the build machine's DNS is set up correctly or not by correctly providing DNS nameserver IP addresses (and the package resolver test uses the Google Public DNS IPs). If you really prefer a DNS name, perhaps use google.com instead? I believe this would resolve for more people than dns.google.com -- this would match the actual package resolver test as well. dns.google.com does not resolve on my machine and may not for other building machines. Perhaps instead use one of the Google Public DNS IP addresses used in the resolver test file since they respond to pings as well? -- 8.8.8.8 or 8.8.4.4 In our case, koji won't run this test. This is the designed behavior. Do you think this is a blocker for the review ? No, not a blocker ;) -- 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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 --- Comment #6 from Shawn Iwinski shawn.iwin...@gmail.com --- Created attachment 612806 -- https://bugzilla.redhat.com/attachment.cgi?id=612806action=edit Generated by fedora-review 0.2.2 (9f8c0e5) last change: 2012-08-09 -- 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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 Shawn Iwinski shawn.iwin...@gmail.com changed: What|Removed |Added Flags|fedora-review? | Flags||fedora-review+ --- Comment #7 from Shawn Iwinski shawn.iwin...@gmail.com --- No blockers. APPROVED -- 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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 Remi Collet fed...@famillecollet.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #8 from Remi Collet fed...@famillecollet.com --- Thanks for the review. I will change used DNS after import. New Package SCM Request === Package Name: php-pear-Net-DNS2 Short Description: PHP Resolver library used to communicate with a DNS server Owners: remi Branches: f17 f18 el6 InitialCC: -- 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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 --- Comment #9 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|NEW |MODIFIED -- 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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 --- Comment #10 from Fedora Update System upda...@fedoraproject.org --- php-pear-Net-DNS2-1.2.3-1.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/php-pear-Net-DNS2-1.2.3-1.fc18 -- 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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 --- Comment #11 from Fedora Update System upda...@fedoraproject.org --- php-pear-Net-DNS2-1.2.3-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/php-pear-Net-DNS2-1.2.3-1.fc17 -- 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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 --- Comment #12 from Fedora Update System upda...@fedoraproject.org --- php-pear-Net-DNS2-1.2.3-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/php-pear-Net-DNS2-1.2.3-1.el6 -- 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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 Shawn Iwinski shawn.iwin...@gmail.com changed: What|Removed |Added CC||shawn.iwin...@gmail.com Assignee|nob...@fedoraproject.org|shawn.iwin...@gmail.com Flags||fedora-review? --- Comment #3 from Shawn Iwinski shawn.iwin...@gmail.com --- In %check, I'm assuming ping -c 1 dns.google.com is to check whether the build host is connected to a public network and will then run all tests? dns.google.com does not resolve on my machine and may not for other building machines. Perhaps instead use one of the Google Public DNS IP addresses used in the resolver test file since they respond to pings as well? -- 8.8.8.8 or 8.8.4.4 -- 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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 --- Comment #4 from Remi Collet fed...@famillecollet.com --- (In reply to comment #3) In %check, I'm assuming ping -c 1 dns.google.com is to check whether the build host is connected to a public network and will then run all tests? Yes. But it also check that resolver works. So, as still prefer a name rather that an ip (and a server allowing ping) dns.google.com does not resolve on my machine and may not for other building machines. Perhaps instead use one of the Google Public DNS IP addresses used in the resolver test file since they respond to pings as well? -- 8.8.8.8 or 8.8.4.4 In our case, koji won't run this test. This is the designed behavior. Do you think this is a blocker for the review ? -- 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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 --- Comment #2 from Remi Collet fed...@famillecollet.com --- On our request, upstream now provides LICENSE and tests, so update to 1.2.3 https://github.com/remicollet/remirepo/commit/2f145f1f69d15eadbd555b8b228f0666882c0574 http://rpms.famillecollet.com/SRPMS/php-pear-Net-DNS2-1.2.3-1.remi.src.rpm Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=4401290 -- 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 847517] Review Request: php-pear-Net-DNS2 - PHP Resolver library used to communicate with a DNS server
https://bugzilla.redhat.com/show_bug.cgi?id=847517 --- Comment #1 from Remi Collet fed...@famillecollet.com --- Minor changes (use php-pear(PEAR) instead of php-pear), so http://rpms.famillecollet.com/SRPMS/php-pear-Net-DNS2-1.2.2-2.remi.src.rpm -- 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