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

allow tag feature for virtual-targets with exact name or without type #393

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

grisumbras
Copy link
Contributor

Proposed changes

Allow <tag> feature to run on virtual targets with exact name or without type.

Currently <tag> is disabled on make targets and other targets with exact name or without type. This makes it unnecessary hard to create targets which has name that depends on the property set. I looked at the history of this behaviour, and it seems to be a remnant of not adding type affixes to sucha targets. At one point <tag> handling was added, but the check for having a type and not having an exact name remained.

Types of changes

  • New feature (non-breaking change which adds functionality)

Alternative

Alternative approach is to keep <tag> disabled when name is exact, and just change make targets to not set name to exact.

@grafikrobot
Copy link
Member

The tag test fails with this change. Can you look into that?

@grisumbras
Copy link
Contributor Author

grisumbras commented May 24, 2024

That's boostcpp.tag, right? I have a PR (boostorg/boost#908) to make it do nothing for untyped targets.

@grafikrobot
Copy link
Member

But thank you for pointing out the Boost PR. I'll have to wait to merge this one until that is resolved.

@grisumbras grisumbras force-pushed the feature/allow-tag-for-exact branch 2 times, most recently from a1cea49 to 7b91f06 Compare May 27, 2024 07:07
@grafikrobot grafikrobot force-pushed the feature/allow-tag-for-exact branch from 7b91f06 to 36cd684 Compare June 11, 2024 01:36
@grafikrobot grafikrobot force-pushed the feature/allow-tag-for-exact branch from 9182238 to d56e361 Compare November 9, 2024 03:56
@grafikrobot
Copy link
Member

That's boostcpp.tag, right? I have a PR (boostorg/boost#908) to make it do nothing for untyped targets.

I guess the problem is that's not a change that can be applied to older Boost versions. Hence with this b2 change all past Boost versions fail. As can be seen in the tests.

@grisumbras
Copy link
Contributor Author

To be honest, I have second thoughts about this PR.

@grafikrobot grafikrobot marked this pull request as draft November 9, 2024 19:30
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