Qpid
  1. Qpid
  2. QPID-4699

C++ INSTALL instructions might be clearer if the "Building from a Source Distribution" was moved to the top.

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.22
    • Component/s: C++ Broker, C++ Client
    • Labels:
      None
    • Environment:

      All

      Description

      With the INSTALL instructions in <qpid>/cpp the "Building from a Source
      Distribution" instructions are fairly near the bottom.

      I kind of see the logic of that since the earlier sections are describing prerequisites, but I wonder if it's worth moving the main install bit closer to the top with a line that says something like "If you have not installed qpid previously see prerequisites below".

      Similarly section "2.2.2. From Source" might be better in the prerequisites section rather than the install section, I think that's particularly worthwhile now because 2.2.2 is talking about an automake build for
      the dependencies and the qpid install is now cmake. I know that I did a double take when I skimmed through and assumed that the instructions hadn't been updated for cmake until I read again.

        Activity

        Hide
        Fraser Adams added a comment -

        Just to be clear, I did an extra update following comments from Steve & Robbie so the latest commit of this is now at r1464493 and not r1464175 as stated in my original comment (though r1464493 is what I gave to Justin when I requested inclusion in 0.22).

        Show
        Fraser Adams added a comment - Just to be clear, I did an extra update following comments from Steve & Robbie so the latest commit of this is now at r1464493 and not r1464175 as stated in my original comment (though r1464493 is what I gave to Justin when I requested inclusion in 0.22).
        Hide
        Justin Ross added a comment -

        Reviewed by Alan, Robbie, and Steve. Approved for 0.22.

        Show
        Justin Ross added a comment - Reviewed by Alan, Robbie, and Steve. Approved for 0.22.
        Hide
        Fraser Adams added a comment -

        Committed updated INSTALL instructions to r1464175

        Show
        Fraser Adams added a comment - Committed updated INSTALL instructions to r1464175

          People

          • Assignee:
            Unassigned
            Reporter:
            Fraser Adams
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development