Skip to content
Snippets Groups Projects

ci: Run tests using dxcompiler.

Merged Giovanni Mascellani requested to merge giomasce/vkd3d:vesuvio into master
1 unresolved thread

This goes on top of !346 (merged). See a pipeline example.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
19 doxygen doxygen-latex graphviz \
20 mesa-vulkan-drivers mesa-vulkan-drivers:i386 \
21 vulkan-tools spirv-headers \
22 libvulkan-dev libvulkan-dev:i386 \
23 libncurses-dev libncurses-dev:i386 \
24 libxcb1-dev libxcb1-dev:i386 \
25 libxcb-util-dev libxcb-util-dev:i386 \
26 libxcb-icccm4-dev libxcb-icccm4-dev:i386 \
27 libxcb-keysyms1-dev libxcb-keysyms1-dev:i386 && \
15 apt-get install -y eatmydata && \
16 eatmydata apt-get dist-upgrade -y && \
17 eatmydata apt-get install -y build-essential pkg-config gcc-multilib gcc-mingw-w64 \
18 autoconf automake libtool flex bison curl \
19 git ca-certificates rsync \
20 doxygen doxygen-latex graphviz \
21 mesa-vulkan-drivers mesa-vulkan-drivers:i386 \
  • Comment on lines +15 to +21

    That doesn't seem useful. If we are re-creating the docker image often enough for this to make a difference, we are doing something wrong.

  • It's probably not particularly useful in production, but during development it can be useful to iterate a few times on the Docker script, and trimming a few minutes can be nice. On the other hand, is there a real reason for which using eatmydata is a problem?

  • It's probably not particularly useful in production, but during development it can be useful to iterate a few times on the Docker script, and trimming a few minutes can be nice. On the other hand, is there a real reason for which using eatmydata is a problem?

    Well, it's an extra package that makes the image bigger, and it's adding some noise to the script. Nothing major of course.

  • Giovanni Mascellani changed this line in version 3 of the diff

    changed this line in version 3 of the diff

  • Well, it's an extra package that makes the image bigger, and it's adding some noise to the script. Nothing major of course.

    Yeah, it's not a big deal, and it makes development somewhat quicker. However, in the interest of getting things in, I removed that change.

  • Please register or sign in to reply
  • added 4 commits

    • 8b1a53d8 - ci: Build widl for 64 bit.
    • 7e63809a - tests: Fail if dxcompiler is not available at runtime.
    • c356668e - ci: Build the DXIL parser.
    • fe53c1f0 - ci: Run tests using dxcompiler.

    Compare with previous version

  • Alexandre Julliard added 13 commits

    added 13 commits

    Compare with previous version

  • Alexandre Julliard approved this merge request

    approved this merge request

  • Please register or sign in to reply
    Loading