Ticket #128 (closed defect: fixed)
Everyone is Here in the Future, but it breaks the reloader.
| Reported by: | binarywaterfall | Owned by: | why |
|---|---|---|---|
| Priority: | major | Milestone: | |
| Component: | bin/camping | Version: | 1.5 |
| Keywords: | Cc: |
Description (last modified by binarywaterfall) (diff)
Hi,
This is a strange problem to have, but it's easy to fix. I've got a Camping app that does system time dependent reports, and in testing it, I had a few mtimes in the future. I think it's as easy as adding a .reject{ |t| t > Time.now } just before the max in this code in the reloader:
86 # The timestamp of the most recently modified app dependency.
87 def mtime
88 ((@requires || []) + [@script]).map do |fname|
89 fname = fname.gsub(/^#{Regexp::quote File.dirname(@script)}\//, '')
90 begin
91 File.mtime(File.join(File.dirname(@script), fname))
92 rescue Errno::ENOENT
93 remove_app
94 @mtime
95 end
96 end.max
97 end
I blogged about it, too.
Change History
Note: See
TracTickets for help on using
tickets.
