The extract function uses gunzip regardless of whether the archive is .tar or .tar.gz. The backup function does check if "gz" is specified, and creates a .tar if it is absent.

See patch attached. Also I made a module to make testing this easier: https://drupal.org/sandbox/gboudrias/2169019

Blocking http://drupal.org/node/2154803

CommentFileSizeAuthor
provision-non_gzipped_backups_cant_be_restored.patch1005 bytesgboudrias
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic commented

This looks like it should work. Can you post a drush snippet to test it?

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic commented

nevermind, I missed the module you'd posted.

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic commented
Status: Needs review » Reviewed & tested by the community

This works quite nicely.

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic commented
Status: Reviewed & tested by the community » Fixed

Fixed in 5efc742 (for 7.x-3.x) and 8bdada6 (for 6.x-2.x).

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

  • Commit 5efc742 on 7.x-3.x, dev-helmo-3.x authored by gboudrias, committed by ergonlogic: 
    Issue #2169025 by gboudrias: Fix non-gzipped backups can't be restored.