Skip to content

Commit

Permalink
Remove WORKSPACE env var from cudf_test temp_directory class (rapidsa…
Browse files Browse the repository at this point in the history
…i#14588)

Removes the `WORKSPACE` env var check from the `cudf_test` `temp_directory` class. This env var is not being used. This is part of a larger effort to consolidate and document libcudf env vars.

Authors:
  - David Wendt (https://github.com/davidwendt)

Approvers:
  - Nghia Truong (https://github.com/ttnghia)
  - Bradley Dice (https://github.com/bdice)
  - Vukasin Milovanovic (https://github.com/vuule)

URL: rapidsai#14588
  • Loading branch information
davidwendt authored and karthikeyann committed Dec 12, 2023
1 parent 881baeb commit 39b6bc5
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions cpp/include/cudf_test/file_utilities.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -16,14 +16,13 @@

#pragma once

#include <cudf/utilities/error.hpp>

#include <cstdio>
#include <cstdlib>
#include <filesystem>
#include <string>

#include <ftw.h>

#include <cudf/utilities/error.hpp>
#include <string>

/**
* @brief RAII class for creating a temporary directory.
Expand All @@ -41,7 +40,6 @@ class temp_directory {
temp_directory(std::string const& base_name)
{
std::string dir_template{std::filesystem::temp_directory_path().string()};
if (auto env_p = std::getenv("WORKSPACE")) dir_template = env_p;

dir_template += "/" + base_name + ".XXXXXX";
auto const tmpdirptr = mkdtemp(const_cast<char*>(dir_template.data()));
Expand Down

0 comments on commit 39b6bc5

Please sign in to comment.