This project is archived and is in readonly mode.
[PATCH] Changes in autoload and some minor fixes
Reported by Łukasz Strzałkowski | August 11th, 2010 @ 09:47 PM | in 3.1
Hello!
I have removed autoloading exceptions classes, and set explict require for them where needed. Also squashed some warnings.
Changes are in my branch on github (strzalek/rails):
http://github.com/strzalek/rails/compare/rails:master...master
Comments and changes to this ticket
-
Łukasz Strzałkowski August 12th, 2010 @ 07:19 AM
- Title changed from Changes in autoload and some minor fixes to [PATCH] Changes in autoload and some minor fixes
- Tag set to autoload, patch
-
José Valim August 12th, 2010 @ 01:16 PM
- Milestone set to 3.1
- Importance changed from to Low
I will apply your changes later when we release Rails 3 final to avoid making big changes in the code now.
-
Łukasz Strzałkowski August 28th, 2010 @ 11:28 AM
I did some cleaning on my branch recently. All changes for this ticket you can now find under branch autoload in my fork: http://github.com/strzalek/rails/commits/autoload
-
José Valim September 2nd, 2010 @ 10:54 AM
- State changed from new to resolved
Applied, except the one which remove a few errors since they are probably used by plugins/extensions.
-
Łukasz Strzałkowski September 2nd, 2010 @ 03:12 PM
Thanks.
I didn't catch this becouse I wasn't even looking after this. Thought that if it is not used in core than no body care about it ;-)
-
José Valim September 2nd, 2010 @ 05:21 PM
Your changes broke the booting of a new rails 3 app. Could you please
investigate or should I revert? -
Łukasz Strzałkowski September 2nd, 2010 @ 07:10 PM
Jose,
Sorry for that. When I was doing this change, I looked carefuly on tests and tried some sample app. Everything was okay.
I've just pulled the newest revision of master, and fired it up with new, clean, rails 3 app without any problem.
Please revert this if it's breaking booting process and if you can provide some feedback how to reproduce it, along with what error is raised during the booting I would be thankful.
Sorry about that one more time. I will try to fix it as soon as I reprodcue it, for now I think it should be reverted as it huge blocker.
-
Łukasz Strzałkowski September 2nd, 2010 @ 07:15 PM
Ok, I've found the error, I'll try fix it quickly.
-
Łukasz Strzałkowski September 2nd, 2010 @ 08:01 PM
Please revert it for now.
I have some freaking problems with gems and rvm localy and I'm unable to fix it now.
-
Repository September 2nd, 2010 @ 08:11 PM
- State changed from resolved to open
(from [599e46bf24419a429ee0bb0ea02afd48f47aa4ec]) Revert "Setup explicit requires for files with exceptions. Removed them from autoloading."
Booting a new Rails application does not work after this commit [#5359 state:open]
This reverts commit 38a421b34d0b414564e919f67d339fac067a56e6.
http://github.com/rails/rails/commit/599e46bf24419a429ee0bb0ea02afd... -
Santiago Pastorino February 2nd, 2011 @ 04:49 PM
This issue has been automatically marked as stale because it has not been commented on for at least three months.
The resources of the Rails core team are limited, and so we are asking for your help. If you can still reproduce this error on the 3-0-stable branch or on master, please reply with all of the information you have about it and add "[state:open]" to your comment. This will reopen the ticket for review. Likewise, if you feel that this is a very important feature for Rails to include, please reply with your explanation so we can consider it.
Thank you for all your contributions, and we hope you will understand this step to focus our efforts where they are most helpful.
-
Santiago Pastorino February 2nd, 2011 @ 04:49 PM
- State changed from open to stale
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>
People watching this ticket
Referenced by
- 5359 [PATCH] Changes in autoload and some minor fixes Booting a new Rails application does not work after this ...