Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Custom temp folder #147

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

tommyblue
Copy link

With these changes the user can select a different temp folder to manage the gitolite repository. If this custom option is ignored, the old behaviour still works and the plugin uses Dir.tmpdir.
In my case this patch is mandatory because the /tmp folder in my server is mounted with the noexec flag and the plugin doesn't work (it causes a 50x error).

Another little fix let this plugin work with ruby 1.9.3

Signed-off-by: Tommaso Visconti <[email protected]>
Signed-off-by: Tommaso Visconti <[email protected]>
Signed-off-by: Tommaso Visconti <[email protected]>
Signed-off-by: Tommaso Visconti <[email protected]>
Signed-off-by: Tommaso Visconti <[email protected]>
@kubitron
Copy link

kubitron commented Mar 5, 2012

Hello. I am not sure whether anyone is answering issues on this list any more.

However, I've the latest commit on my version of the plugin (https://github.com/kubitron/redmine_git_hosting) has configurable paths for scripts and temporary files.

Two things to note:

(1) The scripts are separated out from data (there are two path settings, one for data and one for scripts),

(2) The default location for scripts in my version of the plugin is the ./bin directory under the plugin root. You can move them wherever you like, however. (The README has a lot of info about handling the scripts).

Note that my version of the plugin has a bunch of other advantages (and fixes). If you choose to use my version of the plugin, I will try to answer issues questions promptly...

p.s. My version also fixes the syntax errors you identified for compatibility with 1.9.3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants