Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Installer #478

Closed
wants to merge 3 commits into from
Closed

[WIP] Installer #478

wants to merge 3 commits into from

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Apr 9, 2018

Prepares #366

The goal is to "configure" - "build" (nothing to do) and "install" Alpaka just like any other C++ library via a standardized and well-defined CMake workflow.

To Do

  • rebase against develop
  • move logic from alpakaConfig.cmake to root CMakeLists.txt; generate alpakaConfig.cmake
  • move and install helpers in cmake/ as well
  • remove Findalpaka.cmake: legacy script; only needed when a *Config.cmake package is not present
  • unset CMAKE_SIZEOF_VOID_P before creating alpakaConfig.cmake; docs
  • decide if Alpaka should be spelled uppercase in project name (and all CMake options/variables, config package, etc.)
  • decide when to set ALPAKA_ACC_... vars: ideally allow after install as well

References

Projects:

@ax3l ax3l added the Type:Install installation & packaging label Apr 9, 2018
@ax3l ax3l requested a review from BenjaminW3 April 9, 2018 15:13
@ax3l ax3l force-pushed the topic-install branch 2 times, most recently from 50fc632 to 895ada6 Compare April 10, 2018 12:07
ax3l added 3 commits May 4, 2018 14:48
Script that bumps the version in all places needed.
Prepare to install headers
#
# This file is part of alpaka.
#
# alpaka is free software: you can redistribute it and/or modify
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With regards to #637, this new file's header will need an update :)

@BenjaminW3 BenjaminW3 removed this from the Future milestone May 26, 2019
@j-stephan j-stephan mentioned this pull request Jan 27, 2020
@ax3l
Copy link
Member Author

ax3l commented Feb 11, 2020

Superseded by @j-stephan 's #921 🚀 ✨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants