patch attached - this allows drush to deal with non-compressed tar files.

CommentFileSizeAuthor
#4 drush_make_download_tar_revised2.patch1.17 KBacrollet
#2 drush_make_download_tar_revised.patch1.1 KBacrollet
drush_make.download_tar.patch1.19 KBacrollet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dmitrig01’s picture

dmitrig01 CreditAttribution: dmitrig01 commented
Status: Needs review » Needs work

wouldn't a .gz file be grouped with tgz? I think it should be

case 'tar'
X
case 'gz'
case 'tgz'
Y
case 'zip'
Z

acrollet’s picture

acrollet CreditAttribution: acrollet commented
Status: Needs work » Needs review
FileSize
drush_make_download_tar_revised.patch1.1 KB

absolutely it would - brainfart! fixed version attached.

dmitrig01’s picture

dmitrig01 CreditAttribution: dmitrig01 commented
Status: Needs review » Needs work

what's this part all about?

+    drush_set_error(dt('Could not retrieve package information for @filename.', array('@filename' => $filename)));
+    return;
acrollet’s picture

acrollet CreditAttribution: acrollet commented
Status: Needs work » Needs review
FileSize
drush_make_download_tar_revised2.patch1.17 KB

good question!

dmitrig01’s picture

dmitrig01 CreditAttribution: dmitrig01 commented
Status: Needs review » Fixed

Committed - thanks!

Status: Fixed » Closed (fixed)

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