Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-55808

icecream and ninja affecting configure commands

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.9.0
    • Component/s: None
    • Labels:
      None
    • Backwards Compatibility:
      Fully Compatible
    • Operating System:
      ALL
    • Steps To Reproduce:
      Hide

      Generate ninja build with VERBOSE=1, and notice the config logs for fallocate are empty, it should have the commands used for the configure check printed to the logs.

       

      Also using icecream, after you have fixed ninja breaking the logs, you will notice the same fallocate configure step has icerun in it.

      Show
      Generate ninja build with VERBOSE=1, and notice the config logs for fallocate are empty, it should have the commands used for the configure check printed to the logs.   Also using icecream, after you have fixed ninja breaking the logs, you will notice the same fallocate configure step has icerun in it.
    • Sprint:
      Dev Platform 2021-04-19

      Description

      The configure steps should not be affected by icecream or ninja because these tools are used during the actual build and not in build preparation.

      Ninja was hiding the conf logs, by making a noop function for 
      PRINT_CMD_LINE_FUNC.
       
      Icecream was using icerun for configure commands which use some of the builders besides compilation builders, which in those cases icecream should handle and run locally, however in some cases icecream will hang, causing confusion when this happens during a configure check. In order to reduce the number of moving parts, icecream should take no part in general configure steps.

        Attachments

          Activity

            People

            Assignee:
            daniel.moody Daniel Moody
            Reporter:
            daniel.moody Daniel Moody
            Participants:
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: