Github user jablko commented on the issue:
https://github.com/apache/trafficserver/pull/1392
@zwoop Yes -- I'm happy to make any changes, but I think it's ready to
land. If you approve, I'll merge it.
---
If your project is set up for it, you can reply to this email and have your
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/1392
Should we land this?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user jablko commented on the issue:
https://github.com/apache/trafficserver/pull/1392
@zwoop I revised this:
```Diff
print_feature("BUILD_PERSON", BUILD_PERSON, json);
print_feature("BUILD_GROUP", BUILD_GROUP, json);
print_feature("BUILD_NUMBER",
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1392
clang-analyzer build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Clang/2/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1392
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1328/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1392
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1435/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user jablko commented on the issue:
https://github.com/apache/trafficserver/pull/1392
Would this latest change work? How does traffic_layout --features get used?
```Diff
print_feature("BUILD_PERSON", BUILD_PERSON, json);
print_feature("BUILD_GROUP",
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1392
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1325/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1392
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1432/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/1392
I suspect we might want to backport a fix for this to 7.1.x as well.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1392
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1313/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1392
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1420/ for details.
---
If your project is set up for it, you can reply to this email and have
12 matches
Mail list logo
