[pdal-commits] [PDAL/PDAL] 389be3: Remove '..' in configured temp path for unit tests

GitHub noreply at github.com
Fri Oct 13 10:07:58 PDT 2017


  Branch: refs/tags/1.6
  Home:   https://github.com/PDAL/PDAL
  Commit: 389be3d92dffd1905f97508ef01fc80437866aa8
      https://github.com/PDAL/PDAL/commit/389be3d92dffd1905f97508ef01fc80437866aa8
  Author: Bradley J Chambers <brad.chambers at gmail.com>
  Date:   2017-10-12 (Thu, 12 Oct 2017)

  Changed paths:
    M test/unit/Support.cpp
    M test/unit/TestConfig.hpp.in

  Log Message:
  -----------
  Remove '..' in configured temp path for unit tests

musl has changed it's handling of '.' and '..' when globbing [1]. These only
recently made their way into the alpine:edge image that we use in Travis and
for Docker automated builds [2]. For now, we simply configure the temp path
directly, rather than relying on '..'.

[1] https://github.com/alpinelinux/aports/blob/afdba0ba0b8c0e92c376e7f1bfb151c07bf654b6/main/musl/0068-fix-glob-descent-into-.-and-.-with-GLOB_PERIOD.patch
[2] http://www.openwall.com/lists/musl/2017/01/12/5


  Commit: 58b586fb392085a0ea32dcdedc488f78d7ea2c00
      https://github.com/PDAL/PDAL/commit/58b586fb392085a0ea32dcdedc488f78d7ea2c00
  Author: Howard Butler <howard at hobu.co>
  Date:   2017-10-13 (Fri, 13 Oct 2017)

  Changed paths:
    M test/unit/Support.cpp
    M test/unit/TestConfig.hpp.in

  Log Message:
  -----------
  Merge pull request #1703 from PDAL/alpine-edge-issues

Remove '..' in configured temp path for unit tests


Compare: https://github.com/PDAL/PDAL/compare/ac17d3dd8601...58b586fb3920


More information about the pdal-commits mailing list