Rob Sanheim
2008-Mar-17 04:10 UTC
small patch to make dreaded LoadErrors easier to track down
Could we get this small patch applied? http://dev.rubyonrails.org/ticket/11056 It adds a log warning to help track down the cause of the dreaded: "LoadError: Expected /foo.rb to define Foo" message. These errors can be frustrating to track down, as they are almost always related to missing dependencies or slight typos in constant names, and are probably total show stoppers for newbies. - Rob http://robsanheim.com http://thinkrelevance.com --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to rubyonrails-core@googlegroups.com To unsubscribe from this group, send email to rubyonrails-core-unsubscribe@googlegroups.com For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
Chad Woolley
2008-Mar-17 04:55 UTC
Re: small patch to make dreaded LoadErrors easier to track down
On Sun, Mar 16, 2008 at 9:10 PM, Rob Sanheim <rsanheim@gmail.com> wrote:> > Could we get this small patch applied? > > http://dev.rubyonrails.org/ticket/11056Added verified tag based on prior +1''s, it is in report 12 now... --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to rubyonrails-core@googlegroups.com To unsubscribe from this group, send email to rubyonrails-core-unsubscribe@googlegroups.com For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
Jeremy Kemper
2008-Mar-17 06:26 UTC
Re: small patch to make dreaded LoadErrors easier to track down
On Sun, Mar 16, 2008 at 9:55 PM, Chad Woolley <thewoolleyman@gmail.com> wrote:> On Sun, Mar 16, 2008 at 9:10 PM, Rob Sanheim <rsanheim@gmail.com> wrote: > > > > Could we get this small patch applied? > > > > http://dev.rubyonrails.org/ticket/11056 > > Added verified tag based on prior +1''s, it is in report 12 now...A general note on ticket verification: Please do not +1 if you did not actually verify the patch. That means applying it, testing Rails, and testing your app. Otherwise, three +1 would culminate in ''popular'', not ''verified'', and report 12 would no longer be a useful list of ready-to-commit patches. Thanks, jeremy --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to rubyonrails-core@googlegroups.com To unsubscribe from this group, send email to rubyonrails-core-unsubscribe@googlegroups.com For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---