fixed 2.4.6 spec file #4

Closed
wants to merge 6 commits into
from

Conversation

Projects
None yet
1 participant
@wetnun

wetnun commented Jan 16, 2012

Did some minor changes so you just have to download the tar.gz to your SOURCES folder and do rpmbuild -ba.

@ghost

This comment has been minimized.

Show comment Hide comment
@ghost

ghost Jan 16, 2012

Thanks for the pull request. We'll test it out and merge.

ghost commented Jan 16, 2012

Thanks for the pull request. We'll test it out and merge.

@wetnun

This comment has been minimized.

Show comment Hide comment
@wetnun

wetnun Jan 16, 2012

No problem. I needed a quick spec file and yours was a good base. My service start is failing though, permissions error on /var/log. I was going to look at that when I get back to the office.

Jon

Jon

Sent from my Droid3. Please excuse my speeling.

Greg Hurrell reply@reply.github.com wrote:

Thanks for the pull request. We'll test it out and merge.


Reply to this email directly or view it on GitHub:
#4 (comment)

wetnun commented Jan 16, 2012

No problem. I needed a quick spec file and yours was a good base. My service start is failing though, permissions error on /var/log. I was going to look at that when I get back to the office.

Jon

Jon

Sent from my Droid3. Please excuse my speeling.

Greg Hurrell reply@reply.github.com wrote:

Thanks for the pull request. We'll test it out and merge.


Reply to this email directly or view it on GitHub:
#4 (comment)

@wetnun

This comment has been minimized.

Show comment Hide comment
@wetnun

wetnun Jan 17, 2012

So since I'm back in the office and got more testing done, I had to make a couple more minor changes. My log is now in /var/redis/redis.log (user redis has no permissions to /var/log/ by default), and I make the /var/redis directory in my %pre as well as start redis automatically. I also update the conf to be the recent version since it will throw errors on start with the old one. Changes are pretty small, so feel free to take whatever you need. :)

wetnun commented Jan 17, 2012

So since I'm back in the office and got more testing done, I had to make a couple more minor changes. My log is now in /var/redis/redis.log (user redis has no permissions to /var/log/ by default), and I make the /var/redis directory in my %pre as well as start redis automatically. I also update the conf to be the recent version since it will throw errors on start with the old one. Changes are pretty small, so feel free to take whatever you need. :)

@wetnun

This comment has been minimized.

Show comment Hide comment
@wetnun

wetnun Jan 17, 2012

I rebased my stuff after making some more minor changes. Going to close this pull request and make a new one. :/

wetnun commented Jan 17, 2012

I rebased my stuff after making some more minor changes. Going to close this pull request and make a new one. :/

@wetnun wetnun closed this Jan 17, 2012

@ghost

This comment has been minimized.

Show comment Hide comment
@ghost

ghost Jan 17, 2012

Great, thanks.

ghost commented Jan 17, 2012

Great, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment