sbrinkmann/rc522-rfid

Some remarks

Opened this issue · 4 comments

First of all, thank you for writing this package. After tinkering around with it, I got it to work. Some remarks remain - you might put this in the readme so people don't have to search for it as long as I did.

I don't have the knowledge to solve all of the problems, but I'll look into it. Just wanted to report this to help others.

version for node v0.12.x (and v0.11.x) is here: https://github.com/SebastianPozoga/rc522-rfid

I rewrite accessor for new v8 engine. But actually it is test version. I will added pull request when I end tests into application. You can test with me ;-)

Thanks! I'll be using this package on Wednesday and will give your new version a try.

Hi Sebastian. This are great news. I'll have a look at your version as well and can update mine with your changes. I'll also have a look at my read me file and update it accordingly.

@rvanderveen in answer to point 3: Node.js application will only work if you run it as root.
Well, root permissions is required. To make it better:

PS. @sbrinkmann I created (and tested) a pull request for you ( #5 ). I want add official repository to my service dependencies. It use my fork now. I am hope you approve it soon. Thanks.