Project

General

Profile

Actions

Patch #1892

closed

Patch boot.rb to allow for symlinks in RAILS_ROOT

Added by Richard Hurt about 16 years ago. Updated about 16 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
2008-09-12
Due date:
% Done:

0%

Estimated time:
1.00 h

Description

The .../config/boot.rb file that ships with Redmine does not work well with symlinked directories. Specifically, during packaging for Debian I needed to replace .../vendors/rails with the Debian equivalent. Symlinking .../vendor/rails -> /usr/share/rails broke Redmine so that it wouldn't find the Rails install at all.

The attached patch uses the expand_path() function on the File object to pre-expand the symlinks so they don't cause problems later. This shouldn't affect anything else although I have only tested it on Debian Lenny, not Windows or any other flavor of Unix. If there is a better way to do this then please let me know and I'll update my package.

Thanx!
Richard


Files

boot.rb.patch (353 Bytes) boot.rb.patch Richard Hurt, 2008-09-12 21:38
Actions #1

Updated by Richard Hurt about 16 years ago

This patch is related to issue #1750.

Actions #2

Updated by Jean-Philippe Lang about 16 years ago

  • Status changed from New to Closed

It doesn't seem to affect anything on Windows and I can see that Radiant did the same (http://github.com/radiant/radiant/tree/master/config/boot.rb).
Patch committed in r1820.

Since boot.rb is a standard Rails file (no Redmine specific), I suggest you to report this problem to the Rails team.

Actions #3

Updated by Richard Hurt about 16 years ago

I've commented on the update in the Rails VCS but am not sure about opening a ticket just yet. It looks like they purposefully added the expand_path() to the Windows code but not the Unix code. I'm trying to get more info about why they decided to do that. Maybe there is something going on that we don't know about and I just want to cover all the bases. :)

Thanx!
Richard

Actions

Also available in: Atom PDF