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 th
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 +316,
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_r160146867
##
File path: net/instaweb/http/public/async_fetch.h
##
@@ -316,6 +316,
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 @@
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 @@ vo
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 @@
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 @@
#
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 mes
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 /tmp/ngx_
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 /tmp/ngx_page
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 sa
oschaaf opened a new pull request #1523: Unbreak install after repo transfers
URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1523
The directory name in the zip we download from github's release
archive will be different now that the project was renamed.
It may be renamed a
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 i
REPTILEHAUS commented on issue #1522: Cannot install - issues with
ngx_pagespeed.o
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1522#issuecomment-356014257
Im not sure, Ill scale up the vm now and revert back with the result.
--
REPTILEHAUS commented on issue #1522: Cannot install - issues with
ngx_pagespeed.o
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1522#issuecomment-356017773
Scaled up the instance and its working as it should, really should have
checked that first. thanks man.
---
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; n
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,
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 au
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 /tmp/ngx_pagespeed-1.12.34.3-sta
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, ple
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 message,
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 mes
jadatkins commented on issue #1521: install from script fails due to new
filename
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1521#issuecomment-356028628
@oschaaf @jmarantz that seems to work, thank you!
Th
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 still
oschaaf 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_r160230016
##
File path: pagespeed/apache/header_util.cc
##
@@ -115,6 +115,9 @@ voi
hgl commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356237144
After some digging, I found the root cause to be that this project just got
moved to `apache/incubator-pagespeed-ngx`, so after downloading the archive
fi
hgl commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356237144
After some digging, I found the root cause is that this project just got
moved to `apache/incubator-pagespeed-ngx`, so after downloading the archive
file,
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356242099
@HGL AWESOME.. yes I was very frustrated that even my backup files did
not work anymore...
So I guess you are right
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356242638
I am a pretty newbie... can you share your changed code ?
This is an automated message f
hgl commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356237144
After some digging, I found the root cause to be that this project just got
moved to `apache/incubator-pagespeed-ngx`, so after downloading the archive
fi
hgl commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356253735
@robov in @nberlee's Dockerfile
add
```
mv incubator-pagespeed-ngx-1.12.34.3-stable
/tmp/ngx_pagespeed-1.12.34.3-stable
```
below
hgl commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356254153
@nberlee a quick question, why is [this
line](https://github.com/nberlee/ngx-pagespeed-alpine/blob/3.7/Dockerfile#L47)
necessary? I try to compile without
hgl commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356254153
@nberlee a quick question, why is [this
line](https://github.com/nberlee/ngx-pagespeed-alpine/blob/3.7/Dockerfile#L47)
necessary? I tried to compile witho
hgl commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356253735
@robov in @nberlee's Dockerfile
add
```
mv incubator-pagespeed-ngx-1.12.34.3-stable
/tmp/ngx_pagespeed-1.12.34.3-stable && \
```
be
hgl commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356253735
@robov in @nberlee's Dockerfile
add
```
mv /tmp/incubator-pagespeed-ngx-1.12.34.3-stable
/tmp/ngx_pagespeed-1.12.34.3-stable && \
```
oschaaf commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356254572
(The repo may be renamed again at some stage later on, and this change
hopefully is resilient to that)
---
oschaaf commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356254446
With regard to the renaming of the repo,
https://github.com/apache/incubator-pagespeed-ngx/commit/e733a838c35a0f2108768c2d1ead6fb7ee806cf8
was merged
hgl commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356255779
@oschaaf Thanks for the head up. But it seems `Dockerfile`s from most
unofficial alpine images don't use this script. So they will continue to fail
to bui
hgl commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356255779
@oschaaf Thanks for the head up. But it seems `Dockerfile`s from most
unofficial alpine images don't use this script. So they'll continue to fail to
build
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356276303
@oschaaf @hgl : with the merge to unbreak the install script... do I still
have to rename the dir
hgl commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356277277
Yes, that script is never used in the Dockerfile
This is an automated message from the Apa
hgl commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356277892
But since the name is tentative, that mv command could easily fail when that
happens. A better approach:
```
mkdir -p /usr/local/src/mod-pagespee
hgl commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356277892
But since the name is tentative, that mv command could easily fail when that
happens. A better approach:
```
mkdir -p /tmp/modpagespeed-${PAGESPE
oschaaf commented on issue #1695: OutputSanitizingAsyncFetch: runs right before
PSOL responds
URL:
https://github.com/apache/incubator-pagespeed-mod/pull/1695#issuecomment-356278838
@jmarantz processed your comments in
https://github.com/apache/incubator-pagespeed-mod/pull/1695/commits/ee
hgl commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356277892
But since the name is tentative, that mv command could easily fail when that
happens. A better approach:
```
mkdir -p /tmp/ngx_pagespeed-${PAGESP
nberlee commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356353214
@hgl The sed line was a workaround for a Alpine 3.7 busybox wait bug. (which
never stopped waiting) This seems to be fixed since, making the line now
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356395182
@nberlee
Is there intentionally the hardcoded version in
```
https://github.com/We-Amp/ngx-pagespeed-alpine/blob/master/mod-pagespeed-beta-1.
nberlee commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356418528
@robov Currently yes, once a new mod-pagespeed stable (1.12.34.3) is
released, with @ashishk-1 commit:
https://github.com/apache/incubator-pagespeed-m
nberlee commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356418528
@robov Currently yes, once a new mod-pagespeed stable (1.12.34.3) is
released, with @ashishk-1 commit:
https://github.com/apache/incubator-pagespeed-m
nberlee commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356418528
@robov Currently yes, once a new mod-pagespeed stable (1.12.34.3) is
released, with @ashishk-1 commit:
https://github.com/apache/incubator-pagespeed-m
oschaaf commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356420687
@nberlee @robov maybe we should inventorise if it is viable to pull
maintenance of the alpine image into the project to centralize efforts. Would
you
oppianmatt opened a new issue #1524: build instructions no longer working for
1.13.35.1-beta
URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1524
Linked from the README
https://www.modpagespeed.com/doc/build_ngx_pagespeed_from_source
```
#[check the release notes fo
oppianmatt commented on issue #1524: build instructions no longer working for
all versions
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1524#issuecomment-356427457
tried using the existing version in the readme but it still breaks on both
places
```
inflating:
oppianmatt commented on issue #1524: build instructions no longer working for
all versions
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1524#issuecomment-356427654
after a bit of googling and testing found that psol moved to a new place and
so that needs to be fixed.
oschaaf commented on issue #1524: build instructions no longer working for all
versions
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1524#issuecomment-356429014
Thanks for the report!
Would you be willing to make a pull request for it? (The change should go
into
http
oppianmatt commented on issue #1524: build instructions no longer working for
all versions
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1524#issuecomment-356429334
ok, just testing it a bit more, it works for that part and then later fails
in configure
when I got i
oschaaf commented on issue #1524: build instructions no longer working for all
versions
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1524#issuecomment-356429446
@oppianmatt great, thanks!
This is an automate
oppianmatt commented on issue #1524: build instructions no longer working for
all versions
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1524#issuecomment-356435232
do you know where the binaries are being hosted now they aren't on google?
the links at https://github.com/a
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356498629
I would definately be interested, but I am not bringing much to this table.
My experience in linux is very low and I consider myself a newbie in this spa
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356517069
@nberlee : I am trying to understand your code, especially this part
confuses me :
LD_LIBRARY_PATH=/tmp/modpagespeed-${PAGESPEED_VERSION}/usr/lib ./
oschaaf commented on issue #1524: build instructions no longer working for all
versions
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1524#issuecomment-356522611
Above I see the script attempts to download
https://dl.google.com/dl/page-speed/psol/1.13.35.1-beta.tar.gz
T
oschaaf commented on issue #1524: build instructions no longer working for all
versions
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1524#issuecomment-356526816
I suspect the line with `unzip -d` isn't working as expected. The automated
install had a similar problem, and
oschaaf opened a new pull request #1707: Update
build_ngx_pagespeed_from_source.html
URL: https://github.com/apache/incubator-pagespeed-mod/pull/1707
Update to make it work after the repo transfer
This is an automated message
oschaaf commented on issue #1524: build instructions no longer working for all
versions
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1524#issuecomment-356527294
Created https://github.com/apache/incubator-pagespeed-mod/pull/1707
@oppianmatt could you review that?
oschaaf opened a new pull request #1708: Update README.md
URL: https://github.com/apache/incubator-pagespeed-mod/pull/1708
Fix travis links
This is an automated message from the Apache Git Service.
To respond to the message, p
oschaaf opened a new pull request #1525: Update README.md
URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1525
Fix travis links
This is an automated message from the Apache Git Service.
To respond to the message, p
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356535658
Holy Smoly... my bad... I was looking at the wrong branch... I was looking
at 3.4 instead of 3.7 for this last post
oppianmatt commented on issue #1707: Update build_ngx_pagespeed_from_source.html
URL:
https://github.com/apache/incubator-pagespeed-mod/pull/1707#issuecomment-356543054
need to change later on also the nginx add-module line:
locally I went with:
```
cd incubator-pagespeed-n
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356561161
Is there someone who can help me out ?
I want to build (tedious long) pagespeed in a separate container and include
only the necessary files in the ma
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356562398
I guessed the libpng was missing (only present in the temp build container,
so I added that to the main one. But hten the compiling broke on "missing
pa
nberlee commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356570673
@robov I made a multi-stage Dockerfile see
https://github.com/nberlee/ngx-pagespeed-alpine/tree/3.7-dev but haven't tested
it yet! It should build fin
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356575446
@nberlee : I am testing now..
This is an automated message from the Apache Git Service.
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356576872
Step 8/26 : COPY mod-pagespeed-beta-1.12.34.3.tar.bz2 patches/*.patch /tmp/
COPY failed: stat
/var/lib/docker/tmp/docker-builder857321033/mod-pagespee
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356577063
oops... my bad... forgot to pull your entire sources
This is an automated message from t
oschaaf commented on issue #1707: Update build_ngx_pagespeed_from_source.html
URL:
https://github.com/apache/incubator-pagespeed-mod/pull/1707#issuecomment-356598953
@oppianmatt updated to fix the ./configure line, could you take a another
look?
---
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356605030
@nberlee it compiles nice (I have not tested the actual functionality, just
compiling)
I have a personal preference to have separate files instead of
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356605923
When I do something like this on a container : docker cp
extractfromcontainer:/usr/lib/libpng* compiled/usr/lib
then it cannot find anything... when I
nberlee commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356654833
@robov https://github.com/moby/moby/issues/7710
This is an automated message from the
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356665588
aha, and when copying the entire folder it complains ( i think it was
about the symlinks)
--
oppianmatt commented on issue #1707: Update build_ngx_pagespeed_from_source.html
URL:
https://github.com/apache/incubator-pagespeed-mod/pull/1707#issuecomment-356676091
builds and configures
This is an automated message from
oschaaf closed pull request #1707: Update build_ngx_pagespeed_from_source.html
URL: https://github.com/apache/incubator-pagespeed-mod/pull/1707
This is an automated message from the Apache Git Service.
To respond to the messa
oschaaf closed issue #1524: build instructions no longer working for all
versions
URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1524
This is an automated message from the Apache Git Service.
To respond to the
robov commented on issue #1181: Support Alpine
URL:
https://github.com/apache/incubator-pagespeed-ngx/issues/1181#issuecomment-356742597
After 5 days of a very steep learning curve I am finally getting
somewhere... @nberlee ,
FROM on you multistage image I create my own derivative whi
oschaaf opened a new pull request #1709: Update release notes for 1.13.35.2-beta
URL: https://github.com/apache/incubator-pagespeed-mod/pull/1709
This is an automated message from the Apache Git Service.
To respond to the mes
oschaaf closed pull request #1709: Update release notes for 1.13.35.2-beta
URL: https://github.com/apache/incubator-pagespeed-mod/pull/1709
This is an automated message from the Apache Git Service.
To respond to the message,
googlebot commented on issue #1710: Update config_filters.html
URL:
https://github.com/apache/incubator-pagespeed-mod/pull/1710#issuecomment-356849655
Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at you
vitaliytv opened a new pull request #1710: Update config_filters.html
URL: https://github.com/apache/incubator-pagespeed-mod/pull/1710
fix broken link to insert_image_dimensions
This is an automated message from the Apache Gi
vitaliytv commented on issue #1710: Update config_filters.html
URL:
https://github.com/apache/incubator-pagespeed-mod/pull/1710#issuecomment-356851049
I signed it!
This is an automated message from the Apache Git Service.
To
googlebot commented on issue #1710: Update config_filters.html
URL:
https://github.com/apache/incubator-pagespeed-mod/pull/1710#issuecomment-356851059
CLAs look good, thanks!
This is an automated message from the Apac
oschaaf commented on issue #968: Distribute PSOL and mod_pagespeed as a source
tarball.
URL:
https://github.com/apache/incubator-pagespeed-mod/issues/968#issuecomment-356868405
https://dl.google.com/dl/linux/mod-pagespeed/tar/beta/mod-pagespeed-beta-1.13.35.2-r0.tar.bz2
https://dl.goo
oschaaf closed pull request #1710: Update config_filters.html
URL: https://github.com/apache/incubator-pagespeed-mod/pull/1710
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign p
oschaaf opened a new pull request #1711: New index for modpagespeed.com
URL: https://github.com/apache/incubator-pagespeed-mod/pull/1711
Based on Jeff's earlier work on this, with a couple of changes:
- Link the examples that currently serve as the index as
http://staging.modpagesp
oschaaf commented on issue #1711: New index for modpagespeed.com
URL:
https://github.com/apache/incubator-pagespeed-mod/pull/1711#issuecomment-357050102
@jmarantz @morlovich WDYT?
This is an automated message from the Apache
oschaaf commented on issue #1711: New index for modpagespeed.com
URL:
https://github.com/apache/incubator-pagespeed-mod/pull/1711#issuecomment-357050625
@jeffkaufman ping
(just a FYI, this PR is based on your earlier work on a new index for
modpagespeed.com)
--
oschaaf closed pull request #1708: Update README.md
URL: https://github.com/apache/incubator-pagespeed-mod/pull/1708
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub an
oschaaf closed pull request #1525: Update README.md
URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1525
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub an
jeffkaufman commented on issue #1711: New index for modpagespeed.com
URL:
https://github.com/apache/incubator-pagespeed-mod/pull/1711#issuecomment-357059322
Thanks for picking this up! I'm sorry I didn't finish it!
This is a
jmarantz commented on issue #1711: New index for modpagespeed.com
URL:
https://github.com/apache/incubator-pagespeed-mod/pull/1711#issuecomment-357060042
+1 I think it's really great that you are picking this up, and
organizationally it's a huge win.
I would like to see someone (no
oschaaf commented on issue #1711: New index for modpagespeed.com
URL:
https://github.com/apache/incubator-pagespeed-mod/pull/1711#issuecomment-357065891
@jmarantz I'll ask around tomorrow to figure out what effort it would take
to reuse some design elements.
I do suspect it would invo
oschaaf closed pull request #1711: New index for modpagespeed.com
URL: https://github.com/apache/incubator-pagespeed-mod/pull/1711
This is an automated message from the Apache Git Service.
To respond to the message, please l
1101 - 1200 of 4920 matches
Mail list logo