Forum Replies Created

Viewing 6 posts - 1 through 6 (of 6 total)
  • Author
    Posts
  • in reply to: OSPi 2.1.9 build errors #65060

    Dutchie30
    Participant

    A very late reply from me: Adding #include <stdlib.h> (Thx @Marc!) fixed the issue for me too. I’m all for including this in the next release.

    in reply to: OSPi 2.1.9 build errors #62829

    Dutchie30
    Participant

    It’s the option to do a “quiet” build:

    # head -n 11 build.sh 
    #!/bin/bash
    
    while getopts ":s" opt; do
      case $opt in
        s)
              SILENT=true
              command shift
          ;;
      esac
    done
    echo "Building OpenSprinkler..."

    It fails without -s too:

    # ./build.sh ospi
    Building OpenSprinkler...
    weather.cpp: In function ‘void getweather_callback(char*)’:
    weather.cpp:55:31: error: ‘atoi’ was not declared in this scope
       wt_errCode = atoi(tmp_buffer);
                                   ^
    weather.cpp:61:22: error: ‘atoi’ was not declared in this scope
       v = atoi(tmp_buffer);
                          ^
    weather.cpp:71:22: error: ‘atoi’ was not declared in this scope
       v = atoi(tmp_buffer);
                          ^
    weather.cpp:80:22: error: ‘atoi’ was not declared in this scope
       v = atoi(tmp_buffer);
                          ^
    weather.cpp:89:31: error: ‘atol’ was not declared in this scope
       uint32_t l = atol(tmp_buffer);
                                   ^
    weather.cpp:98:22: error: ‘atoi’ was not declared in this scope
       v = atoi(tmp_buffer);
                          ^
    weather.cpp:110:22: error: ‘atoi’ was not declared in this scope
       v = atoi(tmp_buffer);
                          ^
    Done!
    in reply to: OSPi 2.1.9 build errors #62821

    Dutchie30
    Participant

    Strange. I’m not confident I’m missing the atoi library. A build of 2.1.8 works as expected on the same host:

    [root@opensprinkler OpenSprinkler-Firmware-218]# ./build.sh ospi
    Building OpenSprinkler...
    Done!

    2.1.8 also looks to contain references to atoi in server.cpp, so it looks like I do have the atoi library available. Is it possible the atoi library isn’t declared properly in the new weather.cpp?

    in reply to: Increase maximum program name length #52470

    Dutchie30
    Participant

    Thanks Ray, looking forward to it.

    in reply to: OSPi 2.1.7 build errors on CentOS 7.2.1511 for Rpi3 #44498

    Dutchie30
    Participant

    Good to know Samer, thanks!

    in reply to: OSPi 2.1.7 build errors on CentOS 7.2.1511 for Rpi3 #44495

    Dutchie30
    Participant

    Thanks Ray, looks to be working. Only thing is that doing git pull; ./build.sh ospi; systemctl restart opensprinkler seems to wipe out the config.

    Other than that, awesome job on OpenSprinkler, I’m really looking forward to working with it!

Viewing 6 posts - 1 through 6 (of 6 total)