Closed teto closed 2 years ago
There are a relatively finite list of flags that we pass down to Bazel. Since we have a list of them, we can handle them in most positions in argv. You should be able to just pass most of them, but if the flag you want to pass isn't in the list, you can add it here in a pr
why not pass forward all flags ibazel doesn't recognize ? adding them one by one seems like a lost cause, especially for a programs like bazel with billions flags that keep changing because google doesn't care about backwards compability. I dont know go but surely it's possible to at least for now mentions "unsupported flag" instead of:
panic: runtime error: index out of range [0] with length 0
goroutine 1 [running]:
main.handle(0xc000192000, {0x7ffcc278c268, 0x3}, {0xc00013c020, 0x0, 0x1010123b4874108})
ibazel/main.go:197 +0x2be
main.main()
ibazel/main.go:182 +0x2cc
(or maybe it's fixed on master, sry I have't tried)
If you'd like to revisit the strategy for flag handling, I would love a PR to improve it! How can I help out with that process?
thanks for the offer but I have 50 open PRs, I will try to merge those first. Just keep the issue open in case someone with more abilities/availibility want to take it on ?
There is a pretty good system in place to pass flags that are already in that list. If it's needed, let's open another issue and discuss a design.
I dont understand why you closed this ? why do we need to whitelist flags ? A change is needed hence the issue, closing the ticket wont make the problem go away.
Describe the bug I would like to be able to forward arguments to bazel (with ibazel Version 0.14.0).
Reproduction instructions Private repo but my previous command should work with any target
Expected behavior Can pass arguments to bazel
Version (please complete the following information):
ibazel 2>&1 | head -n 1
to get this) .14.0bazel version
to get this. Please include all the lines) 4.2.2Additional context I've found related issues in the tracker but no solution/workaround yet (putting the command in the bazelrc does not work for this scenario.