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

Updated documentation & fixed some issues #93

Merged
merged 5 commits into from
Apr 24, 2014
Merged

Conversation

hazendaz
Copy link
Member

Updated majority of documentation to show maven setup beyond what was done earlier by Ryan.
Fixed some issues discovered when attempting to import the filter demo.
Added more manifest information which now includes it on all generated jars/wars along with jdk version it is targeting for source and compilation. This is helpful to the users especially for tomcat8 since it requires jdk7.

First cut to update documentation to reflect maven builds.

Note: All documentation needs reviewed as some notes about builds
appears to have changed some time ago and was unsure how to address.
This only addresses general maven build & already present was the
deployment to maven central.  It does not go into maven site generation
or other profiles yet.
Added additional information to poms so users can easily determine what
version of the jdk is expected to be used.  This is more important since
tomcat8 requires java7.
Updated all web.xml files to use universal character encoding rather
than legacy ISO-8859-1.
Corrected issues in waffle filter pom preventing it from working
properly.  Eclipse had issues importing in the form it was.  Also
removed information regarding the prior non maven layout.  Final war
build will no longer contain version number so it is easier to drop and
deploy for users testing.
hazendaz added a commit that referenced this pull request Apr 24, 2014
Updated documentation & fixed some issues
@hazendaz hazendaz merged commit 567c171 into Waffle:master Apr 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant