Transfer of code repositories

2018-01-08 Thread Otto van der Schaaf
Hi All, Last night all pagespeed/ repo's have been transferred to the ASF. To continue to be able to commit via github, your ASF ID needs to be linked to your Github ID. You can do this over at https://id.apache.org (to specify your git username) Status can be checked via

[GitHub] nberlee commented on issue #1181: Support Alpine

2018-01-08 Thread GitBox
nberlee commented on issue #1181: Support Alpine URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-355911513 @hgl I fixed compiling in Alpine 3.7... look at https://github.com/nberlee/ngx-pagespeed-alpine/tree/3.7

[GitHub] oschaaf commented on a change in pull request #1703: Removes deprecated mod_access_compat directives

2018-01-08 Thread GitBox
oschaaf commented on a change in pull request #1703: Removes deprecated mod_access_compat directives URL: https://github.com/apache/incubator-pagespeed-mod/pull/1703#discussion_r160168266 ## File path: install/common/pagespeed.conf.template ## @@ -321,15 +321,19 @@

[GitHub] robov commented on issue #1181: Support Alpine

2018-01-08 Thread GitBox
robov commented on issue #1181: Support Alpine URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356047432 I am not sure yet, but it seems to be related to the nginx version. 1.12.2 seems to be working... 1.13.3 seems to be not. Can anyone confirm ? I am

[GitHub] jmarantz commented on a change in pull request #1695: OutputSanitizingAsyncFetch: runs right before PSOL responds

2018-01-08 Thread GitBox
jmarantz commented on a change in pull request #1695: OutputSanitizingAsyncFetch: runs right before PSOL responds URL: https://github.com/apache/incubator-pagespeed-mod/pull/1695#discussion_r160147479 ## File path: pagespeed/apache/header_util.cc ## @@ -115,6 +115,9 @@

pagespeed podling missing committers

2018-01-08 Thread Otto van der Schaaf
Hi, Greg pointed out that I do not have write access to the incubator-pagespeed-* repo's because the list over at https://whimsy.apache.org/roster/ppmc/pagespeed#committers does not include me. Actually, it looks like currently Kees is the only one listed (kspoelstra@). Could one of the mentors

[GitHub] jadatkins commented on issue #1521: install from script fails due to new filename

2018-01-08 Thread GitBox
jadatkins commented on issue #1521: install from script fails due to new filename URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1521#issuecomment-355955246 Thanks @oschaaf, I can see how that works and it seems to work for me too. However, my work depends on automated

[GitHub] jmarantz commented on a change in pull request #1695: OutputSanitizingAsyncFetch: runs right before PSOL responds

2018-01-08 Thread GitBox
jmarantz commented on a change in pull request #1695: OutputSanitizingAsyncFetch: runs right before PSOL responds URL: https://github.com/apache/incubator-pagespeed-mod/pull/1695#discussion_r160147018 ## File path: net/instaweb/http/public/async_fetch.h ## @@ -316,6

[GitHub] jmarantz commented on a change in pull request #1695: OutputSanitizingAsyncFetch: runs right before PSOL responds

2018-01-08 Thread GitBox
jmarantz commented on a change in pull request #1695: OutputSanitizingAsyncFetch: runs right before PSOL responds URL: https://github.com/apache/incubator-pagespeed-mod/pull/1695#discussion_r160146665 ## File path: net/instaweb/http/async_fetch.cc ## @@ -276,6 +276,33 @@

[GitHub] jadatkins commented on issue #1521: install from script fails due to new filename

2018-01-08 Thread GitBox
jadatkins commented on issue #1521: install from script fails due to new filename URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1521#issuecomment-355955902 thanks @oschaaf This is an automated message from

[GitHub] arosenhagen opened a new issue #1521: install from script fails due to new filename

2018-01-08 Thread GitBox
arosenhagen opened a new issue #1521: install from script fails due to new filename URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1521 The script at https://ngxpagespeed.com/install fails to install ngx_pagespeed due to the new name ```incubator-pagespeed-ngx-$VERSION```.

