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

API rework #404

Merged
merged 46 commits into from
Jun 10, 2024
Merged
Show file tree
Hide file tree
Changes from 45 commits
Commits
Show all changes
46 commits
Select commit Hold shift + click to select a range
d226db9
API rework
sashacmc May 16, 2024
eb95230
Keyexpr rework
sashacmc May 17, 2024
5163aaa
feat: remove reply callback indirection
jean-roland May 16, 2024
7fb2690
feat: add owned_reply and loaned_reply
jean-roland May 16, 2024
a3cc873
feat: implement reply as rc for query callbacks
jean-roland May 16, 2024
4da9aef
fix: change z_reply_err args
jean-roland May 16, 2024
0e4f783
fix: update tests with new reply rc
jean-roland May 16, 2024
539b156
fix: update examples with new reply rc
jean-roland May 16, 2024
9b74390
fix: unify owned_to_ptr functions
jean-roland May 17, 2024
a8c5dde
fix: dummy z_reply_create function args
jean-roland May 17, 2024
827c321
style: missing space on macros.h
jean-roland May 17, 2024
9315dba
fix: updating exposed types
jean-roland May 21, 2024
1fcd37f
Merge pull request #2 from jean-roland/ft_reply_rc
jean-roland May 21, 2024
6159305
Strings rework
sashacmc May 22, 2024
97ba0ed
Fix vec clone
sashacmc May 23, 2024
9b03aa3
Fix payload processing
sashacmc May 23, 2024
e282da0
fix: allow for NULL z_get payload
jean-roland May 24, 2024
28d0f40
fix: add z_locator_to_zstring to avoid memory leak
jean-roland May 24, 2024
a9c49ab
fix: init value encoding for query/reply
jean-roland May 24, 2024
45fe6f5
chore: format issues
jean-roland May 24, 2024
0e551ad
refactor: rename locator_to_str function
jean-roland May 24, 2024
90cdc5f
feat: add a z_byte_from_owned_byte function
jean-roland May 24, 2024
65be6ca
fix: remove char *z_locator_to_string
jean-roland May 27, 2024
fc93fe5
Merge pull request #3 from jean-roland/fix_string
jean-roland May 27, 2024
7ae284d
feat: add value encoding function
jean-roland May 27, 2024
2858314
fix: update modularity test
jean-roland May 27, 2024
39bf7de
Merge pull request #4 from jean-roland/ft_value_encoding
jean-roland May 27, 2024
1a7c73e
Cleanup TODOs
sashacmc May 27, 2024
7fe58b5
Add RC drop cleanup
sashacmc May 28, 2024
3a2716f
Fixing CI issues (#5)
jean-roland May 28, 2024
b8975a4
Fix sting usage in _zp_*_info_session
sashacmc May 28, 2024
d134bc5
Fix OOM error processing
sashacmc May 28, 2024
c1d56d9
Remove z_reply_data_t from API
sashacmc May 28, 2024
46f0020
Add decode bytes function (#6)
jean-roland May 29, 2024
385261b
Implement correct c++ support in macros.h
sashacmc May 29, 2024
0b9017f
Change keyexpr processing in z_declare_subscriber
sashacmc May 29, 2024
a8c236d
Rename z_str_data to z_string_data
sashacmc May 30, 2024
506a668
z_encoding API rework (#7)
jean-roland Jun 4, 2024
3d0595b
Update doc comments for API functions (#8)
jean-roland Jun 6, 2024
bff998e
feat: update doc (#9)
jean-roland Jun 6, 2024
0c720dd
fix: bad type in doc
jean-roland Jun 6, 2024
9d5f0d6
doc: missing member doc for closure types
jean-roland Jun 7, 2024
eada787
fix: declare manually macro-defined types
jean-roland Jun 7, 2024
f8c12a2
fix: remove ignore type and fix missing new line
jean-roland Jun 7, 2024
bf495cb
fix: format conf.py and add attachment feature
jean-roland Jun 7, 2024
dac0182
Rename z_view_str_wrap to z_view_string_wrap
sashacmc Jun 10, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
33 changes: 13 additions & 20 deletions .github/workflows/build-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,18 @@ jobs:
sudo apt install -y ninja-build
FORCE_C99=ON CMAKE_GENERATOR=Ninja make

raweth_build:
name: Build raweth transport on ubuntu-latest
runs-on: ubuntu-latest
steps:
- name: Checkout code
uses: actions/checkout@v4

- name: Build raweth
run: |
sudo apt install -y ninja-build
Z_FEATURE_RAWETH_TRANSPORT=1 CMAKE_GENERATOR=Ninja make

zenoh_build:
name: Build Zenoh from source
runs-on: ubuntu-latest
Expand Down Expand Up @@ -103,7 +115,7 @@ jobs:
run: |
sudo apt install -y ninja-build
CMAKE_GENERATOR=Ninja make
python3 ./build/tests/modularity.py --pub $Z_FEATURE_PUBLICATION --sub $Z_FEATURE_SUBSCRIPTION --queryable $Z_FEATURE_QUERYABLE --query $Z_FEATURE_QUERY --attachment 1
python3 ./build/tests/modularity.py --pub $Z_FEATURE_PUBLICATION --sub $Z_FEATURE_SUBSCRIPTION --queryable $Z_FEATURE_QUERYABLE --query $Z_FEATURE_QUERY --attachment 0
timeout-minutes: 5
env:
Z_FEATURE_PUBLICATION: ${{ matrix.feature_publication }}
Expand All @@ -115,25 +127,6 @@ jobs:
if: always()
run: kill ${{ steps.run-zenoh.outputs.zenohd-pid }}

raweth_build:
name: Build and test raweth transport on ubuntu-latest
runs-on: ubuntu-latest
strategy:
matrix:
feature_reth: [1, 0]
steps:
- name: Checkout code
uses: actions/checkout@v4

- name: Build project and run test
run: |
sudo apt install -y ninja-build
CMAKE_GENERATOR=Ninja make
python3 ./build/tests/raweth.py --reth $Z_FEATURE_RAWETH_TRANSPORT
timeout-minutes: 5
env:
Z_FEATURE_RAWETH_TRANSPORT: ${{ matrix.feature_reth }}

st_build:
needs: zenoh_build
name: Build and test in single thread on ubuntu-latest
Expand Down
4 changes: 2 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -132,8 +132,8 @@ set(Z_FEATURE_SUBSCRIPTION 1 CACHE STRING "Toggle subscription feature")
set(Z_FEATURE_QUERY 1 CACHE STRING "Toggle query feature")
set(Z_FEATURE_QUERYABLE 1 CACHE STRING "Toggle queryable feature")
set(Z_FEATURE_RAWETH_TRANSPORT 0 CACHE STRING "Toggle raw ethernet transport feature")
set(Z_FEATURE_ATTACHMENT 1 CACHE STRING "Toggle attachment feature")
set(Z_FEATURE_INTEREST 1 CACHE STRING "Toggle interest feature")
set(Z_FEATURE_ATTACHMENT 0 CACHE STRING "Toggle attachment feature")
set(Z_FEATURE_INTEREST 0 CACHE STRING "Toggle interest feature") # Toggle to 1 when protocol changes are merged
add_definition(Z_FEATURE_MULTI_THREAD=${Z_FEATURE_MULTI_THREAD})
add_definition(Z_FEATURE_PUBLICATION=${Z_FEATURE_PUBLICATION})
add_definition(Z_FEATURE_SUBSCRIPTION=${Z_FEATURE_SUBSCRIPTION})
Expand Down
2 changes: 1 addition & 1 deletion GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ Z_FEATURE_PUBLICATION?=1
Z_FEATURE_SUBSCRIPTION?=1
Z_FEATURE_QUERY?=1
Z_FEATURE_QUERYABLE?=1
Z_FEATURE_ATTACHMENT?=1
Z_FEATURE_ATTACHMENT?=0
Z_FEATURE_INTEREST?=1
Z_FEATURE_RAWETH_TRANSPORT?=0

Expand Down
Loading
Loading