Feature #7223
openMake SCM Error logging compatible with TorqueBox
0%
Description
I am currently working on integrating Redmine into my client's environment using TorqueBox and was experiencing some difficulty integrating with SVN. Upon further inspection, I noticed that all the internal SCM commands were failing while the rake fetch_changeset command was successful.
Turns out that TorqueBox was prepending "vfs:/" to RAILS_ROOT. Since I was running in dev, it was redirecting the errors to '2>>#{RAILS_ROOT}/log/scm.stderr.log'. This didn't work and the shell execution failed. Since the SVN command didn't even execute the error handling never properly caught it and it died later with a NPE.
This was easy enough to work around but I think that the logger.debug statement should be moved to line 189 (after the error appending block). You may also want to consider additional error handling for other obscure failures which prevent the shell from properly executing.
Updated by Mike Battles about 12 years ago
Forgot to mention I was referring to lib/redmine/scm/adapters/abstract_adapter.rb
Updated by Jean-Philippe Lang about 12 years ago
- Tracker changed from Defect to Feature
- Subject changed from SCM Error logging incompatible with TorqueBox to Make SCM Error logging compatible with TorqueBox