search

asenchi / scrolls

Simple logging
MIT License
158 stars 26 forks source link

Question: Reason for not using the atomic gem? #18

Closed eric closed 12 years ago

eric commented 12 years ago

I noticed you referenced the atomic gem. Is there a reason for "faking" it with a mutex instead of using the gem itself?

asenchi commented 12 years ago

@eric At the time @heroku didn't want any dependencies due to the fact that we shipped this library around so many places (nagios event handlers is no place for bundler :)). We could probably setup a conditional to use atomic if the gem is there, but I'd like to avoid adding any requirements.

heroku commented 12 years ago

In replies all text above this line is added to the ticket

[Heroku] Request received: Re: [scrolls] Question: Reason for not using the atomic gem? (#18): @eric At the time @heroku... (ticket #59468)

Your request (#59468) has been received, and is being reviewed by our support staff.

To review the status of the request and add additional comments, follow the link below: https://support.heroku.com/tickets/59468

asenchi/scrolls, Aug 07 20:59 (PDT)

Re: [scrolls] Question: Reason for not using the atomic gem? (#18): @eric At the time @heroku didn't want any dependencies due to the fact that we shipped this library around so many places (nagios event handlers is no place for bundler :)). We could probably setup a conditional to use atomic if the gem is there, but I'd like to avoid adding any requirements.

Reply to this email directly or view it on GitHub: https://github.com/asenchi/scrolls/issues/18#issuecomment-7574412

This email is a service from Heroku Support

asenchi commented 12 years ago

Kind of silly that Heroku opened a ticket simply via mention on GitHub. Oh well, closing this out for now.