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

Update Mageia 5 (especially for CVE-2015-7547) #1468

Merged
merged 1 commit into from
Feb 22, 2016

Conversation

tianon
Copy link
Member

@tianon tianon commented Feb 22, 2016

See also #1448.

This also removes Mageia 4 (which was EOL on September 19th, 2015; https://www.mageia.org/en/support/#lifecycle).

cc @juanluisbaptiste -- it looks good; I just rebased your "dist" branch on master and squashed the two tarball-touching commits into a single one 👍 (juanluisbaptiste/docker-brew-mageia@4da7cfe)

This also removes Mageia 4 (which was EOL on September 19th, 2015; https://www.mageia.org/en/support/#lifecycle).
@tianon
Copy link
Member Author

tianon commented Feb 22, 2016

Build test of #1468; f6fdb7a (mageia):

$ bashbrew build "mageia"
Cloning mageia (git://github.com/juanluisbaptiste/docker-brew-mageia) ...
Processing mageia:latest ...
Processing mageia:5 ...
$ bashbrew list --uniq "$url" | xargs test/run.sh
testing mageia:latest
    'utc' [1/4]...passed
    'cve-2014--shellshock' [2/4]...passed
    'no-hard-coded-passwords' [3/4]...passed
    'override-cmd' [4/4]...passed

@yosifkit
Copy link
Member

LGTM

yosifkit added a commit that referenced this pull request Feb 22, 2016
Update Mageia 5 (especially for CVE-2015-7547)
@yosifkit yosifkit merged commit a190384 into docker-library:master Feb 22, 2016
@yosifkit yosifkit deleted the mageia branch February 22, 2016 18:21
@tianon
Copy link
Member Author

tianon commented Feb 22, 2016

@juanluisbaptiste if we've prematurely removed Mageia 4, let's get a new tarball up on the dist branch and we can update this again to bring it back 👍

(but, I figured since it's EOL, we might as well remove it since the tag will stick around for users who still need it)

@juanluisbaptiste
Copy link
Contributor

I think its ok, as you say is EOL since September, Iĺl check with other devs and see what they think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants