".resources" directory causes problems for github pages

[marchof]

github/maven-plugins#112

Steps to reproduce

JaCoCo version: 0.7.6.201602180812
Operating system: Windows/gitshell
Tool integration: Maven (github-site plugin)

Expected behaviour

Reports display correctly on github site.

Actual behaviour

The .resources directory is not deployed.

I have raised an issue with the github/maven-plugins project
github/maven-plugins#112
however the problem may well not be solvable by them.

The root cause is that some software, arguably correctly, overlooked a directory starting with a dot.

Your resources directory should not be a hidden directory.

Please consider renaming it to resources.

[marchof]

I think we discussed this before. We need to find a location for resources which will not collide with group or package names.

[timp]

A hyphen in the name would work. web-res ?

[agabrys]

You can also add jacoco keyword to name, e.g. jacoco-web-res.

[Godin]

@marchof out of curiosity - why we need to find a name, which doesn't collide with group or package names? Also what means "group" in this context? I'm asking, because indeed - hyphen allows to avoid collision with package name, but not sure about "group" name.

[marchof]

@Godin because it shares the same namespace than group or package names:

.resources
.sessions.html
index.html
jacoco-maven-plugin
jacoco.csv
jacoco.xml
org.jacoco.agent
org.jacoco.agent.rt
org.jacoco.ant
org.jacoco.core
org.jacoco.examples
org.jacoco.report

If we have name clash the report is probably broken. The same would happen is there is a class named index. With .sessions.html we have another dot resource which should be renamed.

I could live with jacoco-resources and jacoco-sessions.html. If somebody reports a name clash we sue him for trademark infringement ;-)

[marchof]

@Godin Here is a possible solution...

[Godin]

@marchof LGTM 👍 and this fixes problem with our own experimental deployment to GitHub pages - before this change all images at http://jacoco.github.io/jacoco/trunk/coverage/ were broken

[timp]

👍 thanks for speedy response.

[agabrys]

Is there any chance to release 0.7.6.1 version which contains only this patch?

[Godin]

@agabrys please wait for 0.7.7 or use snapshot version