docs: improve make with parallel jobs description. #21942

1. 
   klementtan commented at 12:25 pm on May 13, 2021: contributor

   Changed use -jX here for parallelism to use "-j N" for N parallel jobs

   Rationale: In my opinion use -jX here for parallelism is quite ambiguous as it could be perceived as a single option without any argument. Ie running:

   0make -jX
   

   Embarrassingly this caused me to be stuck for quite a long time until I opened the help menu for make but if I am the only one who faced this issue I would be happy to close this PR.

2. fanquake added the label Docs on May 13, 2021
3. 
   laanwj commented at 3:17 pm on May 13, 2021: member

   ACK, agree that this is a clearer wording.
4. 
   DrahtBot commented at 3:19 pm on May 13, 2021: member

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

   Conflicts

   Reviewers, this pull request conflicts with the following ones:

   • #20610 (doc: update for NetBSD 9.1, add GUI Build Instructions by jarolrod)

   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.

5. 
   jarolrod commented at 5:37 pm on May 13, 2021: member

   Concept ACK

   Two more places where you might want to add a similar note:

   • https://github.com/bitcoin/bitcoin/blob/master/doc/build-unix.md#to-build
   • https://github.com/bitcoin/bitcoin/blob/master/doc/build-windows.md#building-for-64-bit-windows
6. docs: improve make with parallel jobs description. 07bc22ef10
7. klementtan force-pushed on May 14, 2021
8. 
   klementtan commented at 0:46 am on May 14, 2021: contributor

   Concept ACK

   Two more places where you might want to add a similar note:

   • https://github.com/bitcoin/bitcoin/blob/master/doc/build-unix.md#to-build
   • https://github.com/bitcoin/bitcoin/blob/master/doc/build-windows.md#building-for-64-bit-windows

   Thanks for pointing that out. Updated the PR according to this.

9. 
   jarolrod commented at 0:50 am on May 14, 2021: member

   ACK 07bc22ef105dee3c6c465a3ef31e52fd198e786d
10. MarcoFalke merged this on May 14, 2021
11. MarcoFalke closed this on May 14, 2021

    ---

12. sidhujag referenced this in commit b1f415ee9d on May 14, 2021
13. gwillen referenced this in commit 251048913c on Jun 1, 2022
14. klementtan deleted the branch on Jun 5, 2022
15. DrahtBot locked this on Jun 5, 2023

Contributors
klementtan laanwj DrahtBot jarolrod

Labels
Docs