db.cpp:811: undefined reference to `fUseUPnP' #212

issue dholbert opened this issue on May 9, 2011

dholbert commented at 5:29 PM on May 9, 2011: contributor

When building bitcoin with upnp completely disabled in the build via setting this in makefile.unix...

<code>USE_UPNP:=</code> ...then I get this build warning:

<pre> obj/db.o: In function `CWalletDB::LoadWallet()': bitcoin/db.cpp:811: undefined reference to `fUseUPnP' obj/db.o: In function `Unserialize<CDataStream>': bitcoin/serialize.h:136: undefined reference to `fUseUPnP' </pre>

Based on how fHaveUPnP is used in other files, it looks like db.cpp needs to wrap its fHaveUPnP usages with

<pre> #if USE_UPNP ... #endif </pre>
dholbert commented at 5:49 PM on May 9, 2011: contributor

This issue is fixed by the changes in this pull request: #213
dholbert commented at 9:45 PM on May 12, 2011: contributor

This has now been resolved in issue #213.
dholbert closed this on May 12, 2011
dexX7 referenced this in commit c938ece9ad on Dec 14, 2014
sipa referenced this in commit 7863d105ea on Aug 4, 2015
sipa referenced this in commit 780d5b3c7b on Aug 23, 2015
TheBlueMatt referenced this in commit a671356e1f on Oct 20, 2015
sipa referenced this in commit 9177950c74 on Oct 21, 2015
sipa referenced this in commit f4787d1caf on Oct 21, 2015
sipa referenced this in commit 6557a8cd46 on Oct 26, 2015
sipa referenced this in commit ea06490d14 on Oct 27, 2015
sipa referenced this in commit 003bb87153 on Nov 5, 2015
sipa referenced this in commit bfd83199c3 on Nov 11, 2015
sipa referenced this in commit b437ea7ec9 on Nov 12, 2015
sipa referenced this in commit 1d84107924 on Nov 12, 2015
btcdrak referenced this in commit 90c884bffe on Nov 28, 2015
rebroad referenced this in commit 40ead34fbe on Dec 7, 2016
deadalnix referenced this in commit ddc9d5ce69 on Jan 15, 2017
deadalnix referenced this in commit 258720851e on Jan 19, 2017
deadalnix referenced this in commit dc5db04846 on Jan 20, 2017
classesjack referenced this in commit 14c08352cd on Jan 2, 2018
lateminer referenced this in commit 967e04831e on Oct 16, 2019
hyojin5658 referenced this in commit 966071e27b on Feb 22, 2021
hyojin5658 referenced this in commit 335001f414 on Feb 23, 2021
rajarshimaitra referenced this in commit 5bb60a65c6 on Aug 5, 2021
MarcoFalke locked this on Sep 8, 2021

Contributors

dholbert

Linked (view graph)

#213 add #if guard around fUseUPnP to fix build warning #25323 miniscript fuzzer failing after qa-assets update