rapidsai / cuhornet

BSD 3-Clause "New" or "Revised" License
25 stars 26 forks source link

removed master #45

Closed pseudotensor closed 4 years ago

pseudotensor commented 4 years ago

By removing master, now all old scripts from rapidsai break, FYI. One cannot use the build scripts or even make anymore.

pseudotensor commented 4 years ago

image

pseudotensor commented 4 years ago
$ make -j 10
[  1%] Performing update step for 'cugunrock'
[  2%] Performing update step for 'raft'
[  3%] Performing update step for 'cuhornet'
[  4%] Performing update step for 'cub'
fatal: ambiguous argument 'master': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
[  5%] No patch step for 'raft'
[  6%] No patch step for 'cub'
[  7%] No configure step for 'raft'
[  8%] No configure step for 'cub'
[  9%] No build step for 'raft'
[  9%] No build step for 'cub'
[ 10%] No install step for 'raft'
[ 11%] No install step for 'cub'
[ 12%] Completed 'raft'
[ 13%] Completed 'cub'
[ 15%] Built target raft
[ 17%] Built target cub
error: pathspec 'master' did not match any file(s) known to git.
CMake Error at cugraph/cpp/build/cuhornet/tmp/cuhornet-gitupdate.cmake:201 (message):
  Failed to checkout tag: 'master'
pseudotensor commented 4 years ago

In general, this repo needs to be tagged by rapids version if other rapids repos reference it, and then those other references need to be specific to the tag not master/main.

afender commented 4 years ago

Thanks for catching that @pseudotensor I just switched cuGraph's CMake to use cuHornet's latest hashtag in https://github.com/rapidsai/cugraph/pull/1180