You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by "bneradt (via GitHub)" <gi...@apache.org> on 2023/04/07 15:24:43 UTC
[GitHub] [trafficserver] bneradt opened a new pull request, #9588: swoc/TextView.h: add cstdint include
bneradt opened a new pull request, #9588:
URL: https://github.com/apache/trafficserver/pull/9588
This fixes clang-16 builds by adding cstdint for intmax_t and related
definitions. This has a corresponding libswoc PR:
https://github.com/SolidWallOfCode/libswoc/pull/84
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] bneradt commented on pull request #9588: swoc/TextView.h: add cstdint include
Posted by "bneradt (via GitHub)" <gi...@apache.org>.
bneradt commented on PR #9588:
URL: https://github.com/apache/trafficserver/pull/9588#issuecomment-1500404346
> I thought we don't touch external library snapshots like this. Do we?
I have a libswoc PR with this patch here: https://github.com/SolidWallOfCode/libswoc/pull/84 which Alan has merged in.
But I'd like to get a quick cherry-pick with that in locally so that I can enable fedora:38 builds in CI.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] bneradt commented on pull request #9588: swoc/TextView.h: add cstdint include
Posted by "bneradt (via GitHub)" <gi...@apache.org>.
bneradt commented on PR #9588:
URL: https://github.com/apache/trafficserver/pull/9588#issuecomment-1500990125
This patch got included in https://github.com/apache/trafficserver/pull/9592
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] maskit commented on pull request #9588: swoc/TextView.h: add cstdint include
Posted by "maskit (via GitHub)" <gi...@apache.org>.
maskit commented on PR #9588:
URL: https://github.com/apache/trafficserver/pull/9588#issuecomment-1500391425
I thought we don't touch external library snapshots like this. Do we?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] bneradt closed pull request #9588: swoc/TextView.h: add cstdint include
Posted by "bneradt (via GitHub)" <gi...@apache.org>.
bneradt closed pull request #9588: swoc/TextView.h: add cstdint include
URL: https://github.com/apache/trafficserver/pull/9588
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org