Skip to content

Commit

Permalink
[upstream_utils] Remove ConcurrentQueue (#7183)
Browse files Browse the repository at this point in the history
It appears to be broken under stress testing (dropped/duplicate values).

This reverts commit 97c6c86 (#7066).
  • Loading branch information
PeterJohnson authored Oct 11, 2024
1 parent f65f9ed commit dcf5f55
Show file tree
Hide file tree
Showing 6 changed files with 1 addition and 3,815 deletions.
5 changes: 0 additions & 5 deletions .github/workflows/upstream-utils.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,11 +40,6 @@ jobs:
cd upstream_utils
./argparse_lib.py clone
./argparse_lib.py copy-src
- name: Run concurrentqueue.py
run: |
cd upstream_utils
./concurrentqueue.py clone
./concurrentqueue.py copy-src
- name: Run eigen.py
run: |
cd upstream_utils
Expand Down
40 changes: 0 additions & 40 deletions upstream_utils/concurrentqueue.py

This file was deleted.

2 changes: 0 additions & 2 deletions wpiutil/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,6 @@ install(
DIRECTORY
src/main/native/include/
src/main/native/thirdparty/argparse/include/
src/main/native/thirdparty/concurrentqueue/include/
src/main/native/thirdparty/expected/include/
src/main/native/thirdparty/json/include/
src/main/native/thirdparty/llvm/include/
Expand All @@ -209,7 +208,6 @@ target_include_directories(
PUBLIC
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/src/main/native/include>
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/src/main/native/thirdparty/argparse/include>
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/src/main/native/thirdparty/concurrentqueue/include>
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/src/main/native/thirdparty/expected/include>
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/src/main/native/thirdparty/json/include>
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/src/main/native/thirdparty/llvm/include>
Expand Down
5 changes: 1 addition & 4 deletions wpiutil/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -183,9 +183,6 @@ cppHeadersZip {
from('src/main/native/thirdparty/argparse/include/') {
into '/'
}
from('src/main/native/thirdparty/concurrentqueue/include') {
into '/'
}
from('src/main/native/thirdparty/expected/include') {
into '/'
}
Expand Down Expand Up @@ -241,7 +238,7 @@ model {
all {
it.sources.each {
it.exportedHeaders {
srcDirs 'src/main/native/include', 'src/main/native/thirdparty/argparse/include/', 'src/main/native/thirdparty/concurrentqueue/include', 'src/main/native/thirdparty/expected/include', 'src/main/native/thirdparty/fmtlib/include', 'src/main/native/thirdparty/llvm/include', 'src/main/native/thirdparty/sigslot/include', 'src/main/native/thirdparty/json/include', 'src/main/native/thirdparty/memory/include', 'src/main/native/thirdparty/mpack/include', 'src/main/native/thirdparty/protobuf/include'
srcDirs 'src/main/native/include', 'src/main/native/thirdparty/argparse/include/', 'src/main/native/thirdparty/expected/include', 'src/main/native/thirdparty/fmtlib/include', 'src/main/native/thirdparty/llvm/include', 'src/main/native/thirdparty/sigslot/include', 'src/main/native/thirdparty/json/include', 'src/main/native/thirdparty/memory/include', 'src/main/native/thirdparty/mpack/include', 'src/main/native/thirdparty/protobuf/include'
}
}
}
Expand Down
Loading

0 comments on commit dcf5f55

Please sign in to comment.