Summary

sendfile.setup will probably not work on windows

Metadata

• Id: d610613921b200adc4322fee712e4be97232f2b1
• Trac id: 728
• Type: defect
• Reporter: mario
• Owner: certainty
• Cc:
• Status: closed
• Component: extensions
• Estimated difficulty:
• Resolution: fixed
• Priority: major
• Milestone: 4.9.0
• Version: 4.7.x
• Changetime: 2012-09-24 21:47:48 UTC
• Created: 2011-11-17 13:07:18 UTC
• Keywords: sendfile

Description

The latest lines from sendfile.setup are

 ;; cleanup temp-files
 ;; remove this in production
 (system "rm *.o")

I suspect that it'll break the sendfile installation on systems that have no rm (e.g., Windows).

Changes and comments

[2011-11-18 18:31:47 UTC] certainty changed status from new to accepted

[2011-11-18 18:31:47 UTC] certainty wrote:

Thanks Mario. You are very right. I put it there as a quick fix, that was not supposed to last that long. I'll take care of it

[2011-11-19 07:54:58 UTC] certainty changed status from accepted to closed

[2011-11-19 07:54:58 UTC] certainty set resolution to fixed

[2011-11-19 07:54:58 UTC] certainty wrote:

I've just removed this line, as it's easier to just do it yourself if you don't like it than finding a crossplatform way (which is hard for me to test without a windows) of removing the object files.

[2012-09-24 21:47:48 UTC] felix changed milestone from 4.8.0 to 4.9.0

[2012-09-24 21:47:48 UTC] felix wrote:

Milestone 4.8.0 deleted