[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
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=438804 Jussi Lehtola changed: What|Removed |Added CC||jussi.leht...@iki.fi 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
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=438804 Rakesh Pandit <[EMAIL PROTECTED]> changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||DUPLICATE --- Comment #32 from Rakesh Pandit <[EMAIL PROTECTED]> 2008-08-29 10:48:41 EDT --- I had no intention to hijack ;-) In case you are sponsored later feel free to take it back if interested. @Christopher It has been long pending. So, I will start a new ticket so that it is open to other folks for review. In case you are interested in REVIEW may you switch to new ticket. #460660 -- Thanks *** This bug has been marked as a duplicate of 460660 *** -- 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 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
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=438804 --- Comment #31 from David Hollis <[EMAIL PROTECTED]> 2008-08-29 08:09:08 EDT --- Feel free to take ownership on this. It's more important to me that it be packaged and available. If it means less work on my end, I can't complain! -- 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 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
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=438804 --- Comment #30 from Rakesh Pandit <[EMAIL PROTECTED]> 2008-08-23 12:16:53 EDT --- Are you interested in continuing with review? If yes, then you need to follow suggestions here: https://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored But, in case you don't want to continue with review and it is okay-- I am interested in taking it from here onwards. -- 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 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
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=438804 --- Comment #29 from David Hollis <[EMAIL PROTECTED]> 2008-08-14 07:15:36 EDT --- Waiting on sponsorship or for someone to take this package and run with it. -- 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 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
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=438804 --- Comment #28 from Jason Tibbitts <[EMAIL PROTECTED]> 2008-08-09 13:33:31 EDT --- Anything happening here? -- 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 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP Alias: php-pear-Auth https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-06-28 08:28 EST --- Shoot, just noticed that the requires for the subpackages was incorrect (php-pear(Auth_Radius) not php-pear(Auth_RADIUS)). Here's a corrected round: http://web.davehollis.com:81/packages/php-pear-Auth.spec http://web.davehollis.com:81/packages/php-pear-Auth-1.6.1-5.fc9.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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP Alias: php-pear-Auth https://bugzilla.redhat.com/show_bug.cgi?id=438804 Bug 438804 depends on bug 452105, which changed state. Bug 452105 Summary: Review Request: php-pear-Auth-RADIUS - Wrapper Classes for the RADIUS PECL https://bugzilla.redhat.com/show_bug.cgi?id=452105 What|Old Value |New Value Status|ASSIGNED|CLOSED Resolution||CURRENTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2008-06-28 08:19 EST --- Updated with suggestions from #25: http://web.davehollis.com:81/packages/php-pear-Auth.spec http://web.davehollis.com:81/packages/php-pear-Auth-1.6.1-4.fc9.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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: php-pear-Auth - provides methods for creating an
authentication system using PHP
Alias: php-pear-Auth
https://bugzilla.redhat.com/show_bug.cgi?id=438804
--- Additional Comments From [EMAIL PROTECTED] 2008-06-21 21:12 EST ---
Okay more fixes needed:
1. The sub-packages must require their parent package, like so:
Requires: %{name} = %{version}-%{release}
2. Please remove all tabs from the spec file
3. Please do what I mentioned in comment #20 (we need to discuss virtual
provides for pear subpackages)
4. Please update the %files section like so:
%files
%defattr(-,root,root,-)
%doc %{pear_name}-%{version}/docdir/%{pear_name}/*
%{pear_xmldir}/%{pear_name}.xml
%{pear_phpdir}/%{pear_name}*
%{pear_testdir}/%{pear_name}
%exclude %{pear_phpdir}/%{pear_name}/Container/SMBPasswd.php
%exclude %{pear_phpdir}/%{pear_name}/Container/RADIUS.php
%files samba
%defattr(-,root,root,-)
%{pear_phpdir}/%{pear_name}/Container/SMBPasswd.php
%files radius
%defattr(-,root,root,-)
%{pear_phpdir}/%{pear_name}/Container/RADIUS.php
--
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, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP Alias: php-pear-Auth https://bugzilla.redhat.com/show_bug.cgi?id=438804 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn||452105 -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP Alias: php-pear-Auth https://bugzilla.redhat.com/show_bug.cgi?id=438804 Bug 438804 depends on bug 450366, which changed state. Bug 450366 Summary: Review Request: php-pear-HTTP-Client - Easy way to perform multiple HTTP requests and process their results https://bugzilla.redhat.com/show_bug.cgi?id=450366 What|Old Value |New Value Status|ASSIGNED|CLOSED 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP Alias: php-pear-Auth https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-06-18 14:55 EST --- 1) Removed -addons package 2) Added -radius packages 3) Moved line-feed stuff back to %prep 4) Added HTTP_Client req Did need to use the %exclude in the main package to keep the sub-package files from being included there. The -radius package has a req on php-pear(Auth_Radius) though that package does not currently exist in F9 that I see. http://web.davehollis.com:81/packages/php-pear-Auth.spec http://web.davehollis.com:81/packages/php-pear-Auth-1.6.1-3.fc9.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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP Alias: php-pear-Auth https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-06-18 09:17 EST --- Dave, please make the following changes: 1. Instead of making an -addons package, simply rm or %exclude those files or include them in the main package. 2. Make a -radius subpackage which Requires: Auth_Radius and include RADIUS.php in the %files. 3. Remove the %exclude line, it is not needed. 4. Move the sed line back into %prep and supply a patch for the MD5SUM, see http://xulchris.fedorapeople.org/packages/php-pear-HTTP-Client.spec as an example 5. I will add php-pear-HTTP-Client today, please add this to the Requires 6. Perform steps I outlined in comment #20 -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP Alias: php-pear-Auth https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-06-17 13:29 EST --- Ok, I've split-off -samba and -addons and set the appropriate requires. http://web.davehollis.com:81/packages/php-pear-Auth.spec http://web.davehollis.com:81/packages/php-pear-Auth-1.6.1-2.fc9.src.rpm I did have to deal with the CRLF in the README.AUTH in the %install portion rather than %prep because modifying the file changed the MD5 sum and prevented the PEAR install from doing it's thing. I am using sed now instead of dos2unix so there aren't any undesirable dependencies. -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP Alias: php-pear-Auth https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-06-14 12:14 EST --- ping? -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP Alias: php-pear-Auth https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-06-07 11:51 EST --- David, please subscribe to the fedora php mailing list[1] if you havn't already done so and introduce yourself. We need to have a quick discussion on the mailing list regarding how to handle Provides for pear packages which have subpackages like Auth. I have also whipped up a quick HTTP_Client package which we can add to the Requires for Auth once it's approved. [1] https://www.redhat.com/mailman/listinfo/fedora-php-devel-list -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP Alias: php-pear-Auth https://bugzilla.redhat.com/show_bug.cgi?id=438804 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Status|NEW |ASSIGNED -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn||450366 -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP Alias: php-pear-Auth https://bugzilla.redhat.com/show_bug.cgi?id=438804 [EMAIL PROTECTED] changed: What|Removed |Added Alias||php-pear-Auth -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-06-06 10:58 EST --- Dave, please also investigate making a -radius subpackage as well which Requires Auth_Radius. -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-06-06 10:37 EST --- This is looking better. It should be a -2 release however, and you should document all your changes in the changelog section of the spec file. Please also do not forget to add a -samba subpackage which requires the SMBpasswd module (you can remove crypt-chap from the requires since this is redundant). -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-06-06 07:38 EST --- Updated package and spec now available: http://web.davehollis.com:81/packages/php-pear-Auth.spec http://web.davehollis.com:81/packages/php-pear-Auth-1.6.1-1.fc9.noarch.rpm A little bit of refactoring was needed to get the documentation and examples included properly. I'm not certain if that was a 1.5 -> 1.6 thing or if it actually wasn't quite right to begin with. Using the info from the wiki, I'm able to do the dos2unix thing on the doc file with a bit of sed magic and rpmlint has no issues. -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-06-02 16:08 EST --- Okay, it is probably best to require both DB and MDB2 since these packages don't bring in too many extras, and we want this to work out of the box for as many people as possible. Please update the spec to make a -samba subpackage. Also, change the Requires lines to look like: Requires: php-pear(File_Passwd) >= 1.1.0 Instead of: Requires: php-pear-File-Passwd >= 1.1.0 Is dos2unix brought in automatically? Please modify the spec file so that you accomplish the same results using only the tools listed here: https://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions Modifications of this type are better suited in the %prep section of the spec file. Please try to accomplish this there, or if not possible explain why. Shorten the %description section. The first paragraph is ample, and you don't want to keep updating it every time a new feature is added. -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-06-02 13:19 EST --- It looks like Crypt-CHAP may be responsible for a decent number of the requirements. That could get split out into the subpackage. As for DB vs MDB2, that would really be more of an issue as to what the developer is currently using for their DB. With DB being marked deprecated, that could be thrown into the subpackage as well with no problem. -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-06-02 11:24 EST --- Do you know which Requires brings in the most packages? Also, does it make sense to require both DB and MDB2? I will take a closer look at this over the next day or two. 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-06-02 10:17 EST --- Updated package and spec available at: http://web.davehollis.com:81/packages/php-pear-Auth.spec http://web.davehollis.com:81/packages/php-pear-Auth-1.6.1-1.fc9.noarch.rpm It now Requires: all optional elements that enhance functionality though this does mean that it pulls in a bunch of other packages to install and I can imagine some folks getting a little peeved at that. If it is a large enough issue, maybe a simple way to keep it from requiring 100 packages and to prevent 100 sub-packages, have php-pear-Auth include the most typical functionality - LDAP, MDB2 and maybe another one or two and then creating a sub-package that includes the somewhat more obscure containers and has more of the dependencies. -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-05-31 19:09 EST --- Dave I think you have not been following the discussion very closely. Nobody is asking you to package Kerberos, SAP, etc. Ofcouse this would be *impossible*. What we *do* need to do is add requirements for those packages which *are* in Fedora. Also, please note that we do not want separate packages for each requirement, please re-read comment #7 more carefully. -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-05-31 14:20 EST --- My only real concern with breaking out into subpackages is the additional dependencies that will be created that aren't currently in Fedora, some of which may never be. For example, the Kerberos, SAP, vpopmail and MDB (not MDB2) containers. The required plugins for these may FOSS, maybe not. In any case, I don't have the time nor personal need to package those pieces adequately for inclusion and if that holds up the package from being included for others, that doesn't seem like a great trade-off. The pieces that can be sub-packaged so as to satisfy their requirements can be handled easily, though those sub-pkgs will only have 1-2 files each but those containers that can't be resolved are the real issue for me. I can simply include them in the main package and it's left as an exercise for the user to fill in the missing pieces as required, or they can be neutered out of the package entirely and require end-users to make their own sub-packages or to pull in the missing files to satisfy their needs. Those two options seem rather silly to me. Whats the preference? -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-05-31 13:40 EST --- I was planning on sponsoring David, but I'm not getting any results. I've asked David to redo the spec file with additional requirements, but I've not seen anything yet. I suppose we can give it one more week, then just close out these bugs. Dave?? -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-05-30 18:56 EST --- Anything happening here? Chris, were you planning to sponsor David? Unfortunately some other tickets weren't properly blocking the NEEDSPONSOR ticket so I reviewed one of them and now it's in limbo. -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-05-12 07:37 EST --- I can be on IRC, though not at all times and I can't pay close attention to it. I'm slacker775 on freenode. I suppose for Auth plugins that have all necessary support within Fedora for their add-ons, splitting them out may be helpful, and simple enough. For ones that would require external add-ons like SAP, vpopmail, the easiest thing would be to just include them in the main package and let the end-user add the necessary dependencies as required. If we were to not package them, then our package becomes useless, or they have to rig up a hacked Auth package (php-pear-Auth-SAP) that only includes the SAP file(s) which starts to become stupid. -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-05-10 17:28 EST --- I wanted to go ahead and add requirements for all functionality of Auth so that it just works. For example, does Auth even work if there is not a DB package installed? Some requirements like Log we should add too because it adds a lot of extra functionality without bringing in too many extra packages. If we added Auth_Radius however, this might bring in too many extra packages and not benefit most users. So my thought was to split out the radius component into a separate package, which would require Auth_Radius which requires radius. I'm not sure if this is the best approach however, it might just be easier to assume someone using radius and Auth will also have Auth_Radius installed. Again, please let me know if you can communicate via IRC as this will make discussion *much* easier (see comment #4). 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-05-08 13:10 EST --- Does splitting this package up add any benefit? Some of the containers may not be shippable otherwise since they require external plugins that may or not may not be able to ship with Fedora (SAP, vpopmail, future additions). Is the goal for splitting out containers to ensure that the appropriate PHP extensions are required so that the end user can expect it to Just Work (like php-ldap for the LDAP container)? -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-05-05 13:21 EST --- ping? -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-04-07 12:51 EST --- Dave, can you access the #fedora-devel channel on the freenode IRC network? I prefer this form of communication for those I sponsor. http://fedoraproject.org/wiki/Communicate#IRC -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-04-07 08:26 EST --- I think the sub-packages would just include the specific container files and appropriate Requires: lines... For example, a php-pear-Auth-LDAP might have these specific bits: Requires: php-ldap %files /usr/share/pear/Auth/Container/LDAP.php (appropriate macros used of course) I'm not certain if it's terribly necessary to split up however, though it would likely be the only way to be able to ensure that a particular component would have the right bits installed without the main package requiring everything under the sun... -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 --- Additional Comments From [EMAIL PROTECTED] 2008-04-06 14:24 EST --- I was going to package this myself[1], there is some discussion on splitting this up into optional requirements[2]. What would need to be packaged in order to satisfy all optional components of this package? [1] http://fedoraproject.org/wiki/ChristopherStone#head-d7744c4c8ee8b851590aa038b984494254d0fda4 [2] http://fedoraproject.org/wiki/PackagingDrafts/PHP#head-d17f63e05f4104fca50f0862125e4cf8490591bf -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438804] Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Auth - provides methods for creating an authentication system using PHP https://bugzilla.redhat.com/show_bug.cgi?id=438804 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||177841 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2008-03-25 08:46 EST --- This is my first Fedora RPM and I do not currently have a sponsor. -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
