-
-
Notifications
You must be signed in to change notification settings - Fork 135
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
feat: implement structure_test #31
Conversation
ea4e833
to
ede2d6c
Compare
c63bea7
to
b073841
Compare
b073841
to
12ef4d0
Compare
8d39609
to
4989f92
Compare
4989f92
to
d64c23c
Compare
dbb7d89
to
66c966d
Compare
# for why this is needed. | ||
_LAUNCHER_TMPL = """ | ||
export BAZEL_BINDIR=. | ||
source {executable_path} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why does it use source
instead of exec
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1- avoid $0 type of bugs. RUNFILES helper relies on that heavily.
2- any future implicit reliance of executable_path being the entrypoint.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tried with exec. it doesn't work with it.
d4fa9d9
to
3658ad2
Compare
3658ad2
to
1ab00e6
Compare
7ceb53c
to
6b518cc
Compare
6b518cc
to
d1bbc21
Compare
@thesayyn WDYT about naming it |
No description provided.