Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

update to tapestry 5.3.8 and java8 (5.4-beta-31 exists)#1548

Closed
based2 wants to merge 1 commit intoTechEmpower:masterfrom
based2:patch-2
Closed

update to tapestry 5.3.8 and java8 (5.4-beta-31 exists)#1548
based2 wants to merge 1 commit intoTechEmpower:masterfrom
based2:patch-2

Conversation

@based2
Copy link
Copy Markdown

@based2 based2 commented Apr 21, 2015

No description provided.

@hamiltont
Copy link
Copy Markdown
Contributor

blocking on changes requested in #1547

@zane-techempower
Copy link
Copy Markdown
Contributor

Looks like this is close, but not quite there. Seems like there is a disagreement between jdk version and target release, in this case it seems like the target is 1.8 i.e. Java8. However the process is using Java7 to try to compile the code.

See: http://stackoverflow.com/questions/28291691/fatal-error-compiling-invalid-target-release-1-8-help-1

@msmith-techempower
Copy link
Copy Markdown
Member

Will require install.sh to be updated to fw_depends java8 from java7.

@mcocciaTE
Copy link
Copy Markdown
Contributor

I can get everything to compile when I change source $IROOT/java7.installed it java8 in the setup.sh but it causes all of the tests to give a generic server error back. I'm looking more into why.

@mcocciaTE
Copy link
Copy Markdown
Contributor

I believe the version of maven we use has an issue with java8. If we updated maven to 3.1.1+ it should work.

@mcocciaTE
Copy link
Copy Markdown
Contributor

Updating maven did not seem to work. Poking around more.

@based2
Copy link
Copy Markdown
Author

based2 commented Jun 13, 2015

@mcocciaTE mcocciaTE mentioned this pull request Jun 16, 2015
@mcocciaTE
Copy link
Copy Markdown
Contributor

So maven ended up working but redis was not. #1684 is my fixes which get everything happy and so I will close this PR in favour of #1684

@mcocciaTE mcocciaTE closed this Jun 16, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants