Bazel: emboss_cpp_library uses incorrect import path when in an external repository about emboss HOT 6 CLOSED

Maybe --import-dir should be set to external/repo-name instead of "." and " ".

from emboss.

src.path resolves to the right path (external/repo-name/.../src), but src.root resolves to the directory containing "external" and is used for --import-dir. It seems like we want to use "src.root + /external/repo-name" to resolve imports.

from emboss.

Yes, imports = ["--import-dir=" + root.path + "external/pigweed/" for root in dep_roots.to_list()] in _emboss_library_impl works. I'm not sure how to properly calculate "external/pigweed" though.

from emboss.

This seems to work:

    adjusted_src_root = src.root.path
    if src.path.startswith("external/"):
        segments = src.path.split("/")
        first_two_segments = "/".join(segments[:3])
        adjusted_src_root = paths.dirname(first_two_segments)

    dep_roots = depset(
        direct = [adjusted_src_root],
        transitive = [dep.transitive_roots for dep in deps],
    )
    imports = ["--import-dir=" + root for root in dep_roots.to_list()]

from emboss.

That's sort of horrible, but probably good enough for now.

It would probably be worth reaching out to the Bazel team to see if they have any suggestions -- I was basically trying to copy the aspect for Proto, but that aspect is allowlisted for a bunch of interfaces that I could not use directly.

from emboss.

Sounds like proto has the same problem. I found a similar solution on stackoverflow:
https://stackoverflow.com/questions/55796252/how-to-write-bazel-rules-that-work-with-external-repositories

from emboss.