[Patch][Precise]Enable LZO support

Bug #925315 reported by David Weber
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
bacula (Ubuntu)
Fix Released
Wishlist
James Page

Bug Description

"LZO provides much faster compression and decompression speed but lower compression ratio than GZIP." [1]
It would be very nice to enable this in precise
You can find a patched version in my ppa[2]

Diff attached

Thanks!

[1] http://bacula.org/5.2.x-manuals/en/main/main/New_Features_in_5_2_1.html
[2] https://code.launchpad.net/~wb-munzinger/+recipe/bacula-daily

Tags: patch

Related branches

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "Enable LZO" of this bug report has been identified as being a patch in the form of a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-sponsors team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Great idea David. The patch needs a little work:

First, reference this bug# in the changelog entry as (LP: #925315)

Second, you do not need to pre-depend on the library. It will be linked in and automatically added to Depends: by ${shlibs:Depends}. Pre-Depends are only things that are needed to run the preinst maintainer script, which you definitely don't need here.

Third, the version should be 5.2.3-0ubuntu3 instead of 5.2.3-0ubuntud. You would have gotten that automatically if you had run 'dch -i' to edit the changelog.

Once you've uploaded an updated patch, make sure to subscribe ubuntu-sponsors to this bug, and somebody will take a look at the patch ASAP.

Finally, THANKS!

Changed in bacula (Ubuntu):
status: New → Triaged
importance: Undecided → Wishlist
Revision history for this message
David Weber (wb-munzinger) wrote :

Thanks for your kind help!

I hope I've done this time everything right :)

James Page (james-page)
Changed in bacula (Ubuntu):
assignee: nobody → James Page (james-page)
status: Triaged → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package bacula - 5.2.5-0ubuntu2

---------------
bacula (5.2.5-0ubuntu2) precise; urgency=low

  * Enable LZO support (LP: #925315)
 -- David Weber <email address hidden> Mon, 06 Feb 2012 09:33:25 +0100

Changed in bacula (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
James Page (james-page) wrote :

Looks good; uploaded!

Thanks for you contribution!

Revision history for this message
Bart Verwilst (verwilst) wrote :

I'm not sure what's the matter, but trying to enable compression = LZO gives me the error "Fatal error: Compression deflateParams error: -2" when trying to do a backup. Switching back to GZIP fixes it. 5.2.5-0ubuntu6

Revision history for this message
David Weber (wb-munzinger) wrote :

Both the client and the server should have a version installed which has LZO enabled. Is this the case? If it is, please create a new bug. You can cc me and I will take a look into it.
LZO compression works fine for Terabytes of data for me.

Revision history for this message
Bart Verwilst (verwilst) wrote :

Confirmed to work with both client and server on this version, my bad!

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.