alibaba / alibaba/GraphScope

[BUG] compiler building should respect the debug/release arg parsed from makefile

Open
#1,943 0 comments 0 reactions 2 assignees Claimed by @BingqingLyu View on GitHub
bug component:gie
Dominant language
C++
Stars
3.6k
Forks
468
Avg merge
29m
Merged PRs (30d)
1

Description

the makefile has an arg assigning whether it is build in debug/release mode;
i.e., https://github.com/alibaba/GraphScope/blob/main/Makefile#L114

however, it seems when building compiler,
it overlooked this arg, i.e.,
https://github.com/alibaba/GraphScope/blob/main/interactive_engine/executor/ir/compiler/build_ir_core.sh#L8

In addition, I strongly recommand we remove this bash for simplicity,
just invoke `cargo` in the [here](https://github.com/alibaba/GraphScope/blob/main/interactive_engine/executor/ir/compiler/pom.xml#L259) instead.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.