Skip to content
This repository has been archived by the owner on Nov 25, 2024. It is now read-only.

explicitly include cstdlib #13

Closed
Tracked by #38
stadlmax opened this issue Nov 30, 2022 · 0 comments · Fixed by #24
Closed
Tracked by #38

explicitly include cstdlib #13

stadlmax opened this issue Nov 30, 2022 · 0 comments · Fixed by #24
Assignees
Milestone

Comments

@stadlmax
Copy link
Contributor

The code contains abort() in few places without explicitly including the matching headers. With the Dockerfile given in this repository, this somehow does not cause any issues. To allow more straightforward builds from source in other environments, it might be better to explicitly import those headers, e.g. cstdlib.

@BradReesWork BradReesWork added this to the 23.08 milestone Jul 13, 2023
@rapids-bot rapids-bot bot closed this as completed in #24 Jul 28, 2023
rapids-bot bot pushed a commit that referenced this issue Jul 28, 2023
This branch is a collection of all the refactoring improvements.  

closes #13 
closes #14 
closes #17 
closes #18 
closes rapidsai/cugraph-ops#492

Authors:
  - Brad Rees (https://github.com/BradReesWork)
  - https://github.com/dongxuy04
  - Junhong Liu (https://github.com/hello-11)
  - Tingyu Wang (https://github.com/tingyu66)

Approvers:
  - Tingyu Wang (https://github.com/tingyu66)
  - https://github.com/dongxuy04
  - AJ Schmidt (https://github.com/ajschmidt8)

URL: #24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants