phpldapadmin should optionally depend on php5-cgi

Bug #126746 reported by Dulmandakh Sukhbaatar
6
Affects Status Importance Assigned to Milestone
phpldapadmin (Ubuntu)
Fix Released
Undecided
Canonical Server

Bug Description

phpldapadmin trying to install libapache2-mod-php5 when I'm configured web server to run php apps as cgi using php5-cgi. So let's add php5-cgi as dependency to phpldapadmin.

it was on feisty, lighttpd, php5-cgi

Changed in phpldapadmin:
assignee: nobody → motu
Revision history for this message
YannTech (yanntech) wrote :

Seems problem when you want suppress phpldapadmin.
Indeed when you make : apt-get remove phpldapadmin the package search to reload apache2 but with lighttpd ...

Revision history for this message
Daniel Hahler (blueyed) wrote :

YannTech, this seems to be another bug: what you want is that lighttpd gets reloaded instead of apache, right? Please file a new bug report for it.

About the Depends itself: I'll provide a debdiff to include php5-cgi there, optionally, too. Also, I'll remove the php4 packages then, because PHP4 is not supported in Ubuntu anymore.

Changed in phpldapadmin:
assignee: motu → blueyed
status: New → In Progress
Revision history for this message
YannTech (yanntech) wrote :

dAniel,

I don't know if this is a new bug but phpldapadmin think apache2 is present on the computer and reload apache2 on suppress request (apt-get remove).
If the new package don't include libapache2-mod-php5 in dependencies, why he try reload apache on suppress ?

Revision history for this message
Daniel Hahler (blueyed) wrote :

@YannTech:
According to the debian/postrm script, which gets executed when removing the package, those webservers get restarted where you've said to install support for them when installing the package.
So this does not appear to be really a bug, especially since this checks if the webserver init script is existing and executable. At least with Ubuntu Gutsy.
If you have a bug report about it, _please open a new bug report_.

Daniel Hahler (blueyed)
Changed in phpldapadmin:
assignee: blueyed → nobody
Daniel Hahler (blueyed)
Changed in phpldapadmin:
status: In Progress → Confirmed
Daniel Hahler (blueyed)
Changed in phpldapadmin:
status: Confirmed → Fix Committed
Revision history for this message
Daniel Holbach (dholbach) wrote :

Rick: can your or somebody else of the server team check out the patch?

Changed in phpldapadmin:
assignee: nobody → dendrobates
Changed in phpldapadmin:
assignee: dendrobates → ubuntu-server
Changed in phpldapadmin:
assignee: ubuntu-server → canonical-server
Revision history for this message
Mathias Gug (mathiaz) wrote :

Thanks for your patch.

Could you specify which file has been modified in the changelog entry ?

Changed in phpldapadmin:
status: Fix Committed → In Progress
Revision history for this message
Daniel Hahler (blueyed) wrote :

Sure, here's the debdiff with the file references.

Revision history for this message
Daniel Hahler (blueyed) wrote :

phpldapadmin (0.9.8.4-1ubuntu1) gutsy; urgency=low

  * debian/control: Optionally depend on php5-cgi (think lighttpd)
    (LP: #126746)
  * debian/control: Remove Depends on php4 packages, because they are not
    supported in Ubuntu anymore.

 -- dAniel hAhler <email address hidden> Tue, 11 Sep 2007 01:01:33 +0200

Changed in phpldapadmin:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.