Skip to content
Snippets Groups Projects
  1. May 20, 2021
  2. May 12, 2021
  3. May 11, 2021
  4. Apr 30, 2021
    • Simon McVittie's avatar
      runtime: Add support for creating runtime from a mtree(5) · 799b7931
      Simon McVittie authored
      
      At the moment we deploy the runtime from a giant tarball to avoid Steam
      downloader limitations, but that leads to a noticeable delay the first
      time we launch a game after a new runtime version has been downloaded.
      
      Now that the Steam download mechanism can deal better with larger
      numbers of smaller files, we're considering returning to the original
      design where the runtime depot contains unpacked files. However, the
      Steam download mechanism doesn't preserve permissions, modification
      times, or filenames that differ only by case, and has not always
      preserved empty directories, so we need a way to deal with all of
      those things.
      
      By reading a manifest written in a subset of the BSD mtree(5) format,
      we can create directories and symlinks, and set permissions modification
      times on regular files. As a bonus, it's actually slightly faster to
      duplicate a runtime with hard-links (--copy-runtime mode) by reading the
      manifest than by reading the actual directory tree, because the manifest
      is more likely to be contiguous on disk.
      
      In principle the mtree(5) manifest could also be used to validate that
      the runtime content has not become corrupted by checking files against
      their sha256sums. This isn't implemented here (and it would have to be
      done only on demand rather than routinely, because it would be slow),
      but the parser does at least read the sha256.
      
      In the tests, we now need to remove the mtree manifest when copying
      and editing a runtime. When we edit a runtime in-place, it no longer
      conforms to the manifest, so this can't necesarily be expected to work.
      
      Signed-off-by: default avatarSimon McVittie <smcv@collabora.com>
      799b7931
    • Simon McVittie's avatar
      pressure-vessel: Add code to parse and apply a mtree(5) manifest · 1bdf777d
      Simon McVittie authored
      
      This can be used to create directories, symlinks and zero-byte files,
      create files as a clone (hard-link or copy) of files in a reference
      directory, assert that files exist, set executable/non-executable
      permissions, and set files' modification times.
      
      When cloning files from a reference directory, the default is to use
      the same filename as the destination, but that can be overridden
      (for example to deal with inconvenient/non-Windows-friendly filenames).
      This is also actually slightly faster than pv_cheap_tree_copy(),
      presumably because the manifest is contiguous on-disk and the
      directories are not necessarily.
      
      Setting directories' modification times is implemented, but probably
      won't work in practice, because any files we create in the directory
      will alter its mtime.
      
      Signed-off-by: default avatarSimon McVittie <smcv@collabora.com>
      1bdf777d
  5. Apr 27, 2021
  6. Apr 22, 2021
  7. Apr 21, 2021
  8. Apr 15, 2021
  9. Mar 23, 2021
  10. Mar 16, 2021
  11. Mar 04, 2021
  12. Feb 25, 2021
  13. Feb 23, 2021
  14. Feb 22, 2021
  15. Feb 19, 2021
  16. Feb 04, 2021
    • Simon McVittie's avatar
      pressure-vessel: Accept runtime with or without ./files · 5570e4a8
      Simon McVittie authored
      
      Previously, if we were using a Flatpak-style runtime, the argument
      to --runtime= had to be its ./files subdirectory. Now we accept the
      top-level directory as an alternative.
      
      This will make it more straightforward for pressure-vessel-wrap to
      take responsibility for unpacking and deploying runtimes that are
      shipped as a tarball, which is currently done in the SteamLinuxRuntime
      shell scripts, resulting in error handling and locking being less
      robust.
      
      Signed-off-by: default avatarSimon McVittie <smcv@collabora.com>
      5570e4a8
  17. Jan 28, 2021
  18. Jan 27, 2021
    • Simon McVittie's avatar
      tests: Don't look at the real environment for driver_environment() · 000b8dc0
      Simon McVittie authored
      
      If one of the variables we are interested in happens to be set
      already, and doesn't get overridden, then the test will fail.
      For example, `DRI_PRIME=1 meson test -C _build` would fail.
      
      This will become a lot more likely to happen when I start logging
      ubiquitous environment variables like `DISPLAY`.
      
      We can't just set envp = NULL initially, because g_environ_setenv()
      in Ubuntu 12.04 won't accept NULL as a valid environment block (although
      newer versions do).
      
      Signed-off-by: default avatarSimon McVittie <smcv@collabora.com>
      000b8dc0
  19. Jan 25, 2021
Loading