Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dockerfile: updated to latest NewRelic #130

Merged
merged 1 commit into from
Apr 4, 2017
Merged

Conversation

bryanlatten
Copy link
Contributor

@bryanlatten bryanlatten commented Apr 3, 2017

@@ -93,9 +94,11 @@ RUN curl -L https://download.newrelic.com/php_agent/archive/${NEWRELIC_VERSION}/
cd newrelic-php5-${NEWRELIC_VERSION}-linux-musl && \
echo "\n" | ./newrelic-install install && \
chown root:root /root/newrelic-php5-${NEWRELIC_VERSION}-linux-musl/agent/x64/newrelic-20151012.so && \
cp /root/newrelic-php5-${NEWRELIC_VERSION}-linux-musl/agent/x64/newrelic-20151012.so /usr/lib/php7/modules/newrelic.so && \
mv /root/newrelic-php5-${NEWRELIC_VERSION}-linux-musl/agent/x64/newrelic-20151012.so /usr/lib/php7/modules/newrelic.so && \
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

previously a copy, leaving this and older php agents in place. move the important one, delete the others.

~10MB shaved

Also fixed some space-filling techniques
@mannytoledo
Copy link
Collaborator

👍

1 similar comment
@ko-be
Copy link
Collaborator

ko-be commented Apr 4, 2017

👍

@bryanlatten bryanlatten merged commit 982b532 into master Apr 4, 2017
@bryanlatten bryanlatten deleted the newrelic-space branch May 16, 2017 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants