util: Check for curl in Posix PATH #17842

pull emilengler wants to merge 1 commits into bitcoin:master from emilengler:2020-01-check-curl-path changing 2 files +26 −2
  1. emilengler commented at 7:54 PM on January 1, 2020: contributor

    Currently, we don't check if curl is actually installed on the user computer when suggesting it in the RPC help.

    This PR checks for cURL in the PATH environment variable (it is defined by Posix). If a file called "curl" is found in the path, we return the current code command. Otherwise we just return an empty string. If the envvar can't be obtained for some reasons, we display the command anyways.

  2. util: Check for curl in Posix PATH 5b0028cab6
  3. emilengler force-pushed on Jan 1, 2020
  4. DrahtBot added the label RPC/REST/ZMQ on Jan 1, 2020
  5. DrahtBot commented at 9:49 PM on January 1, 2020: member

    <!--e57a25ab6845829454e8d69fc972939a-->

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    <!--174a7506f384e20aa4161008e828411d-->

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #17623 (rpc: add extensive file checks for dumptxoutset and dumpwallet by brakmic)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  6. fanquake commented at 3:24 AM on January 2, 2020: member

    Concept NACK. bitcoind shouldn't be probing users systems to check what programs they may have installed. The examples are just that, examples.

  7. emilengler commented at 5:42 AM on January 2, 2020: contributor

    But how is an example useful if it isn't usable on the machine?

  8. luke-jr commented at 6:24 AM on January 2, 2020: member

    Agree with @fanquake : Concept NACK

  9. elichai commented at 10:25 AM on January 2, 2020: contributor

    But how is an example useful if it isn't usable on the machine?

    The user could install it. if you have a more "universal" example than let's just replace the current one with the more universal one. but if the check is just to remove an example then I don't think it's useful, I'd rather show the user an example which A. might give him a notion of how the request should look like. B. he might just install curl if he wants to copy-paste the example.

  10. emilengler commented at 12:20 PM on January 2, 2020: contributor

    Ok, will close then

  11. emilengler closed this on Jan 2, 2020

  12. DrahtBot locked this on Feb 15, 2022

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-04-13 15:14 UTC

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