-
Notifications
You must be signed in to change notification settings - Fork 60
PEAR_Exception Class Not found #68
Comments
Yep, indeed. Will also meditate on what is the best way to include it :) PR is always welcome! |
One way is to just get Exception stuff from github with package type zip. {
"type": "package",
"package": {
"name": "pear/Pear_Exception",
"version": "dev-master",
"dist": {
"url": "https://github.com/pear/PEAR_Exception/archive/master.zip",
"type": "zip"
}
}
} then Include require: "pear/Pear_Exception": "dev-master" All it needs to do is to be autoloaded correctly. |
Seems quite a good solution to me. Pls, feel free to file the pull request. |
It is Console_CommandLine bug. I'm currently fixing phrozn-related pear repositories and it will be enough just to have Console_CommandLine in Phrozn/composer.json. Not sure how long it will take until they'll accept all my PRs. @farazdagi This would be temporary fix. I would suggest to wait until PEAR repos will be fixed and do not include this PR. |
@Pawka That's probably the best way to fix this. |
@Pawka @farrelley Agreed, let's wait and see when PEAR component is fixed. |
hey @farrelley, would you mind pasting your composer.json down here? wasn't able to get it working tks! |
@fgrehm - #68 (comment) I just added that. Just need to add in the autoloading to psr-0 section. That's the work around. |
@farralley tks for the answer but I wasn't able to get it working |
@fgrehm Add |
The PEAR_Exception class is not found. It need to be added to the composer file somehow. It's extended through Console_CommandLine_Exception
PEAR_Exception GitHub repo is https://github.com/pear/PEAR_Exception/
Will issue PR if I can figure out how to get this in.
The text was updated successfully, but these errors were encountered: