FLUID-3455: temp file cleanup could be moved into the php code (postProcessor.php) and removed from scripted cron job

Metadata

Source
FLUID-3455
Type
Bug
Priority
Major
Status
Closed
Resolution
Won't Fix
Assignee
N/A
Reporter
Laurel Williams
Created
2009-12-18T15:44:22.000-0500
Updated
2014-03-03T11:20:11.950-0500
Versions
  1. 1.4
Fixed Versions
  1. 1.5
Component
  1. Infusion Builder

Description

Currently, a shell script is used to clean up files in the tmp directory. This must be installed as a cron job. This task could be moved into the php code.

Attachments

Comments

  • Laurel Williams commented 2010-01-21T14:52:22.000-0500

    I've added a patch FLUID-3455-1.patch.txt which includes code to remove the temp directories. This needs to be tested on unix for two reasons.
    1) I hard coded "/" as the directory separator, as it didn't work to use DIRECTORY_SEPARATOR on windows, but "/" did work.
    2) I ran across a windows maximum file length limitation of 260 characters and could not delete directories which had files in it with that length or more in the total path name. So the function deleteDirectories does not fully work for a windows machine if the path names are extremely long. Infusion is pretty deep, so this can happen. But mostly we want this function to work on unix, so this may not matter.

  • Justin Obara commented 2014-03-03T11:20:11.949-0500

    builder.fluidproject.org has been taken down and there are no immediate plans to replace it. Currently all builds are expected to be generated directly from the grunt build scripts.

    http://fluid.2324889.n4.nabble.com/Infusion-Builder-issues-tp8926.html