Skip to content

Conversation

@kekxv
Copy link

@kekxv kekxv commented Nov 30, 2025

No description provided.

@kekxv
Copy link
Author

kekxv commented Nov 30, 2025

image

@kekxv
Copy link
Author

kekxv commented Nov 30, 2025

#401

@kekxv
Copy link
Author

kekxv commented Nov 30, 2025

@armandomontanez
Even though the method is a bit strange, it works really well.

@kekxv kekxv marked this pull request as draft December 1, 2025 01:44
@armandomontanez
Copy link
Collaborator

Hmmm I'd like to see if we could be more concise with this. I'd expect this to be the most important part:

  filegroup(
      name = "multicall_support_files",
      srcs = glob([
+         "bin/**",
          "libexec/**/*",
      ]),
  )

Theoretically, you should be able to use cc_args directory formatting to expose the directory path:

cc_args(
    name = "toolchain_exec_prefix",
    actions = [
        "@rules_cc//toolchains/actions:compile_actions",
        "@rules_cc//toolchains/actions:link_actions",
    ],
    args = [
        "-D",
        "{gcc_exec_prefix}",
    ],
    format = {"gcc_exec_prefix": "@gcc-linux-x86_64//:toolchain_root"},
)

@kekxv
Copy link
Author

kekxv commented Dec 2, 2025

@armandomontanez Do you have time to process this? When will it be ready? We've been waiting for so long. 😞

@armandomontanez
Copy link
Collaborator

Sorry for taking so long. I'm not super sure we need all of this. I'm happy to help figure out how to reduce this to be a bit more constrained to exactly what's needed. Do you have a quick way to reproduce the specific problems you're encountering and hoping to solve with this?

@kekxv
Copy link
Author

kekxv commented Dec 3, 2025

@armandomontanez The main issue right now is that the GCC within the rule_based_toolchain isn't working properly. I need to use a shell script to filter inputs or perform some other operations to get it to work.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants