Skip to content

Conversation

@SolidWallOfCode
Copy link
Member

This is needed both to remove redundancy and to avoid circular dependencies, as libswoc internals depend on the functions in the string_view utils.

Copy link
Contributor

@bneradt bneradt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@bneradt
Copy link
Contributor

bneradt commented Nov 17, 2022

The builds are failing due to linking issues. Here's a link to the Ubuntu failure:
https://ci.trafficserver.apache.org/job/Github_Builds/job/ubuntu/389/console

@SolidWallOfCode SolidWallOfCode force-pushed the ts-swoc-string-view-util branch from aeb310f to 1ccd30a Compare November 17, 2022 19:57
@SolidWallOfCode
Copy link
Member Author

Oh, I'm tracking the linking issues. The problem is all of that works perfectly locally. This is part of resolving the linking issues with #9193 .

@SolidWallOfCode SolidWallOfCode force-pushed the ts-swoc-string-view-util branch from 1ccd30a to 7d05fc4 Compare November 17, 2022 22:15
@SolidWallOfCode SolidWallOfCode force-pushed the ts-swoc-string-view-util branch from 7d05fc4 to 922c69d Compare November 17, 2022 23:56
@SolidWallOfCode
Copy link
Member Author

Seriously? That fix worked? Gaaaaaaaaaaaaaaaaaaaaaaah!

@SolidWallOfCode SolidWallOfCode merged commit ce009fb into apache:10-Dev Nov 18, 2022
@zwoop zwoop modified the milestones: 10-Dev, 10.0.0 Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants