Skip to content
This repository was archived by the owner on Mar 21, 2024. It is now read-only.

Consider eliminating config #include from version.h #443

Closed
jaredhoberock opened this issue Nov 6, 2013 · 1 comment · Fixed by #451
Closed

Consider eliminating config #include from version.h #443

jaredhoberock opened this issue Nov 6, 2013 · 1 comment · Fixed by #451
Labels
type: enhancement New feature or request.

Comments

@jaredhoberock
Copy link
Contributor

Reported in this thread:

https://groups.google.com/d/msg/thrust-users/VYORJ7_rDwU/xi-TUtZBNkAJ
@jaredhoberock
Copy link
Contributor Author

If we do this, we should document that version.h is the only header which will not cause backends to get configured with the rationale that a user may wish to use the version number to manually configure backends himself.

jaredhoberock added a commit to jaredhoberock/thrust that referenced this issue Dec 3, 2013
@jaredhoberock
Avoid including <thrust/detail/config.h> in thrust/version.h.
1a126ee 
Fixes NVIDIA#443
@jaredhoberock jaredhoberock mentioned this issue Dec 3, 2013
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
type: enhancement New feature or request.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Avoid including <thrust/detail/config.h> in thrust/version.h. jaredhoberock/thrust
1 participant
@jaredhoberock