#728 closed defect (fixed)
sendfile.setup will probably not work on windows
Reported by: | Mario Domenech Goulart | Owned by: | certainty |
---|---|---|---|
Priority: | major | Milestone: | 4.9.0 |
Component: | extensions | Version: | 4.7.x |
Keywords: | sendfile | Cc: | |
Estimated difficulty: |
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).
Change History (3)
comment:1 Changed 13 years ago by
Status: | new → accepted |
---|
comment:2 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
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.
Note: See
TracTickets for help on using
tickets.
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