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

intellij_aspect does not recognize additional_compiler_inputs as a prerequisite #6659

Closed
luyi326 opened this issue Aug 24, 2024 · 3 comments
Closed
Assignees
Labels
awaiting-maintainer Awaiting review from Bazel team on issues lang: c++ C++ rules integration product: CLion CLion plugin type: bug

Comments

@luyi326
Copy link
Contributor

luyi326 commented Aug 24, 2024

Description of the bug:

I'm using CLion with C++ and in one of our cc_library we have @$(execpath {BAZEL_TARGET}) in cops. This target is defined inside additional_compiler_inputs above, and build this rule in command line works fine. But if I run bazel sync, I get something that looks like this:

ERROR: path/to/my/buildBUILD:xx:xx: in @@intellij_aspect//:intellij_info_bundled.bzl%intellij_info_aspect[optimize_building_jars="enabled"] aspect on cc_library rule //path/to/my:target: label '//path/to/rule/in:additional_compiler_inputs' in $(location) expression is not a declared prerequisite of this rule

Which category does this issue belong to?

CLion

What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.

This in BUILD:

cc_library(
    name = "test",
    srcs = [
        ":test.cpp",
    ],
    hdrs = glob(["*.hpp"]),
    additional_compiler_inputs = [
        "//some_target/for/compiler/input",
    ],
    copts = [
        "@$(execpath //some_target/for/compiler/input)",
    ],
)

Which Intellij IDE are you using? Please provide the specific version.

CLion 2024.1.5

What programming languages and tools are you using? Please provide specific versions.

C++

What Bazel plugin version are you using?

Any version

Have you found anything relevant by searching the web?

No

Any other information, logs, or outputs that you want to share?

I have a branch that fixes this issue for me that I'd like to make a pull request on master...luyi326:intellij:yi/cpp

@luyi326 luyi326 added awaiting-maintainer Awaiting review from Bazel team on issues type: bug labels Aug 24, 2024
@github-actions github-actions bot added the product: CLion CLion plugin label Aug 24, 2024
@tpasternak tpasternak added the lang: c++ C++ rules integration label Aug 27, 2024
@luyi326
Copy link
Contributor Author

luyi326 commented Sep 20, 2024

Hello can you let me make a pull request for this? It's a really simple fix

@LeFrosch
Copy link
Collaborator

@luyi326 Hi, I would really appreciate a PR. I can check it on Monday.

@luyi326 luyi326 mentioned this issue Sep 21, 2024
3 tasks
@luyi326
Copy link
Contributor Author

luyi326 commented Sep 21, 2024

#6774

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-maintainer Awaiting review from Bazel team on issues lang: c++ C++ rules integration product: CLion CLion plugin type: bug
Projects
None yet
Development

No branches or pull requests

4 participants