Ticket #128 (closed defect: fixed)

Opened 19 months ago

Last modified 14 months ago

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

Changed 19 months ago by binarywaterfall

  • description modified (diff)

Changed 19 months ago by binarywaterfall

  • description modified (diff)

Changed 19 months ago by binarywaterfall

  • description modified (diff)

Changed 14 months ago by zimbatm

Fixed in [208]. Thanks for you patience :)

Changed 14 months ago by zimbatm

  • status changed from new to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.