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 Jetty #469

Merged
merged 1 commit into from
Feb 9, 2015
Merged

Add Jetty #469

merged 1 commit into from
Feb 9, 2015

Conversation

md5
Copy link
Contributor

@md5 md5 commented Feb 7, 2015

Please review. I'll open a docs PR shortly.

Closes #203

@yosifkit
Copy link
Member

yosifkit commented Feb 9, 2015

LGTM
Build test of #469; 7523911 (jetty):

$ url="https://raw.githubusercontent.com/docker-library/official-images/7523911829e8da387dc1af27da4a1147e1a76bda/library/jetty"
$ bashbrew build "$url"
Cloning jetty (git://github.com/md5/docker-jetty) ...
Processing jetty:7.6.16-jre7 ...
Processing jetty:7.6-jre7 ...
Processing jetty:7-jre7 ...
Processing jetty:7.6.16 ...
Processing jetty:7.6 ...
Processing jetty:7 ...
Processing jetty:7.6.16-jre8 ...
Processing jetty:7.6-jre8 ...
Processing jetty:7-jre8 ...
Processing jetty:8.1.16-jre7 ...
Processing jetty:8.1-jre7 ...
Processing jetty:8-jre7 ...
Processing jetty:8.1.16 ...
Processing jetty:8.1 ...
Processing jetty:8 ...
Processing jetty:8.1.16-jre8 ...
Processing jetty:8.1-jre8 ...
Processing jetty:8-jre8 ...
Processing jetty:9.2.7-jre7 ...
Processing jetty:9.2-jre7 ...
Processing jetty:9-jre7 ...
Processing jetty:jre7 ...
Processing jetty:9.2.7 ...
Processing jetty:9.2 ...
Processing jetty:9 ...
Processing jetty:latest ...
Processing jetty:9.2.7-jre8 ...
Processing jetty:9.2-jre8 ...
Processing jetty:9-jre8 ...
Processing jetty:jre8 ...
$ bashbrew list "$url" | xargs test/run.sh
testing jetty:7.6.16-jre7
    'utc' [1/1]...passed
testing jetty:7.6-jre7
    'utc' [1/1]...passed
testing jetty:7-jre7
    'utc' [1/1]...passed
testing jetty:7.6.16
    'utc' [1/1]...passed
testing jetty:7.6
    'utc' [1/1]...passed
testing jetty:7
    'utc' [1/1]...passed
testing jetty:7.6.16-jre8
    'utc' [1/1]...passed
testing jetty:7.6-jre8
    'utc' [1/1]...passed
testing jetty:7-jre8
    'utc' [1/1]...passed
testing jetty:8.1.16-jre7
    'utc' [1/1]...passed
testing jetty:8.1-jre7
    'utc' [1/1]...passed
testing jetty:8-jre7
    'utc' [1/1]...passed
testing jetty:8.1.16
    'utc' [1/1]...passed
testing jetty:8.1
    'utc' [1/1]...passed
testing jetty:8
    'utc' [1/1]...passed
testing jetty:8.1.16-jre8
    'utc' [1/1]...passed
testing jetty:8.1-jre8
    'utc' [1/1]...passed
testing jetty:8-jre8
    'utc' [1/1]...passed
testing jetty:9.2.7-jre7
    'utc' [1/1]...passed
testing jetty:9.2-jre7
    'utc' [1/1]...passed
testing jetty:9-jre7
    'utc' [1/1]...passed
testing jetty:jre7
    'utc' [1/1]...passed
testing jetty:9.2.7
    'utc' [1/1]...passed
testing jetty:9.2
    'utc' [1/1]...passed
testing jetty:9
    'utc' [1/1]...passed
testing jetty:latest
    'utc' [1/1]...passed
testing jetty:9.2.7-jre8
    'utc' [1/1]...passed
testing jetty:9.2-jre8
    'utc' [1/1]...passed
testing jetty:9-jre8
    'utc' [1/1]...passed
testing jetty:jre8
    'utc' [1/1]...passed

@tianon
Copy link
Member

tianon commented Feb 9, 2015

So many tags, like all our Java-based images. 😝

LGTM

tianon added a commit that referenced this pull request Feb 9, 2015
@tianon tianon merged commit 29f3837 into docker-library:master Feb 9, 2015
@tianon
Copy link
Member

tianon commented Feb 9, 2015

Welcome to the club, @md5 ❤️

@tianon
Copy link
Member

tianon commented Feb 9, 2015

You're willing to pass the baton if upstream becomes interested in maintaining this, right? 👍

@md5
Copy link
Contributor Author

md5 commented Feb 9, 2015

@tianon Of course.

@tianon
Copy link
Member

tianon commented Feb 9, 2015 via email

@md5 md5 deleted the add-jetty branch February 9, 2015 18:23
@md5
Copy link
Contributor Author

md5 commented Feb 9, 2015

@tianon Thanks for checking.

FWIW, I'll mention that I've signed up for jetty-announce and signed a CLA with the Eclipse Foundation to smooth the path for any patches I may need to make.

I'll also make an effort to be on #docker-library on Freenode. My registered nick is "mdillon" on Freenode if you want to +v me on the IRC channel.

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.

Provide jetty image
3 participants