Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • C clm
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 2
    • Merge requests 2
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • clean-and-itasksclean-and-itasks
  • clm
  • Merge requests
  • !13

Don't redirect compiler stderr to stdout on Windows

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Camil Staps requested to merge keep-separate-stdout-and-stderr into master Oct 14, 2020
  • Overview 2
  • Commits 2
  • Pipelines 2
  • Changes 1

It is not clear to me why this would be useful, except a slight performance enhancement because you need one less thread. This causes a problem in testproperties, which calls clm and passes stdout and stderr through. When testproperties is called by cleantest, cleantest expects only JSON test events on stdout, while stderr may be used for warnings. So if the compiler warnings are all on stderr and clm provides no extra output on stdout (as is the case after this commit), the cleantest user will get information about compilation errors/warnings while the tests continue to work.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: keep-separate-stdout-and-stderr