Skip to content

Fix bzl_mod symlink issues in doxygen rule #171

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

Merged

Conversation

martin4861
Copy link
Contributor

@martin4861 martin4861 commented Aug 12, 2025

With the introduction of bzl_mod the doxygen rule was broken and needs to be fixed. This means splitting up the preprocessing of doxygen input files into smaller actions.

  • Test with Bazel 7 in orion-engine
  • Test with Bazel 6 in starling-core

@martin4861 martin4861 marked this pull request as ready for review August 12, 2025 15:32
@martin4861 martin4861 requested a review from a team as a code owner August 12, 2025 15:32
@martin4861 martin4861 marked this pull request as draft August 12, 2025 15:39
@martin4861 martin4861 force-pushed the martin4861/fix-bazelmod-symlink-issues-in-doxygen-rule branch from 7dcca1d to a5e9ea6 Compare August 13, 2025 07:01
@martin4861 martin4861 force-pushed the martin4861/fix-bazelmod-symlink-issues-in-doxygen-rule branch from a5e9ea6 to 8b57c4f Compare August 13, 2025 07:54
@martin4861 martin4861 marked this pull request as ready for review August 13, 2025 08:05
@martin4861 martin4861 requested a review from armallen August 13, 2025 09:22
@martin4861 martin4861 requested a review from reimerix August 13, 2025 10:32
@martin4861 martin4861 changed the title Fix bzl_mod symlink issues in doxygen rule [WIP] Fix bzl_mod symlink issues in doxygen rule Aug 13, 2025
@martin4861 martin4861 requested a review from sbmueller August 14, 2025 07:24
@martin4861 martin4861 merged commit 049eaca into main Aug 14, 2025
1 check passed
@martin4861 martin4861 deleted the martin4861/fix-bazelmod-symlink-issues-in-doxygen-rule branch August 14, 2025 07:54
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.

3 participants