Makefile dependencies broken by autotools? #3070

issue gavinandresen opened this issue on October 9, 2013
  1. gavinandresen commented at 4:43 AM on October 9, 2013: contributor

    So I compiled in my master branch, then checked out my 'smartfee' branch and re-compiled, and get:

    Undefined symbols for architecture x86_64:
      "CTransaction::nMinRelayTxFee", referenced from:
          PaymentServer::processPaymentRequest(PaymentRequestPlus&, QList<SendCoinsRecipient>&) in libbitcoinqt.a(libbitcoinqt_a-paymentserver.o)
    

    Indeed, CTransaction::nMinRelayTxFee is gone in the smartfee branch. But why didn't the new autotools make figure that out?

    Workaround is to do a 'make clean', but dependencies should work properly or we'll spend a lot of time tracking down bugs that go away if we make clean and then recompile.

  2. wtogami commented at 7:39 AM on October 10, 2013: contributor

    "make clean" works because it regenerates the makefiles from scratch.

    https://www.flameeyes.eu/autotools-mythbuster/automake/maintainer.html This may be relevant.

    It sounds like you want make to detect that it needs to regenerate its own rules. I'm uncertain if that will play nice with faketime in gitian?

  3. rebroad commented at 11:51 PM on April 7, 2014: contributor

    #3989 - it's been suggested that moving away from recursive Makefiles is the best solution.

  4. sipa commented at 12:29 AM on April 8, 2014: member

    As we're moving towards more modularized source trees, we'll need a solution for interdependencies anyway, I guess? (autotools noob here)

  5. rebroad commented at 8:09 AM on May 2, 2014: contributor

    #4118 is a relative to this issue.

  6. laanwj commented at 8:11 AM on May 2, 2014: member

    @rebroad can you please stop pimping your issue everywhere; if you have a solution, that's great, but refering to another issue where the problem is re-stated is not constructive.

  7. gavinandresen commented at 3:30 PM on June 6, 2014: contributor

    Fixed-- I'm a happy camper.

  8. gavinandresen closed this on Jun 6, 2014

  9. Bushstar referenced this in commit 1905422561 on Apr 8, 2020
  10. DrahtBot locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-05-01 15:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me