The else and message are redundant to the FindAPR cmake script above. The preferred by the script is to do 

```
set(APR_FIND_REQUIRED "Yes")
set(APRUTIL_FIND_REQUIRED "Yes")
find_package(APR)
```

Then follow this with your add_definitions, include_directories, and link_directories. You do not need to have them in an if other then the USE_APR if as it will FATAL_ERROR before reaching them if APR or APRUTIL is not present.