Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.10.12

Bug #2738 Deps on XML package should be optional
Submitted: 2004-11-11 10:40 UTC
From: dufuz Assigned: ryansking
Status: Closed Package: Config
PHP Version: Irrelevant OS: N/A
Roadmaps: (Not assigned)    
Subscription  


 [2004-11-11 10:40 UTC] dufuz
Description: ------------ The dep on those XML packages should be optional since they're only needed with some types of configs.

Comments

 [2004-11-13 04:06 UTC] ryansking at mac dot com
Please be patient with me, I'm a new maintainer to the Config package. You make a good point, I think. I'll look into doing this for the next release. However, don't expect to many more releases of Config 1. I'm hoping to start working on Config2 soon (for php5).
 [2004-11-13 13:01 UTC] dufuz
Well this was just little act of QA (even though I'm not member of that team ;P) just change these lines in package.xml: <dep type="pkg" rel="has">XML_Parser</dep> <dep type="pkg" rel="has">XML_Util</dep> To: <dep type="pkg" rel="has" optional="yes">XML_Parser</dep> <dep type="pkg" rel="has" optional="yes">XML_Util</dep> And then you'll be fine, and everything will be randy dandy in the next release :)
 [2004-11-13 17:08 UTC] ryansking at mac dot com
I will do this. However, I'm not going to make a special release for this, I'm going to wait until I get a couple more bugs knocked out.
 [2004-11-13 17:13 UTC] dufuz
Didn't expect a special release for this ;) Anyway I recommend that you commit this change and move this bug to close (Use quick fix: Fixed in CVS or something similar) :)
 [2004-11-13 17:43 UTC] ryansking at mac dot com
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better.