[GitHub] trafficserver issue #1492: Fix for ignored return value in Ubuntu builds.

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1492
  
clang-analyzer build *successful*! See 
https://ci.trafficserver.apache.org/job/clang-analyzer-github/181/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1492: Fix for ignored return value in Ubuntu builds.

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1492
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/linux-github/1511/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1492: Fix for ignored return value in Ubuntu builds.

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1492
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/freebsd-github/1616/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1492: Fix for ignored return value in Ubuntu builds.

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1492
  
Intel CC build *successful*! See 
https://ci.trafficserver.apache.org/job/icc-github/49/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1471: is_transparent_passthrough_allowed always returns...

2017-02-24 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the issue:

https://github.com/apache/trafficserver/pull/1471
  
Looks good.


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver pull request #1492: Fix for ignored return value in Ubuntu bui...

2017-02-24 Thread SolidWallOfCode
GitHub user SolidWallOfCode opened a pull request:

https://github.com/apache/trafficserver/pull/1492

Fix for ignored return value in Ubuntu builds.

I think this is the problem with the Ubuntu builds.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/SolidWallOfCode/trafficserver 
i-agree-with-phil-5

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/trafficserver/pull/1492.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1492


commit 0de98efeed88979d0439171513d3df467a51312f
Author: Alan M. Carroll 
Date:   2017-02-25T06:48:45Z

Fix for ignored return value in Ubuntu builds.




---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1479: Add static type checking to configuration overrid...

2017-02-24 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the issue:

https://github.com/apache/trafficserver/pull/1479
  
Gah, what a tarpit. The conflict resolution collided with another override 
update and that caused the regression test failure. Should be fixed now.


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1479: Add static type checking to configuration overrid...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1479
  
clang-analyzer build *successful*! See 
https://ci.trafficserver.apache.org/job/clang-analyzer-github/180/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1479: Add static type checking to configuration overrid...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1479
  
Linux build *failed*! See 
https://ci.trafficserver.apache.org/job/linux-github/1510/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1479: Add static type checking to configuration overrid...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1479
  
FreeBSD build *failed*! See 
https://ci.trafficserver.apache.org/job/freebsd-github/1615/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1479: Add static type checking to configuration overrid...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1479
  
Intel CC build *failed*! See 
https://ci.trafficserver.apache.org/job/icc-github/48/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1479: Add static type checking to configuration overrid...

2017-02-24 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the issue:

https://github.com/apache/trafficserver/pull/1479
  
I guess. I don't really think this is all that much better than the 
template logic, except that it's easier to understand for non template people.


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1491: MemView: Fix strlen issue for OmniOS.

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1491
  
clang-analyzer build *successful*! See 
https://ci.trafficserver.apache.org/job/clang-analyzer-github/179/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1491: MemView: Fix strlen issue for OmniOS.

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1491
  
Intel CC build *successful*! See 
https://ci.trafficserver.apache.org/job/icc-github/47/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1491: MemView: Fix strlen issue for OmniOS.

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1491
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/linux-github/1509/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1491: MemView: Fix strlen issue for OmniOS.

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1491
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/freebsd-github/1614/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver pull request #1491: MemView: Fix strlen issue for OmniOS.

2017-02-24 Thread SolidWallOfCode
GitHub user SolidWallOfCode opened a pull request:

https://github.com/apache/trafficserver/pull/1491

MemView: Fix strlen issue for OmniOS.

Rather than put `ink_platform.h` in the header file, the construct using 
`strlen` was moved to the implementation file and `ink_platform.h` included 
there. The reason for this it is very important to me to be able to use these 
classes in plugins and that isn't possible if `ink_platform.h` is a required 
header.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/SolidWallOfCode/trafficserver MemView-2

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/trafficserver/pull/1491.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1491


commit 9687b4c9244c994086ea22399e4f4975ea40fe16
Author: Alan M. Carroll 
Date:   2017-02-25T01:30:25Z

MemView: Fix strlen issue for OmniOS.




---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1465: Add new testing system to the test subdirectory

2017-02-24 Thread dragon512
Github user dragon512 commented on the issue:

https://github.com/apache/trafficserver/pull/1465
  
I am not sure why it should be under "ci" tree. That is stuff for the open 
source CI system. Why would tests for ATS go under the ci data file? Tests are 
for developer, not the ci system. Sure the ci system would run it, much like it 
build code outside the ci tree. We don't put our source under ci. With the TSQA 
code it is confusing that you look there, and what you should run. I want to 
clean this up.

