Exception if non root user uses the library

[akatran]

The library will throw and error if user has not effective root access.

The error could be handled from above layers like another library or main program!

[akatran]

In my case I use the following code to handle the exception. My environment is a Raspberry Pi.

try {
    gw.begin(nodeID, 0x4c, RF24_1MBPS);
  }
  catch (int e){
    fprintf(stderr, "Root access required!\n") ;
    exit(e);
  }

The library is compatible with Arduino.
Actually the function geteuid() will not be included by the linker.
I have tested with Arduino Nano.

[akatran]

This is the second attempt to address #424.

[TMRh20]

Won’t this break things for users running as non-root? With spidev for example, the lib can be run as non root user. Should the fix be put into the bcm driver instead of rf24?

…

On Apr 20, 2018, at 16:41, Apostolos Katranitsas ***@***.***> wrote: This is the second attempt to address #424. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

[akatran]

I thought not to put this in bcm driver in order to be easy updating the driver.

If you make changes to the bcm library before you put it into the repo then it is fine to add one more but if not then you will have to apply this everytime!

If the late is not a problem then I will apply it into the bcm driver.

May I propose to put it into the spi.cpp? That way you can easily update the bcm driver and will be used only for Raspberry Pi.

[TMRh20]

Yeah, SPI.cpp seems like the best place for it.

…

On Apr 21, 2018, at 13:26, Apostolos Katranitsas ***@***.***> wrote: I thought not to put this in bcm driver in order to be easy updating the driver. If you make changes to the bcm library before you put it into the repo then it is fine to add one more but if not then you will have to apply this everytime! If the late is not a problem then I will apply it into the bcm driver. May I propose to put it into the spi.cpp? That way you can easily update the bcm driver and will be used only for Raspberry Pi. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

[akatran]

Per @TMRh20 comment I'll move the exception into SPI.cpp!

[akatran]

OK, it is ready!