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

This removes test project ID that will conflict with modular ID. #527

Merged
merged 1 commit into from Mar 4, 2024

Conversation

grafikrobot
Copy link
Member

@grafikrobot grafikrobot commented Mar 2, 2024

This removes the superfluous test project ID that will conflict with the modular project IDs for all libraries. When libraries are modular they will have the standard global ID of /boost/lib-name. Having colliding IDs will cause build errors.

#modular-boost https://github.com/users/grafikrobot/projects/1

@ldionne
Copy link
Member

ldionne commented Mar 4, 2024

I am not sure whether you want this merged right away or whether you need to wait before you do it -- please merge at your discretion. Thanks!

@grafikrobot
Copy link
Member Author

@ldionne I don't have permission to merge. And, yes, it would be great if it gets merged as soon as possible.

@ldionne ldionne merged commit e66cd6b into boostorg:master Mar 4, 2024
3 checks passed
@grafikrobot
Copy link
Member Author

PS. I just noticed I created the PR for master. Can you also apply it to the develop branch?

@ldionne
Copy link
Member

ldionne commented Mar 4, 2024

Everything happens on master nowadays, develop isn't used and has gotten behind.

@pdimov
Copy link
Member

pdimov commented Mar 4, 2024

Well that's not supposed to happen, develop should at least be kept in sync, even if unused.

@ldionne
Copy link
Member

ldionne commented Mar 4, 2024

I just synced the branches.

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.

None yet

3 participants