Please merge phpldapadmin 1.1.0.7-1 (universe) from Debian unstable (universe)

Bug #376766 reported by Adam Sommer
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
phpldapadmin (Ubuntu)
Fix Released
Wishlist
Siegfried Gevatter

Bug Description

Binary package hint: phpldapadmin

  * New upstream release.
  * debian/po/es.po: added. (Closes: #520504)

Adam Sommer (asommer)
Changed in phpldapadmin (Ubuntu):
assignee: nobody → Adam Sommer (asommer)
importance: Undecided → Wishlist
status: New → In Progress
Revision history for this message
Adam Sommer (asommer) wrote :

I think this the appropriate debdiff, but if not or I need to make additional changes please let me know.

Thanks,
Adam

Changed in phpldapadmin (Ubuntu):
assignee: Adam Sommer (asommer) → nobody
status: In Progress → Confirmed
Revision history for this message
Siegfried Gevatter (rainct) wrote :

Thank you for your patch. Here are some problems I see with it:

You mention "Removed the "-m 644" because files are already set to 644." in debian/changelog but this is already done in the latest Debian revision; you also fail to mention the "php_value memory_limit 32M" change. Additionally, bug 223842 and perhaps also bug 333620 will be fixed with this upload, so it wouldn't hurt to mention them in the changelog.

Further, are you sure you want all the changes in the locale/ directory?

Changed in phpldapadmin (Ubuntu):
assignee: nobody → Adam Sommer (asommer)
status: Confirmed → In Progress
Revision history for this message
Yonas (yonas-y) wrote :

Hey guys,

This is my first time installing phpldapadmin. Seeing that 1.1.0.5 is more than a year old, I was about to create this bug report, but Adam beat me to it by 2 days!! o_O

Cheers,
Yonas

Revision history for this message
Adam Sommer (asommer) wrote :

Thanks Siegfried for reviewing my patch. Here's a better one that I think addresses the issues you brought up.

If there are additional changes needed please let me know.

Changed in phpldapadmin (Ubuntu):
assignee: Adam Sommer (asommer) → Siegfried Gevatter (rainct)
Revision history for this message
Siegfried Gevatter (rainct) wrote :

Uploaded, thanks.

Changed in phpldapadmin (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package phpldapadmin - 1.1.0.7-1ubuntu1

---------------
phpldapadmin (1.1.0.7-1ubuntu1) karmic; urgency=low

  [ Adam Sommer ]
  * Merge from debian unstable, remaining changes (LP: #376766):
    - Merged call to dh_install to install debian/additional-templates/*
    - Adds php_value memory_limit 32M to the apache.conf.
  * Configuration error fix. (LP: #333620)
  * Alias objects now work. (LP: #223842)

phpldapadmin (1.1.0.7-1) unstable; urgency=low

  * New upstream release.
  * debian/po/es.po: added. (Closes: #520504)

phpldapadmin (1.1.0.6-1) unstable; urgency=low

  * New upstream release. (Closes: #518578)
  * debian/rules: removed "-m 644" from the dh_install call. (Closes: #518847)
  * debian/postrm: remove config.php at purge time. (Closes: #519086)
  * debian/patches/hungarian.dpatch: fixed a hungarian translation.
    (Closes: #505559)

 -- Adam Sommer <email address hidden> Thu, 14 May 2009 22:05:53 -0400

Changed in phpldapadmin (Ubuntu):
status: Fix Committed → 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.