build: don't warn when doxygen isn't found #19301

pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:no_warn_doxygen_missing changing 1 files +0 −3

fanquake commented at 10:44 AM on June 17, 2020: member

Doxygen isn't so important that we need to warn when it is missing. I'd assume it might even be missing more often than not for most builds.

build: don't warn when doxygen isn't found

Doxygen isn't so important that we need to warn when it is missing. I'd
assume it might even be missing more often than not for most builds.

fa84edb93c

fanquake added the label Build system on Jun 17, 2020
MarcoFalke commented at 10:50 AM on June 17, 2020: member

Fine with me ACK fa84edb93c85f7709fc53abf9c6daae5d1bb3b28
hebasto approved
hebasto commented at 11:51 AM on June 17, 2020: member
ACK fa84edb93c85f7709fc53abf9c6daae5d1bb3b28, I have reviewed the code and it looks OK, I agree it can be merged.

It was so annoying to see on test VMs:
```
$ ./configure > /dev/null 
configure: WARNING: Doxygen not found
```
fanquake merged this on Jun 18, 2020
fanquake closed this on Jun 18, 2020
fanquake deleted the branch on Jun 18, 2020
laanwj commented at 9:23 AM on June 18, 2020: member

Posthumous ACK
PastaPastaPasta referenced this in commit 849b3217df on Jun 27, 2021
PastaPastaPasta referenced this in commit ef2071b81b on Jun 28, 2021
PastaPastaPasta referenced this in commit b3fa7e391e on Jun 29, 2021
PastaPastaPasta referenced this in commit 565d3e104d on Jul 1, 2021
PastaPastaPasta referenced this in commit b0a9b68693 on Jul 1, 2021
PastaPastaPasta referenced this in commit 1a20c25a0d on Jul 15, 2021
DrahtBot locked this on Feb 15, 2022

Contributors

Labels