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

3.4 remove wps jasper reports #754

Merged
merged 3 commits into from
Sep 15, 2016

Conversation

stephanr
Copy link
Member

@stephanr stephanr commented Aug 23, 2016

On the FOSS4G Bonn CopdeSprint we (@tfr42, @dstenger, @lgoltz ) discussed to drop in 3.4 the wps extension for the jasperreports.
This was initiated to remove the (old) dependecies of the version of jaspersoft and its dependencies.

At last, as far as i know, the jaspersoft-wps-modules were not used in the webservices bundle.

This pull request moves the workspace and the modules to the uncoupled.

TODO:

  • Check if the handbook containes references to the jrxml module, which also have to be removed

@stephanr stephanr added in progress bug error issue and bug (fix) and removed in progress labels Aug 23, 2016
@stephanr stephanr added this to the 3.4 milestone Aug 23, 2016
@stephanr stephanr self-assigned this Aug 23, 2016
@stephanr stephanr changed the title [WIP] 3.4 remove wps jasper reports 3.4 remove wps jasper reports Aug 30, 2016
@stephanr
Copy link
Member Author

I just checked, the handbook does not contain a reference to the jrxml process provider.

@lgoltz lgoltz added the needs rebase PR is not up to date and needs rebase label Aug 30, 2016
removed:
  - commons-digester:commons-digester:2.1
  - net.sf.jasperreports:jasperreports:5.0.4
  - com.lowagie:itext:4.2.1
@stephanr
Copy link
Member Author

After rebase and close/open the pull request builds fine

@stephanr stephanr removed in progress needs rebase PR is not up to date and needs rebase labels Sep 15, 2016
@copierrj copierrj merged commit ba1c2bf into deegree:master Sep 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug error issue and bug (fix)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants