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

Bump rakudo-star to latest version #3689

Merged
merged 1 commit into from
Nov 13, 2017
Merged

Conversation

hoelzro
Copy link
Contributor

@hoelzro hoelzro commented Nov 10, 2017

No description provided.

@tianon
Copy link
Member

tianon commented Nov 13, 2017

Diff:
warning: insecure protocol git:// detected: git://github.com/perl6/docker
diff --git a/_bashbrew-list b/_bashbrew-list
index e675f54..2f6fbad 100644
--- a/_bashbrew-list
+++ b/_bashbrew-list
@@ -1,2 +1,2 @@
-rakudo-star:2017.07
+rakudo-star:2017.10
 rakudo-star:latest
diff --git a/rakudo-star_latest/Dockerfile b/rakudo-star_latest/Dockerfile
index fba6524..9b29a5c 100644
--- a/rakudo-star_latest/Dockerfile
+++ b/rakudo-star_latest/Dockerfile
@@ -1,9 +1,10 @@
-FROM buildpack-deps:jessie-scm
+FROM buildpack-deps:stretch-scm
 MAINTAINER Rob Hoelz
 
 RUN groupadd -r perl6 && useradd -r -g perl6 perl6
 
-ENV rakudo_version=2017.07
+ARG rakudo_version=2017.10
+ENV rakudo_version=${rakudo_version}
 
 RUN buildDeps=' \
         gcc \

@tianon
Copy link
Member

tianon commented Nov 13, 2017

LGTM

Build test of #3689; 6a0f01e; amd64 (rakudo-star):

$ bashbrew build rakudo-star:2017.10
warning: insecure protocol git:// detected: git://github.com/perl6/docker
Building bashbrew/cache:0426c33a8af5206c7656ad2285e29f9490972161b60c8df52327830e6d6f74e4 (rakudo-star:2017.10)
Tagging rakudo-star:2017.10
Tagging rakudo-star:latest

$ test/run.sh rakudo-star:2017.10
testing rakudo-star:2017.10
	'utc' [1/4]...passed
	'cve-2014--shellshock' [2/4]...passed
	'no-hard-coded-passwords' [3/4]...passed
	'override-cmd' [4/4]...passed

@tianon tianon merged commit 1733668 into docker-library:master Nov 13, 2017
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.

3 participants