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

Checkstyle binary is not part of docker image #50

Open
yukiisbored opened this issue Dec 4, 2016 · 4 comments
Open

Checkstyle binary is not part of docker image #50

yukiisbored opened this issue Dec 4, 2016 · 4 comments

Comments

@yukiisbored
Copy link
Member

yukiisbored commented Dec 4, 2016

It seems the checkstyle.jar file is not part of this docker image because every time I use it with this docker image, It downloads the jar file.

$ docker run -it --rm --volumes=$(pwd):/work --workdir=/work coala/base coala
Executing section java...
[INFO][02:57:59] Downloading '/root/.local/share/coala-bears/CheckstyleBear/checkstyle.jar' for bear CheckstyleBear from http://sourceforge.net/projects/checkstyle/files/checkstyle/6.15/checkstyle-6.15-all.jar.

To confirm the fact it's not part of this docker image, I ran bash inside the docker image and see if it's there.

# ls -lah /root/.local/share/coala-bears/CheckstyleBear/checkstyle.jar
ls: cannot access '/root/.local/share/coala-bears/CheckstyleBear/checkstyle.jar': No such file or directory
# ls -lah /usr/local/share/coala-bears/CheckstyleBear/checkstyle.jar
ls: cannot access '/usr/local/share/coala-bears/CheckstyleBear/checkstyle.jar': No such file or directory
# find / -name "*checkstyle*.jar"
#
@jayvdb
Copy link
Member

jayvdb commented Jan 3, 2017

Easy fix is to install checkstyle, and symlink the jar into /usr/local/share/coala-bears/CheckstyleBear/.

There is a related issue coala/coala-bears#626 for configurable locations, so that it is possible to avoid creating /usr/local/share/coala-bears/CheckstyleBear/ with symlink just so the bear finds it.

Worth noting that openSUSE checkstyle are all 5.8 or lower, which isnt good enough.

However the Docker could still use maven to fetch these packages, so coala/coala-bears#1250 is another path forward.

fneu added a commit that referenced this issue Mar 7, 2017
fneu added a commit that referenced this issue Mar 7, 2017
fneu added a commit that referenced this issue Mar 7, 2017
fneu added a commit that referenced this issue Mar 7, 2017
fneu added a commit that referenced this issue Mar 7, 2017
jayvdb added a commit to jayvdb/docker-coala-base that referenced this issue Mar 8, 2017
@jayvdb
Copy link
Member

jayvdb commented Mar 8, 2017

We still have a problem with 0.10, as it uses a different filename for the cached file.
ttps://travis-ci.org/coala/docker-coala-base/builds/208787231

@jayvdb
Copy link
Member

jayvdb commented Mar 25, 2017

@jayvdb jayvdb reopened this Mar 25, 2017
@jayvdb
Copy link
Member

jayvdb commented May 22, 2017

The Checkstyle and Scalastyle fetches in Dockerfile should be replaced with calling https://github.com/coala/coala-bears/blob/master/.ci/deps.coala-bears.sh

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

No branches or pull requests

3 participants