Moodle

Availability of zip/unzip not checked during the installation

Details

  • Type: Sub-task Sub-task
  • Status: Closed Closed
  • Priority: Trivial Trivial
  • Resolution: Fixed
  • Affects Version/s: 1.9
  • Fix Version/s: 2.0
  • Component/s: Installation
  • Labels:
    None
  • Environment:
    Linux, Debian Etch
  • Affected Branches:
    MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_20_STABLE

Description

Availability of zip/unzip not checked during the installation, but it's required when making backup.

Issue Links

Activity

Hide
Dan Marsden added a comment -

in admin > server > system paths >you can specify an external zip library when using linux based servers.... so Zip is only "required" for windows based installs. - might be worthwhile displaying a warning though?

Dan

Show
Dan Marsden added a comment - in admin > server > system paths >you can specify an external zip library when using linux based servers.... so Zip is only "required" for windows based installs. - might be worthwhile displaying a warning though? Dan
Hide
Eloy Lafuente (stronk7) added a comment -

But availability of zip/unzip isn't ever required to be checked because those libraries are bundled with Moodle, so all Moodle servers are able to zip/unzip contents.

Another thing is the "extra" feature of being able to use zip & unzip binaries.

But, in any case, I cannot find any reason to check for zip support in installer. Moodle provides it internally.

Or am I missing anything?

Show
Eloy Lafuente (stronk7) added a comment - But availability of zip/unzip isn't ever required to be checked because those libraries are bundled with Moodle, so all Moodle servers are able to zip/unzip contents. Another thing is the "extra" feature of being able to use zip & unzip binaries. But, in any case, I cannot find any reason to check for zip support in installer. Moodle provides it internally. Or am I missing anything?
Hide
Petr Škoda (skodak) added a comment -

pclzip needs zlib extension - this should be in standard distributions, but unfortunately some allow you to remove it

Show
Petr Škoda (skodak) added a comment - pclzip needs zlib extension - this should be in standard distributions, but unfortunately some allow you to remove it
Hide
Petr Škoda (skodak) added a comment -

fixed now in new 2.0 installer, thanks

Show
Petr Škoda (skodak) added a comment - fixed now in new 2.0 installer, thanks

Dates

  • Created:
    Updated:
    Resolved: