Skip to content

Conversation

@fmeum
Copy link
Collaborator

@fmeum fmeum commented Oct 15, 2024

It doesn't need to be staged for every compile or link action. The action that uses it stages all_files.

It doesn't need to be staged for every compile or link action. The action that uses it stages `all_files`.
@fmeum
Copy link
Collaborator Author

fmeum commented Oct 15, 2024

A local run of buildifier -lint=warn -r . doesn't show any findings, but the CI run fails (without output). Not sure what to do about that.

copybara-service bot pushed a commit that referenced this pull request Oct 30, 2024
PiperOrigin-RevId: 691372917
Change-Id: Idebbc95f1e24990ec61d5224fbc28cd238a8fdbd
@fmeum fmeum closed this Oct 30, 2024
@fmeum fmeum deleted the validate-library branch November 5, 2024 16:41
@comius
Copy link
Collaborator

comius commented Nov 11, 2024

copybara-service bot pushed a commit that referenced this pull request Nov 11, 2024
*** Reason for rollback ***

Breaks cc_builtin_tests

*** Original change description ***

Copybara Merge: #257

BEGIN_PUBLIC
Copybara import of the project:

--
12a8bd5 by Fabian Meumertzheim <fabian@meumertzhe.im>:

Move static library validation helper to `ar_files`

It doesn't need to be staged for every compile or link action. The action that uses it stages `all_files`.

END_PUBLIC

***

PiperOrigin-RevId: 695300596
Change-Id: If10c95c1867716c82a09ad3acdfd479beedcc6c5
@fmeum
Copy link
Collaborator Author

fmeum commented Nov 11, 2024

I'll take a look at some point, but it's not a high priority improvement.

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