Re: Progressive download

2015-10-06 Thread Brian Geffon
It just sounds like he was using tsxs to build the plugin and linked with only 1 of 6 or so of the compilation units required. Compiling with -O0 shouldn't make a difference. Brian On Wed, Oct 7, 2015 at 2:54 PM, Nick Kew wrote: > On Wed, 2015-10-07 at 13:24 +0800, Brian Geffon wrote: > > Sound

Re: Progressive download

2015-10-06 Thread Nick Kew
On Wed, 2015-10-07 at 13:24 +0800, Brian Geffon wrote: > Sounds like it wasn't properly linked, it's missing the symbol Some build inconsistency between the plugin and the core. > BgFetchConfig::readConfig(const char *config_file) Could the compiler have optimised out that symbol? Does compiling

Re: Progressive download

2015-10-06 Thread Brian Geffon
Sounds like it wasn't properly linked, it's missing the symbol BgFetchConfig::readConfig(const char *config_file) Which is configs.cc. I checked Makefile.am for background_fetch and it appears that it should be compiling and linking properly. Did you build this plugin in a strange way, for exampl

[GitHub] trafficserver pull request: TS-3956: Header_rewrite applies strang...

2015-10-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/300 --- 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

[GitHub] trafficserver pull request: TS-3956: Header_rewrite applies strang...

2015-10-06 Thread bgaff
Github user bgaff commented on the pull request: https://github.com/apache/trafficserver/pull/300#issuecomment-145764790 I managed to screw up the diff in the second commit ;/ I'm gonna land the code change and tests (obviously not screwed up like that) ;) --- If your project is set