Skip to content
Snippets Groups Projects
  1. Aug 11, 2020
  2. Aug 10, 2020
  3. Aug 07, 2020
  4. Aug 06, 2020
  5. Aug 05, 2020
  6. Aug 03, 2020
    • alcroito's avatar
      AutoGen: Add test to check for correct AutoMoc dependencies · 7445c9a5
      alcroito authored
      When using Qt 5.15.0 or above together with Ninja, check that touching
      a source file of a dependency does not needlessly re-run AUTOMOC for
      the dependee target.
      7445c9a5
    • alcroito's avatar
      AutoGen: Fix over-specified direct dependencies of custom command · a79056bb
      alcroito authored
      The AutoMoc timestamp creating custom command explicitly depended
      on all dependencies of the origin target (associated to the AutoGen
      target).
      
      When an origin target depended on a shared library 'libfoo.so',
      if it was re-linked, the AutoMoc custom command would touch its
      output timestamp file, and thus cause needless rebuilding of sources,
      despite the shared library not having any influence on the AutoMoc
      generated files.
      
      Introduce a new '<target>_autogen_timestamp_deps' utility target,
      which will serve as an 'order-only' dependency for the custom command.
      
      This will prevent needless rebuilding, because touching 'libfoo.so'
      will not cause the custom command to be re-executed.
      
      The new AutoMoc dependency tree looks like:
          '_autogen_timestamp_deps (serves as order-only dep)'
       <- '<target_autogen>/timestamp' file ( + moc deps file)
       <- '<target>_autogen' target.
      
      Fixes: #21020
      a79056bb
    • Brad King's avatar
      Merge branch 'ninja-multi-rsp-remove-path' into release-3.17 · 507fecd7
      Brad King authored
      Merge-request: !5094
      507fecd7
    • Kyle Edwards's avatar
      Ninja: Restore shorter path to response files · cdb50af2
      Kyle Edwards authored and Brad King's avatar Brad King committed
      In commit 99ed39b0 (Ninja Multi-Config: Make link response files
      per-config, 2020-07-15, v3.17.4~3^2), we added the target directory to
      the response file under the mistaken assumption that two different
      targets with the same name could be in different directories. However,
      this causes the path to the response file to be too long to fit on a
      command line.  Take the path back out, while leaving in the per-config
      split.
      
      Fixes: #21050
      cdb50af2
    • lauritsriple's avatar
      GoogleTest: Restore support for list arguments in TEST_EXECUTOR · 8829d2ca
      lauritsriple authored and Brad King's avatar Brad King committed
      Refactoring in commit 889a7146 (GoogleTestAddTests: Refactor into
      callable method, 2020-03-16, v3.18.0-rc1~450^2~3) accidentally
      parsed `TEST_EXECUTOR` as a single-value argument instead of a list.
      8829d2ca
  7. Jul 30, 2020
Loading