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

SConstruct mixes tabs and spaces -- the tabs should be 8 spaces

    XMLWordPrintableJSON

Details

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Major - P3 Major - P3
    • 2.3.0
    • None
    • None
    • None
    • Fully Compatible
    • ALL

    Description

      The SConstruct file at the top of the source tree in master and 2.2 branches uses tabs for indenting at lines 558, 560, 562 and 563.

      The code works correctly because Python interprets each tab as 8 spaces, but it looks wrong in any editor set to show 4 spaces per tab. We should replace each tab with 8 spaces (which will make it look right) to prevent future accidents (and for consistency with the "don't use tabs" rule).

      Attachments

        Activity

          People

            tad Tad Marshall
            tad Tad Marshall
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: