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

Add php 5.3 and 5.4 fixing the update script #1

Merged
merged 1 commit into from
Sep 2, 2014
Merged

Add php 5.3 and 5.4 fixing the update script #1

merged 1 commit into from
Sep 2, 2014

Conversation

yosifkit
Copy link
Member

No description provided.

@yosifkit
Copy link
Member Author

@tianon, @Moghedrin can you review?

&& make -j"$(nproc)" \
&& make install \
&& apt-get purge -y autoconf2.13 curl \
&& apt-get autoremove -y \
Copy link
Member

Choose a reason for hiding this comment

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

Can we use apt-get purge -y --auto-remove autoconf2.13 curl instead of this separate autoremove line? (I know there are other repos that ought to do this too, but this one can at least start off on the right foot.)

Copy link
Member

Choose a reason for hiding this comment

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

Oh, we should probably purge bison and libbison-dev too, right?

Copy link
Member Author

Choose a reason for hiding this comment

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

What is the difference between apt-get purge -y --auto-remove pkg and apt-get autoremove -y pkg?

Not yet sure what effect the bison removal would have. /me goes testing.

Copy link
Member

Choose a reason for hiding this comment

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

So, apt-get purge -y --auto-remove package is the same as apt-get purge -y package && apt-get autoremove -y except it does everything in one step, and even does "purge" on the autoremoved packages instead of just the main package.

@tianon
Copy link
Member

tianon commented Aug 29, 2014

Other than my minor comments, LGTM

cc @Moghedrin

@yosifkit
Copy link
Member Author

I will probably throw in PHP 5.6 as it was just released today. Should I put 5.6 as latest?

@tianon
Copy link
Member

tianon commented Aug 29, 2014

Legit! If it's considered a "stable" release upstream, absolutely. :)

@yosifkit
Copy link
Member Author

As stable as php. http://php.net/archive/2014.php#id2014-08-28-1

@tianon
Copy link
Member

tianon commented Aug 29, 2014

http://us3.php.net/downloads.php

Yep, that's "latest". :)

@yosifkit
Copy link
Member Author

Added dpkg -r and --auto-remove. Added php 5.6.

one more check: @tianon, @Moghedrin

@daghack
Copy link
Contributor

daghack commented Sep 2, 2014

LGTM, will merge. 👍

daghack added a commit that referenced this pull request Sep 2, 2014
Add php 5.3 and 5.4 fixing the update script
@daghack daghack merged commit d143b7e into docker-library:master Sep 2, 2014
@yosifkit yosifkit deleted the more-versions branch September 2, 2014 21:11
tianon added a commit that referenced this pull request Jun 14, 2016
Run update.sh to make the feature available.
@abalage abalage mentioned this pull request Jul 29, 2016
tianon pushed a commit that referenced this pull request Aug 9, 2016
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