Skip to content
This repository was archived by the owner on Dec 21, 2022. It is now read-only.

Cleanup Redis #25

Open
janfabry wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Cleanup Redis #25

janfabry wants to merge 1 commit into from

Conversation

@janfabry
Copy link
Contributor

@janfabry janfabry commented Nov 14, 2013

  • Update download URL
  • Compile in src/ dir
  • Create separate conf_dir and log_dir
  • No need to touch logfile, just chown dir
  • Remove duplicate methods (incorrect copy-paste)

@janfabry
Cleanup Redis
4714849 
 - Update download URL
 - Compile in src/ dir
 - Create separate conf_dir and log_dir
 - No need to touch logfile, just chown dir
 - Remove duplicate methods (incorrect copy-paste)
@jonathanslenders
Copy link
Owner

jonathanslenders commented Nov 24, 2013

Can't be merged. But I'm not going to merge this anyway. I'd like you to copy the Redis node to our own code base. It shouldn't be in the main repository. We can open source it maybe later on as deploy_redis or something like that.

@jonathanslenders jonathanslenders force-pushed the master branch 2 times, most recently from cf54f50 to 32c1030 Compare October 21, 2014 08:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@janfabry @jonathanslenders