[GitHub] jmarantz commented on a change in pull request #1703: Removes deprecated mod_access_compat directives

2018-01-08 Thread GitBox
jmarantz commented on a change in pull request #1703: Removes deprecated mod_access_compat directives URL: https://github.com/apache/incubator-pagespeed-mod/pull/1703#discussion_r160155613 ## File path: install/common/pagespeed.conf.template ## @@ -321,15 +321,19 @@

[GitHub] REPTILEHAUS opened a new issue #1522: Cannot install - issues with ngx_pagespeed.o

2018-01-08 Thread GitBox
REPTILEHAUS opened a new issue #1522: Cannot install - issues with ngx_pagespeed.o URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1522 I have a docker environment that i have deployed a couple of times and it has always worked except now when provisioning a new server the

[GitHub] robov commented on issue #1181: Support Alpine

2018-01-08 Thread GitBox
robov commented on issue #1181: Support Alpine URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356011231 @ashishk-1 : When I try to build your dockerfile I get : adding module in /tmp/ngx_pagespeed-1.12.34.3-stable ./configure: error: no

[GitHub] oschaaf closed pull request #1706: Only perform semanage sequence once on CentOS

2018-01-08 Thread GitBox
oschaaf closed pull request #1706: Only perform semanage sequence once on CentOS URL: https://github.com/apache/incubator-pagespeed-mod/pull/1706 This is an automated message from the Apache Git Service. To respond to the

[GitHub] robov commented on issue #1181: Support Alpine

2018-01-08 Thread GitBox
robov commented on issue #1181: Support Alpine URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356010210 @nberlee : When I try to build your dockerfile I get : ``` adding module in /tmp/ngx_pagespeed-1.12.34.3-stable ./configure: error: no

[GitHub] oschaaf commented on issue #1522: Cannot install - issues with ngx_pagespeed.o

2018-01-08 Thread GitBox
oschaaf commented on issue #1522: Cannot install - issues with ngx_pagespeed.o URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1522#issuecomment-356012833 Could the build process be running out of memory? This

[GitHub] jmarantz commented on a change in pull request #1523: Unbreak install after repo transfers

2018-01-08 Thread GitBox
jmarantz commented on a change in pull request #1523: Unbreak install after repo transfers URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1523#discussion_r160193605 ## File path: scripts/build_ngx_pagespeed.sh ## @@ -612,7 +612,8 @@ Not deleting $directory;

[GitHub] oschaaf closed issue #1522: Cannot install - issues with ngx_pagespeed.o

2018-01-08 Thread GitBox
oschaaf closed issue #1522: Cannot install - issues with ngx_pagespeed.o URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1522 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] oschaaf commented on issue #1522: Cannot install - issues with ngx_pagespeed.o

2018-01-08 Thread GitBox
oschaaf commented on issue #1522: Cannot install - issues with ngx_pagespeed.o URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1522#issuecomment-356022083 No problem, happy to hear its working now! This is an

[GitHub] hgl commented on issue #1181: Support Alpine

2018-01-08 Thread GitBox
hgl commented on issue #1181: Support Alpine URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356024910 @nberlee Thanks for sharing. But I got the same error too. ``` configuring additional modules adding module in

[GitHub] oschaaf closed issue #1521: install from script fails due to new filename

2018-01-08 Thread GitBox
oschaaf closed issue #1521: install from script fails due to new filename URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1521 This is an automated message from the Apache Git Service. To respond to the

[GitHub] oschaaf closed pull request #1523: Unbreak install after repo transfers

2018-01-08 Thread GitBox
oschaaf closed pull request #1523: Unbreak install after repo transfers URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1523 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] oschaaf commented on issue #1521: install from script fails due to new filename

2018-01-08 Thread GitBox
oschaaf commented on issue #1521: install from script fails due to new filename URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1521#issuecomment-356026172 Should be fixed, please retry! This is an automated