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

Remove -w dupbuild completely, always error on duplicate edges #2356

Merged
merged 2 commits into from
Dec 7, 2023

Conversation

jhasse
Copy link
Collaborator

@jhasse jhasse commented Nov 29, 2023

Step 5, fixes #931.

@jhasse jhasse added the cleanup label Nov 29, 2023
@jhasse jhasse added this to the 1.12.0 milestone Nov 29, 2023
@jhasse jhasse merged commit 9772a29 into ninja-build:master Dec 7, 2023
10 checks passed
JeffyCN added a commit to JeffyCN/meta-rockchip that referenced this pull request May 8, 2024
The '-w dupbuild' has been removed in newest ninja
see ninja-build/ninja#2356

Change-Id: I1303a1e4aa85550667ced33184c929d0a691f16f
Signed-off-by: Jeffy Chen <[email protected]>
@jhasse jhasse deleted the remove-w-dupbuild branch August 28, 2024 07:15
birunts added a commit to ARM-software/bob-build that referenced this pull request Sep 19, 2024
Newer ninja vesrions do not support `dupbuild` flag anymore:
ninja-build/ninja#2356

Put `dupbuild` flag only if ninja supports it.
This forces to apply patch on archived `blueprint` submodule.

Change-Id: I0210375bb1b5f23b3a6bb3cfe8674ed08d98a6ae
Signed-off-by: Sebastian Birunt <[email protected]>
birunts added a commit to ARM-software/bob-build that referenced this pull request Sep 19, 2024
Newer ninja vesrions do not support `dupbuild` flag anymore:
ninja-build/ninja#2356

Put `dupbuild` flag only if ninja supports it.
This forces to apply patch on archived `blueprint` submodule.

Change-Id: I0210375bb1b5f23b3a6bb3cfe8674ed08d98a6ae
Signed-off-by: Sebastian Birunt <[email protected]>
birunts added a commit to ARM-software/bob-build that referenced this pull request Oct 7, 2024
Newer `ninja-build` does not have `dupbuild`
warning flag anymore:
ninja-build/ninja#2356
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Turn duplicate rules for the same target into an error
2 participants