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

Bugfix/restore build information via ModuleInfo #432

Merged
merged 2 commits into from
Nov 11, 2014

Conversation

stephanr
Copy link
Member

The current build information logic is deactivates, as the deegree-maven-plugin is only available via separate profile flag, so no ModuleInformation about the loaded Modules (jar) are currently available at startup / in console.

This pull restores the logic without using an external plugin (to reduce complexity and extra files). It uses the already available manifest.mf generation process of maven and extends it to include deegree-* key/value pairs to include the build information.

- changed ModuleInfo to read from mafifest.mf instead of buildinfo.properties
- removed deegree-maven-plugin:generate-buildinfo from build
@stephanr stephanr added this to the 3.4 milestone Oct 17, 2014
@stephanr stephanr added the bug error issue and bug (fix) label Oct 17, 2014
@stephanr stephanr self-assigned this Oct 17, 2014
@stephanr
Copy link
Member Author

The deegree 3.3 line is not affected (the generation of buildinfo.properties is still active there).

MrSnyder added a commit that referenced this pull request Nov 11, 2014
Bugfix/restore build information via ModuleInfo
@MrSnyder MrSnyder merged commit d8058b8 into deegree:master Nov 11, 2014
@MrSnyder MrSnyder deleted the bugfix/restore-build-information branch November 11, 2014 14:43
@tfr42
Copy link
Member

tfr42 commented Nov 13, 2014

Thank you for your contribution. Your pull requests has been accepted by the TMC. Please find more information in the meeting chat available at http://wiki.deegree.org/deegreeWiki/TmcMeeting/TmcMeeting20141111.
This pull request will be part of the 3.4-pre16 release (unstable).

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