Skip to content
This repository has been archived by the owner on Nov 29, 2018. It is now read-only.

added flexible path prefix #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

added flexible path prefix #14

wants to merge 2 commits into from

Conversation

ManfredTremmel
Copy link

At the moment gwt-pushstate reduces the path always to / as base e.g. if you use a tomcat Project with path "test" uses the url http://www.example.org:8080/test/Test.html it will be reduced to http://www.example.org:8080/ what doesn't work anymore. I've added a flexible pathprovider which simply returns the prefix, in the project you can add a Implementation of the PathProviderInterface and replace the interface in the project gwt.xml with the implementation. I've added this at http://www.knightsoft-net.de/base16k/ (base16k it the prefix).

@buildhive
Copy link

@ManfredTremmel
Copy link
Author

Hm, interesting question, my local build works without any problem. I've just downloaded and compiled it:

...

Tests run: 15, Failures: 0, Errors: 0, Skipped: 0

[INFO]
[INFO] --- maven-jar-plugin:2.3.2:jar (default-jar) @ gwt-pushstate ---
[INFO] Building jar: /tmp/gwt-pushstate-master/target/gwt-pushstate-1.1.0-SNAPSHOT.jar
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 3:25.950s
[INFO] Finished at: Thu Oct 02 19:42:02 CEST 2014
[INFO] Final Memory: 29M/146M
[INFO] ------------------------------------------------------------------------

@jbarop
Copy link
Owner

jbarop commented Oct 7, 2014

Thanks! Looking into it right now.

No worry about the failing tests. I thinking about replacing them with real browser tests

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

Successfully merging this pull request may close these issues.

3 participants