Skip to content

Commit

Permalink
[SAI-PTF]Include sai expermential for generate the rpc headers (#1660) (
Browse files Browse the repository at this point in the history
#1661)

When generate the sai_adapter it depends on the files in  sai expermental
https://github.com/opencomputeproject/SAI/blob/master/meta/Makefile#L72
```
CFLAGS += -I../inc -I../experimental $(WARNINGS)
```
In gensairpc
https://github.com/opencomputeproject/SAI/blob/master/meta/gensairpc.pl#L132
```
our $EXPERIMENTAL_DIR = catdir( $sai_dir,      'experimental' );
```
But when generate the py headers for RPC service, it doesn't include that.
In order to make them matched, then add that dependences.

Test done:
Local compile and checked the headers

Signed-off-by: richardyu-ms <richard.yu@microsoft.com>

Signed-off-by: richardyu-ms <richard.yu@microsoft.com>
  • Loading branch information
richardyu-ms authored Nov 20, 2022
1 parent bee5d7c commit bb50de8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/saithriftv2/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ $(PY_SOURCES): $(METADIR)sai.thrift
$(THRIFT) -o ./ --gen py -r $^

$(SAI_PY_HEADERS): $(SAI_HEADERS)
$(CTYPESGEN) --output-language=py32 -I/usr/include -I$(SAI_HEADER_DIR) --include /usr/include/linux/limits.h $^ -o $@
$(CTYPESGEN) --output-language=py32 -I/usr/include -I$(SAI_HEADER_DIR) -I../../experimental --include /usr/include/linux/limits.h $^ -o $@

$(ODIR)/%.o: gen-cpp/%.cpp meta
$(CXX) $(CPPFLAGS) -c $< -o $@ -I../../meta
Expand Down

0 comments on commit bb50de8

Please sign in to comment.