The "tests" tree should contain all the test for ats, including unit tests. 
I had made arguments for clarify it would be much better is all the source was 
under a src/ tree much like the documentation ->doc/, examples -> example/, 
contribution->contrib/  and plugins -> plugins/ are under there own tree. If 
the concern is creating an extra directory at the top level. I would say that 
if we really want to clean this up we should move lib, proxy, mgmt, iocore, cmd 
under a src/ directory. I am not the only one that has echoed this 
point-of-view as such a layout would greatly clean up the tree and make it more 
obvious out-of-the-box where to start looking


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1465: Add new testing system to the test subdirectory

2017-02-24 Thread zwoop
Github user zwoop commented on the issue:

https://github.com/apache/trafficserver/pull/1465
  
Should this be in the "ci" tree rather that its own top-level tree? 


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1479: Add static type checking to configuration overrid...

2017-02-24 Thread zwoop
Github user zwoop commented on the issue:

https://github.com/apache/trafficserver/pull/1479
  
@SolidWallOfCode If you fix the merge conflicts now, I'll +1 this against 
my better judgement :-).


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1490: Inconsistent "exit" strategy on remap.config (and...

2017-02-24 Thread zwoop
GitHub user zwoop opened an issue:

https://github.com/apache/trafficserver/issues/1490

Inconsistent "exit" strategy on remap.config (and possibly other 
configurations)

It seems, remap.config will allow some bad configurations to still startup, 
but not others. This seems inconsistent, and we should make it behave. I think 
(but not sure) that our desired behavior in general is:

1. Don't startup with a bad configuration

2. Don't switch (config reload) from a good configuration to a bad 
configuration, instead keep using the old one.






---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver pull request #1488: This allows old ssl_multicert.config to st...

2017-02-24 Thread zwoop
Github user zwoop closed the pull request at:

https://github.com/apache/trafficserver/pull/1488


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1489: Change default for proxy.config.ssl.server.multic...

2017-02-24 Thread zwoop
GitHub user zwoop opened an issue:

https://github.com/apache/trafficserver/issues/1489

Change default for proxy.config.ssl.server.multicert.exit_on_load_fail to 
"1"

The default behavior is inconsistent with how the rest of ATS works, so I 
think we should change this to "1".






---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1488: This allows old ssl_multicert.config to still fun...

2017-02-24 Thread zwoop
Github user zwoop commented on the issue:

https://github.com/apache/trafficserver/pull/1488
  
I'm gonna close this PR for now, I think this code could be cleaned up, but 
setting proxy.config.ssl.server.multicert.exit_on_load_fail=1 seems to behave 
more reasonable than the defaults.


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1427: Crash in ATSConsistentHash::lookup_by_hashval

2017-02-24 Thread randall
Github user randall commented on the issue:

https://github.com/apache/trafficserver/issues/1427
  
@jrushford This is with OEL 6.5 and gcc 4.9.3. The environment and compiler 
for us is still the same between running 6.1.x and 7.0.0.

I can't attach the full parent config, but it's something like this:
```
dest_domain=subdomain.domain.com scheme=https 
parent="parent1.subdomain.domain.com:443,parent3.subdomain.domain.com:443,parent5.subdomain.domain.com:443,parent5.subdomain.domain.com:443"
 
secondary_parent="parent2.subdomain.domain.com:443,parent4.subdomain.domain.com:443,parent6.subdomain.domain.com:443,parent8.subdomain.domain.com:443"
 round_robin=consistent_hash go_direct=false
dest_domain=subdomain.domain.com scheme=http 
parent="parent1.subdomain.domain.com:80,parent3.subdomain.domain.com:80,parent5.subdomain.domain.com:80,parent5.subdomain.domain.com:80"
 
secondary_parent="parent2.subdomain.domain.com:80,parent4.subdomain.domain.com:80,parent6.subdomain.domain.com:80,parent8.subdomain.domain.com:80"
 round_robin=consistent_hash go_direct=false
```

It's around 80 lines of that (split between http and https)



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1488: This allows old ssl_multicert.config to still fun...

2017-02-24 Thread zwoop
Github user zwoop commented on the issue:

https://github.com/apache/trafficserver/pull/1488
  
I see now, there's a new configuration, 
proxy.config.ssl.server.multicert.exit_on_load_fail, which would exit on load 
errors. Surprised that this is not the default to "1" though.

However, the fix above would retain (what I think is) correct behavior on 
reloads, and if proxy.config.ssl.server.multicert.exit_on_load_fail is off (0), 
you'd still be able to load the initially broken configuration.

Why anyone wants a broken configuration is beyond me, but I'm ok with this, 
but think it should be changed to "1" for v8.0.0.


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver pull request #1488: This allows old ssl_multicert.config to st...

2017-02-24 Thread zwoop
Github user zwoop commented on a diff in the pull request:

https://github.com/apache/trafficserver/pull/1488#discussion_r103039976
  
--- Diff: iocore/net/SSLUtils.cc ---
@@ -2007,7 +2007,10 @@ SSLParseCertificateConfiguration(const 
SSLConfigParams *params, SSLCertLookup *l
 if (ssl_extract_certificate(_info, sslMultiCertSettings)) {
   // There must be a certificate specified unless the tunnel 
action is set
   if (sslMultiCertSettings.cert || sslMultiCertSettings.opt != 
SSLCertContext::OPT_TUNNEL) {
-ssl_store_ssl_context(params, lookup, );
+if (ssl_store_ssl_context(params, lookup, 
) == nullptr) {
+  Error("failed to load SSL server contexts");
+  return false;
+}
   } else {
--- End diff --

I'd be fine adding additional actionable checks here if you like, I'm just 
not familiar with most of this code.


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver pull request #1488: This allows old ssl_multicert.config to st...

2017-02-24 Thread zwoop
Github user zwoop commented on a diff in the pull request:

https://github.com/apache/trafficserver/pull/1488#discussion_r103039820
  
--- Diff: iocore/net/SSLUtils.cc ---
@@ -2007,7 +2007,10 @@ SSLParseCertificateConfiguration(const 
SSLConfigParams *params, SSLCertLookup *l
 if (ssl_extract_certificate(_info, sslMultiCertSettings)) {
   // There must be a certificate specified unless the tunnel 
action is set
   if (sslMultiCertSettings.cert || sslMultiCertSettings.opt != 
SSLCertContext::OPT_TUNNEL) {
-ssl_store_ssl_context(params, lookup, );
+if (ssl_store_ssl_context(params, lookup, 
) == nullptr) {
+  Error("failed to load SSL server contexts");
+  return false;
+}
   } else {
--- End diff --

Yeh, it's not used, but the point is that we stop parsing the configuration 
at this point, which leaves lookup->is_valid == false (correctly). I believe in 
the old version, the status of the lookup is the status of the *last* 
certificate in the configuration.


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1488: This allows old ssl_multicert.config to still fun...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1488
  
clang-analyzer build *successful*! See 
https://ci.trafficserver.apache.org/job/clang-analyzer-github/178/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver pull request #1488: This allows old ssl_multicert.config to st...

2017-02-24 Thread shinrich
Github user shinrich commented on a diff in the pull request:

https://github.com/apache/trafficserver/pull/1488#discussion_r103023456
  
--- Diff: iocore/net/SSLUtils.cc ---
@@ -2007,7 +2007,10 @@ SSLParseCertificateConfiguration(const 
SSLConfigParams *params, SSLCertLookup *l
 if (ssl_extract_certificate(_info, sslMultiCertSettings)) {
   // There must be a certificate specified unless the tunnel 
action is set
   if (sslMultiCertSettings.cert || sslMultiCertSettings.opt != 
SSLCertContext::OPT_TUNNEL) {
-ssl_store_ssl_context(params, lookup, );
+if (ssl_store_ssl_context(params, lookup, 
) == nullptr) {
+  Error("failed to load SSL server contexts");
+  return false;
+}
   } else {
--- End diff --

I'm a bit confused why this fixes anything.  I don't see any callers 
checking the return value.


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1488: This allows old ssl_multicert.config to still fun...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1488
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/linux-github/1508/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1488: This allows old ssl_multicert.config to still fun...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1488
  
Intel CC build *successful*! See 
https://ci.trafficserver.apache.org/job/icc-github/46/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1465: Add new testing system to the test subdirectory

2017-02-24 Thread dragon512
Github user dragon512 commented on the issue:

https://github.com/apache/trafficserver/pull/1465
  
I just pushed a fix for CentOs and RHEL system to bootstrap better. I 
believe this should take care of the issue @jpeach reported given was running 
on one of those systems


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1465: Add new testing system to the test subdirectory

2017-02-24 Thread dragon512
Github user dragon512 commented on the issue:

https://github.com/apache/trafficserver/pull/1465
  
@bryancall So from what I can see at the moment. master is broken. 7.1 is 
working fine minus the custom logfile test. The remap_http and header-rewrite 
fail with

- < Proxy-Connection: keep-alive
   ?   --
   
   + < Connection: keep-alive
 < Server: ATS/7.1.0

This shows ATS is returning something different (ie we had Proxy-Connection 
now it is Connection)

 > GET http://www.example.com:80/ HTTP/1.1
 > Host: www.example.com
 > User-Agent: curl/7.50.1
 > Accept: */*
   + > 
   - > Proxy-Connection: Keep-Alive
   - ``
 < HTTP/1.1 200 OK
 < Date: Fri, 24 Feb 2017 19:45:35 GMT
 < Age: 0
 < Transfer-Encoding: chunked
   - < Proxy-Connection: keep-alive
   ?   --
   
   + < Connection: keep-alive
 < Server: ATS/7.1.0
 < 
 { [5 bytes data]

which says we changed the header order and case change of Keep-Alive to 
keep-alive.

ATS 6.2 did not error, but 6.2.1 errors the same as 7.1


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1488: This allows old ssl_multicert.config to still fun...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1488
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/freebsd-github/1613/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver pull request #1488: This allows old ssl_multicert.config to st...

2017-02-24 Thread zwoop
GitHub user zwoop opened a pull request:

https://github.com/apache/trafficserver/pull/1488

This allows old ssl_multicert.config to still function on reload

The problem is that if a certificate fails to load, for whatever reason 
that might be, ATS still switches the configuration, leaving the server in a 
crippled state. This patch fixes this, to the normal behavior of ATS retaining 
the old configurations if the new fails to load.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/zwoop/trafficserver FixSSLLoadFailure

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/trafficserver/pull/1488.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1488


commit 2f636d23c493039626fae51c7cd37047140bb657
Author: Leif Hedstrom 
Date:   2017-02-24T19:47:29Z

This allows old ssl_multicert.config to still function on reload




---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1209: Remove unused and never to be used classes from M...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1209
  
clang-analyzer build *successful*! See 
https://ci.trafficserver.apache.org/job/clang-analyzer-github/177/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Build failed in Jenkins: ubuntu_16_10-master » gcc,ubuntu_16_10,release #144

2017-02-24 Thread jenkins
See 


--
[...truncated 228.96 KB...]
depbase=`echo Diags.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT Diags.lo -MD -MP -MF $depbase.Tpo -c -o 
Diags.lo ../../../lib/ts/Diags.cc &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT Diags.lo -MD -MP -MF .deps/Diags.Tpo -c 
../../../lib/ts/Diags.cc  -fPIC -DPIC -o .libs/Diags.o
depbase=`echo EventNotify.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT EventNotify.lo -MD -MP -MF $depbase.Tpo -c -o 
EventNotify.lo ../../../lib/ts/EventNotify.cc &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT EventNotify.lo -MD -MP -MF 
.deps/EventNotify.Tpo -c ../../../lib/ts/EventNotify.cc  -fPIC -DPIC -o 
.libs/EventNotify.o
depbase=`echo fastlz.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CC   --mode=compile ccache cc -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=gnu99 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror -mcx16  -MT 
fastlz.lo -MD -MP -MF $depbase.Tpo -c -o fastlz.lo ../../../lib/ts/fastlz.c &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache cc -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=gnu99 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror -mcx16 -MT 
fastlz.lo -MD -MP -MF .deps/fastlz.Tpo -c ../../../lib/ts/fastlz.c  -fPIC -DPIC 
-o .libs/fastlz.o
depbase=`echo Hash.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 

[GitHub] trafficserver issue #1264: TS-3692: Remove code associated with proxy.config...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1264
  
clang-analyzer build *successful*! See 
https://ci.trafficserver.apache.org/job/clang-analyzer-github/176/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Build failed in Jenkins: ubuntu_14_04-master » gcc,ubuntu_14_04,release #2460

2017-02-24 Thread jenkins
See 


--
[...truncated 224.27 KB...]
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ 
-DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib  -Dlinux 
-D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT BaseLogFile.lo -MD -MP -MF $depbase.Tpo -c -o 
BaseLogFile.lo ../../../lib/ts/BaseLogFile.cc &&\
mv -f $depbase.Tpo $depbase.Plo
depbase=`echo Bitops.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ 
-DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib  -Dlinux 
-D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT Bitops.lo -MD -MP -MF $depbase.Tpo -c -o 
Bitops.lo ../../../lib/ts/Bitops.cc &&\
mv -f $depbase.Tpo $depbase.Plo
depbase=`echo ConsistentHash.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ 
-DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib  -Dlinux 
-D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT ConsistentHash.lo -MD -MP -MF $depbase.Tpo -c 
-o ConsistentHash.lo ../../../lib/ts/ConsistentHash.cc &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT Bitops.lo -MD -MP -MF .deps/Bitops.Tpo -c 
../../../lib/ts/Bitops.cc  -fPIC -DPIC -o .libs/Bitops.o
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT Arena.lo -MD -MP -MF .deps/Arena.Tpo -c 
../../../lib/ts/Arena.cc  -fPIC -DPIC -o .libs/Arena.o
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT BaseLogFile.lo -MD -MP -MF 
.deps/BaseLogFile.Tpo -c ../../../lib/ts/BaseLogFile.cc  -fPIC -DPIC -o 
.libs/BaseLogFile.o
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 

[GitHub] trafficserver issue #1209: Remove unused and never to be used classes from M...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1209
  
FreeBSD build *failed*! See 
https://ci.trafficserver.apache.org/job/freebsd-github/1612/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Build failed in Jenkins: ubuntu_16_04-master » gcc,ubuntu_16_04,release #593

2017-02-24 Thread jenkins
See 


--
[...truncated 227.54 KB...]
depbase=`echo ContFlags.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT ContFlags.lo -MD -MP -MF $depbase.Tpo -c -o 
ContFlags.lo ../../../lib/ts/ContFlags.cc &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT ContFlags.lo -MD -MP -MF .deps/ContFlags.Tpo 
-c ../../../lib/ts/ContFlags.cc  -fPIC -DPIC -o .libs/ContFlags.o
depbase=`echo Diags.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT Diags.lo -MD -MP -MF $depbase.Tpo -c -o 
Diags.lo ../../../lib/ts/Diags.cc &&\
mv -f $depbase.Tpo $depbase.Plo
depbase=`echo EventNotify.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT EventNotify.lo -MD -MP -MF $depbase.Tpo -c -o 
EventNotify.lo ../../../lib/ts/EventNotify.cc &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT Diags.lo -MD -MP -MF .deps/Diags.Tpo -c 
../../../lib/ts/Diags.cc  -fPIC -DPIC -o .libs/Diags.o
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT EventNotify.lo -MD -MP -MF 
.deps/EventNotify.Tpo -c ../../../lib/ts/EventNotify.cc  -fPIC -DPIC -o 
.libs/EventNotify.o
depbase=`echo fastlz.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CC   --mode=compile ccache cc -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 

Build failed in Jenkins: ubuntu_12_04-master » gcc,ubuntu_12_04,release #2232

2017-02-24 Thread jenkins
See 


--
[...truncated 230.02 KB...]
libtool: compile:  ccache /usr/bin/g++-4.9 -DHAVE_CONFIG_H -I. 
-I../../../lib/ts -I../../lib -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.5 
-I
 -D_GNU_SOURCE -I/usr/include -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall 
-Wextra -Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT Arena.lo -MD -MP -MF .deps/Arena.Tpo -c 
../../../lib/ts/Arena.cc  -fPIC -DPIC -o .libs/Arena.o
libtool: compile:  ccache /usr/bin/g++-4.9 -DHAVE_CONFIG_H -I. 
-I../../../lib/ts -I../../lib -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.5 
-I
 -D_GNU_SOURCE -I/usr/include -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall 
-Wextra -Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT Bitops.lo -MD -MP -MF .deps/Bitops.Tpo -c 
../../../lib/ts/Bitops.cc  -fPIC -DPIC -o .libs/Bitops.o
libtool: compile:  ccache /usr/bin/g++-4.9 -DHAVE_CONFIG_H -I. 
-I../../../lib/ts -I../../lib -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.5 
-I
 -D_GNU_SOURCE -I/usr/include -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall 
-Wextra -Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT ConsistentHash.lo -MD -MP -MF 
.deps/ConsistentHash.Tpo -c ../../../lib/ts/ConsistentHash.cc  -fPIC -DPIC -o 
.libs/ConsistentHash.o
libtool: compile:  ccache /usr/bin/g++-4.9 -DHAVE_CONFIG_H -I. 
-I../../../lib/ts -I../../lib -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.5 
-I
 -D_GNU_SOURCE -I/usr/include -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall 
-Wextra -Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT BaseLogFile.lo -MD -MP -MF 
.deps/BaseLogFile.Tpo -c ../../../lib/ts/BaseLogFile.cc  -fPIC -DPIC -o 
.libs/BaseLogFile.o
depbase=`echo ContFlags.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache 
/usr/bin/g++-4.9 -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib  -Dlinux 
-D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.5 
-I
 -D_GNU_SOURCE -I/usr/include -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall 
-Wextra -Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT ContFlags.lo -MD -MP -MF $depbase.Tpo -c -o 
ContFlags.lo ../../../lib/ts/ContFlags.cc &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache /usr/bin/g++-4.9 -DHAVE_CONFIG_H -I. 
-I../../../lib/ts -I../../lib -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.5 
-I
 -D_GNU_SOURCE -I/usr/include -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall 
-Wextra -Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT ContFlags.lo -MD -MP -MF .deps/ContFlags.Tpo 
-c ../../../lib/ts/ContFlags.cc  -fPIC -DPIC -o .libs/ContFlags.o
depbase=`echo Diags.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache 
/usr/bin/g++-4.9 -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib  -Dlinux 
-D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 

[GitHub] trafficserver issue #1209: Remove unused and never to be used classes from M...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1209
  
Linux build *failed*! See 
https://ci.trafficserver.apache.org/job/linux-github/1507/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1267: TS-5021: Separate parent_is_proxy from other pare...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1267
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/freebsd-github/1610/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1264: TS-3692: Remove code associated with proxy.config...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1264
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/freebsd-github/1611/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver pull request #1482: MemView: Change typed array access method ...

2017-02-24 Thread SolidWallOfCode
Github user SolidWallOfCode closed the pull request at:

https://github.com/apache/trafficserver/pull/1482


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1264: TS-3692: Remove code associated with proxy.config...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1264
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/linux-github/1506/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1267: TS-5021: Separate parent_is_proxy from other pare...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1267
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/linux-github/1505/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1209: Remove unused and never to be used classes from M...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1209
  
Intel CC build *failed*! See 
https://ci.trafficserver.apache.org/job/icc-github/45/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1264: TS-3692: Remove code associated with proxy.config...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1264
  
Intel CC build *failed*! See 
https://ci.trafficserver.apache.org/job/icc-github/44/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1267: TS-5021: Separate parent_is_proxy from other pare...

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1267
  
Intel CC build *failed*! See 
https://ci.trafficserver.apache.org/job/icc-github/43/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1264: TS-3692: Remove code associated with proxy.config...

2017-02-24 Thread zwoop
Github user zwoop commented on the issue:

https://github.com/apache/trafficserver/pull/1264
  
[approve ci]


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1267: TS-5021: Separate parent_is_proxy from other pare...

2017-02-24 Thread zwoop
Github user zwoop commented on the issue:

https://github.com/apache/trafficserver/pull/1267
  
[approve ci]


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1209: Remove unused and never to be used classes from M...

2017-02-24 Thread zwoop
Github user zwoop commented on the issue:

https://github.com/apache/trafficserver/pull/1209
  
[approve ci]


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1158: TS-5015: Convert storage configuration file to Lu...

2017-02-24 Thread zwoop
Github user zwoop commented on the issue:

https://github.com/apache/trafficserver/pull/1158
  
I still think we need to have a discussion on what these new "Lua" style 
configs should be named, such that we don't have to come up with contrived 
renaming. I'm ok with a prefix of any of

* .cfg
* .conf
* .lua
* .luacfg
* .luaconf

etc. @shukitchan Can you take this to the mailing list for consensus? Also, 
if we do change this, we should change metrics.config and logging.config 
accordingly, however, for 7.x both names would need to be support for 
compatibility.


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1114: TS-4976: Remove useless casts from plugins.

2017-02-24 Thread zwoop
Github user zwoop commented on the issue:

https://github.com/apache/trafficserver/pull/1114
  
This needs rebasing and then I'll +1 it.


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1483: HTTP Request Debugging Feature

2017-02-24 Thread chenglongwei
Github user chenglongwei commented on the issue:

https://github.com/apache/trafficserver/issues/1483
  
Hi @zwoop @jpeach @SolidWallOfCode , could you give some feedbacks. This 
debug feature is very useful when dealing with plugin-related bugs and help us 
make sense of various changes happening throughout trafficserver.


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1487: esi/combo_handler plugin doesn't register

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1487
  
clang-analyzer build *successful*! See 
https://ci.trafficserver.apache.org/job/clang-analyzer-github/173/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1487: esi/combo_handler plugin doesn't register

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1487
  
clang-analyzer build *successful*! See 
https://ci.trafficserver.apache.org/job/clang-analyzer-github/174/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1487: esi/combo_handler plugin doesn't register

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1487
  
Intel CC build *successful*! See 
https://ci.trafficserver.apache.org/job/icc-github/41/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Build failed in Jenkins: ubuntu_16_10-master » gcc,ubuntu_16_10,release #143

2017-02-24 Thread jenkins
See 


--
[...truncated 227.57 KB...]
depbase=`echo ContFlags.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT ContFlags.lo -MD -MP -MF $depbase.Tpo -c -o 
ContFlags.lo ../../../lib/ts/ContFlags.cc &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT ContFlags.lo -MD -MP -MF .deps/ContFlags.Tpo 
-c ../../../lib/ts/ContFlags.cc  -fPIC -DPIC -o .libs/ContFlags.o
depbase=`echo Diags.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT Diags.lo -MD -MP -MF $depbase.Tpo -c -o 
Diags.lo ../../../lib/ts/Diags.cc &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT Diags.lo -MD -MP -MF .deps/Diags.Tpo -c 
../../../lib/ts/Diags.cc  -fPIC -DPIC -o .libs/Diags.o
depbase=`echo EventNotify.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT EventNotify.lo -MD -MP -MF $depbase.Tpo -c -o 
EventNotify.lo ../../../lib/ts/EventNotify.cc &&\
mv -f $depbase.Tpo $depbase.Plo
depbase=`echo fastlz.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CC   --mode=compile ccache cc -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=gnu99 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror -mcx16  -MT 
fastlz.lo -MD -MP -MF $depbase.Tpo -c -o fastlz.lo ../../../lib/ts/fastlz.c &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 

Build failed in Jenkins: ubuntu_14_04-master » gcc,ubuntu_14_04,release #2459

2017-02-24 Thread jenkins
See 


--
[...truncated 225.10 KB...]
mv -f $depbase.Tpo $depbase.Plo
depbase=`echo ConsistentHash.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ 
-DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib  -Dlinux 
-D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT ConsistentHash.lo -MD -MP -MF $depbase.Tpo -c 
-o ConsistentHash.lo ../../../lib/ts/ConsistentHash.cc &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT BaseLogFile.lo -MD -MP -MF 
.deps/BaseLogFile.Tpo -c ../../../lib/ts/BaseLogFile.cc  -fPIC -DPIC -o 
.libs/BaseLogFile.o
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT Bitops.lo -MD -MP -MF .deps/Bitops.Tpo -c 
../../../lib/ts/Bitops.cc  -fPIC -DPIC -o .libs/Bitops.o
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT ConsistentHash.lo -MD -MP -MF 
.deps/ConsistentHash.Tpo -c ../../../lib/ts/ConsistentHash.cc  -fPIC -DPIC -o 
.libs/ConsistentHash.o
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT Arena.lo -MD -MP -MF .deps/Arena.Tpo -c 
../../../lib/ts/Arena.cc  -fPIC -DPIC -o .libs/Arena.o
depbase=`echo ContFlags.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ 
-DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib  -Dlinux 
-D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT ContFlags.lo -MD -MP -MF $depbase.Tpo -c -o 
ContFlags.lo ../../../lib/ts/ContFlags.cc &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 

[GitHub] trafficserver issue #1487: esi/combo_handler plugin doesn't register

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1487
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/linux-github/1503/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1487: esi/combo_handler plugin doesn't register

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1487
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/linux-github/1504/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1487: esi/combo_handler plugin doesn't register

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1487
  
Intel CC build *successful*! See 
https://ci.trafficserver.apache.org/job/icc-github/42/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Build failed in Jenkins: ubuntu_12_04-master » gcc,ubuntu_12_04,release #2231

2017-02-24 Thread jenkins
See 


--
[...truncated 230.02 KB...]
libtool: compile:  ccache /usr/bin/g++-4.9 -DHAVE_CONFIG_H -I. 
-I../../../lib/ts -I../../lib -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.5 
-I
 -D_GNU_SOURCE -I/usr/include -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall 
-Wextra -Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT Arena.lo -MD -MP -MF .deps/Arena.Tpo -c 
../../../lib/ts/Arena.cc  -fPIC -DPIC -o .libs/Arena.o
libtool: compile:  ccache /usr/bin/g++-4.9 -DHAVE_CONFIG_H -I. 
-I../../../lib/ts -I../../lib -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.5 
-I
 -D_GNU_SOURCE -I/usr/include -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall 
-Wextra -Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT BaseLogFile.lo -MD -MP -MF 
.deps/BaseLogFile.Tpo -c ../../../lib/ts/BaseLogFile.cc  -fPIC -DPIC -o 
.libs/BaseLogFile.o
libtool: compile:  ccache /usr/bin/g++-4.9 -DHAVE_CONFIG_H -I. 
-I../../../lib/ts -I../../lib -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.5 
-I
 -D_GNU_SOURCE -I/usr/include -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall 
-Wextra -Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT Bitops.lo -MD -MP -MF .deps/Bitops.Tpo -c 
../../../lib/ts/Bitops.cc  -fPIC -DPIC -o .libs/Bitops.o
libtool: compile:  ccache /usr/bin/g++-4.9 -DHAVE_CONFIG_H -I. 
-I../../../lib/ts -I../../lib -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.5 
-I
 -D_GNU_SOURCE -I/usr/include -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall 
-Wextra -Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT ConsistentHash.lo -MD -MP -MF 
.deps/ConsistentHash.Tpo -c ../../../lib/ts/ConsistentHash.cc  -fPIC -DPIC -o 
.libs/ConsistentHash.o
depbase=`echo ContFlags.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache 
/usr/bin/g++-4.9 -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib  -Dlinux 
-D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.5 
-I
 -D_GNU_SOURCE -I/usr/include -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall 
-Wextra -Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT ContFlags.lo -MD -MP -MF $depbase.Tpo -c -o 
ContFlags.lo ../../../lib/ts/ContFlags.cc &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache /usr/bin/g++-4.9 -DHAVE_CONFIG_H -I. 
-I../../../lib/ts -I../../lib -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.5 
-I
 -D_GNU_SOURCE -I/usr/include -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall 
-Wextra -Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT ContFlags.lo -MD -MP -MF .deps/ContFlags.Tpo 
-c ../../../lib/ts/ContFlags.cc  -fPIC -DPIC -o .libs/ContFlags.o
depbase=`echo Diags.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache 
/usr/bin/g++-4.9 -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib  -Dlinux 
-D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 

Build failed in Jenkins: ubuntu_16_04-master » gcc,ubuntu_16_04,release #592

2017-02-24 Thread jenkins
See 


--
[...truncated 227.53 KB...]
depbase=`echo ContFlags.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT ContFlags.lo -MD -MP -MF $depbase.Tpo -c -o 
ContFlags.lo ../../../lib/ts/ContFlags.cc &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT ContFlags.lo -MD -MP -MF .deps/ContFlags.Tpo 
-c ../../../lib/ts/ContFlags.cc  -fPIC -DPIC -o .libs/ContFlags.o
depbase=`echo Diags.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT Diags.lo -MD -MP -MF $depbase.Tpo -c -o 
Diags.lo ../../../lib/ts/Diags.cc &&\
mv -f $depbase.Tpo $depbase.Plo
depbase=`echo EventNotify.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CXX   --mode=compile ccache c++ -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16  -MT EventNotify.lo -MD -MP -MF $depbase.Tpo -c -o 
EventNotify.lo ../../../lib/ts/EventNotify.cc &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=c++11 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror 
-Wno-invalid-offsetof -mcx16 -MT Diags.lo -MD -MP -MF .deps/Diags.Tpo -c 
../../../lib/ts/Diags.cc  -fPIC -DPIC -o .libs/Diags.o
depbase=`echo fastlz.lo | sed 's|[^/]*$|.deps/&|;s|\.lo$||'`;\
/bin/bash ../../libtool  --tag=CC   --mode=compile ccache cc -DHAVE_CONFIG_H 
-I. -I../../../lib/ts -I../../lib  -Dlinux -D_LARGEFILE64_SOURCE=1 
-D_COMPILE64BIT_SOURCE=1 -D_REENTRANT -D__STDC_LIMIT_MACROS=1 
-D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 
-I
 -D_GNU_SOURCE -DOPENSSL_NO_SSL_INTERN -std=gnu99 -g -pipe -Wall -Wextra 
-Wno-ignored-qualifiers -Wno-unused-parameter -O3 
-feliminate-unused-debug-symbols -fno-strict-aliasing -Werror -mcx16  -MT 
fastlz.lo -MD -MP -MF $depbase.Tpo -c -o fastlz.lo ../../../lib/ts/fastlz.c &&\
mv -f $depbase.Tpo $depbase.Plo
libtool: compile:  ccache c++ -DHAVE_CONFIG_H -I. -I../../../lib/ts -I../../lib 
-Dlinux -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 -D_REENTRANT 
-D__STDC_LIMIT_MACROS=1 -D__STDC_FORMAT_MACROS=1 -I/usr/include/tcl8.6 

[GitHub] trafficserver issue #1487: esi/combo_handler plugin doesn't register

2017-02-24 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1487
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/freebsd-github/1609/ for details.
 



---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1486: esi/combo_handler plugin doesn't call TSPluginReg...

2017-02-24 Thread danobi
Github user danobi closed the issue at:

https://github.com/apache/trafficserver/issues/1486


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver pull request #1487: esi/combo_handler plugin doesn't register

2017-02-24 Thread SolidWallOfCode
Github user SolidWallOfCode closed the pull request at:

https://github.com/apache/trafficserver/pull/1487


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1487: esi/combo_handler plugin doesn't register

2017-02-24 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the issue:

https://github.com/apache/trafficserver/pull/1487
  
Apparently no one uses ComboHandler, if this wasn't noticed.


---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver pull request #1487: esi/combo_handler plugin doesn't register

2017-02-24 Thread danobi
GitHub user danobi opened a pull request:

https://github.com/apache/trafficserver/pull/1487

esi/combo_handler plugin doesn't register

The plugin doesn't call TSPluginRegister. The causes ATS to crash upon
startup if combo_handler is enabled.

See issue #1486.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/danobi/trafficserver 
combo_missing_tspluginregister

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/trafficserver/pull/1487.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1487






---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1486: esi/combo_handler plugin doesn't call TSPluginReg...

2017-02-24 Thread danobi
GitHub user danobi opened an issue:

https://github.com/apache/trafficserver/issues/1486

esi/combo_handler plugin doesn't call TSPluginRegister

This causes ATS to crash if ATS is run with combo_handler.






---
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---