FirePHP's license is GPL compatible, which means we can ship FirePHP with Devel. We should either do that, or allow FirePHP to load from sites/all/libraries/FirePHP.

Thanks!

Comments

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Status: Active » Postponed (maintainer needs more info)

No, it doesn't mean that. See http://drupal.org/node/422996

Further Library module integration is welcome. There is some there already. Please reopen with more specifics about how it should work.

chOP’s picture

chOP CreditAttribution: chOP commented

Any implementation of this should support firephp installation via pear. Add it to libraries by all means, but if it exists already in the PHP pear library, use that instead.

RobLoach’s picture

RobLoach
he/him
CreditAttribution: RobLoach commented

I've also submitted a patch for Composer support: https://github.com/firephp/firephp-core/pull/6 .

chOP’s picture

chOP CreditAttribution: chOP commented

I've added this issue #1718328: Use FirePHP installed from PEAR if available to devel 7.x-1.x to have it use the PEAR installation of FirePHP if available. Otherwise, it already copies FirePHP to the Drupal libraries path. Please try out the patch and see if it works for you.

willzyx’s picture

willzyx CreditAttribution: willzyx commented
Issue summary: View changes

Closing for lack of activity. Feel free to reopen if the issue still exists

willzyx’s picture

willzyx CreditAttribution: willzyx commented
Status: Postponed (maintainer needs more info) » Closed (won't fix)