Uploaded image for project: 'Islandora'
  1. Islandora
  2. ISLANDORA-1478

Audio Solution pack without LAME installed



    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Done
    • Fix Version/s: 7.x-1.7
    • Component/s: Audio Solution Pack
    • Labels:


      This enhancement is for work that has already been produced. If the design is not desirable by the solution pack owners, we will redesign and reimplement as they see fit. The goal is to contribute code that solves the issue below.

      Currently the audio solution pack requires that LAME is installed. The main issue faced by someone trying to use the solution pack without LAME is that the drupal hook_requirements method of the solution pack uses REQUIREMENT_WARNING which will not allow normal operation of drupal's update.php to update the database - requiring an override. The minor issue is that messages are sent to the watchdog log on every audio ingest.

      In this case, someone is not able to install LAME on the Drupal server, but is able to generate the PROXY_MP3 as a derivative to store in Fedora. The desired result is that the Drupal administrator should be able to install the module, use update.php, and the user would be able to use the player that the solution pack uses.

      This work has already been done in the following way, but my customer (Smithsonian Institution) is open to redesign and rework by the company I work for:

      included a string constant to indicate that the LAME encoder should not be run
      if executable path is set to this string value, no attempt to run LAME or message to the watchdog log on audio ingest includes/admin.form.inc:
      include text on form for what should be used in the field that points to the executable to indicate LAME is not installed islandora_audio.install:
      if the executable path is set to the string constant, issue REQUIREMENT_INFO as severity instead of REQUIREMENT_WARNING




            • Assignee:
              randersonquotientinc Robert Anderson
              randersonquotientinc Robert Anderson
            • Votes:
              0 Vote for this issue
              3 Start watching this issue


              • Created: