Skip to content

[bugfix] Fix Windows build script and Log4j2 config path resolution#201

Closed
Laurettta wants to merge 437 commits intoevolvedbinary:mainfrom
Laurettta:7.x.x/hotfix/windows-log4j2-config-file-uri
Closed

[bugfix] Fix Windows build script and Log4j2 config path resolution#201
Laurettta wants to merge 437 commits intoevolvedbinary:mainfrom
Laurettta:7.x.x/hotfix/windows-log4j2-config-file-uri

Conversation

@Laurettta
Copy link
Copy Markdown

Thank you for contributing to Elemental!

To help the community judge your PR (Pull Request), please include the following:

Description:

Fix two Windows-specific issues:

  1. Fix Windows build script to use mvn directly instead of the relative mvnw.cmd path
  2. Normalise Log4j2 configuration file path to a file: URI so it resolves correctly on Windows

Reference:

N/A

Tests:

These are build and startup fixes. No automated test added.

adamretter and others added 30 commits September 2, 2025 17:26
…e function call location to the same as the function definition location
…roker-pool-services-manager-logging

[7.x.x] Correct the log destination in the Broker Pool Services Manager
…rofiling-fots-prefix

[7.x.x] Show prefix of XPath standard library functions in the profiling output
…estxq-function-call-location

[7.x.x] Correctly set the location of the RESTXQ Function Call
…ode-path-to-string

[7.x.x] Fix an issue in NodePath#toString()
…optimize-fn-reverse

[7.x.x] Optimize fn:reverse by removing unnecessary work
…improved-custom-linked-list

[7.x.x] Replace the custom Linked List with a better implementation
…be configured from test classes by annotation
The documentation for POSTing queries to the REST API says very little about how
external variables should be bound. The comments in this file are helpful, but
only if you know the namespace declarations!
…es are checked against the types of the Variable Declarations
adamretter and others added 26 commits April 28, 2026 15:22
…opendocument-openxml-media-types

[7.x.x] Fix the storage of OpenDocument files and Office Open XML files
…/bump-exist-compatibility

[7.x.x] Make eXist-db 6.4.0 the minimum compatible version for EXPath Packages
…fix-name-getContextItemDeclaration

[7.x.x] Correct typo of 'getContextItemDeclartion' to 'getContextItemDeclaration'
…n Client and Jetty Server startup so that Auto Deployment of EXPath packages can be disabled from the CLI
…/support-calabash3

[7.x.x] Support for executing XQuery from XML Calabash 3
… compatibility

  Log4j2 cannot resolve a bare Windows path set via the log4j2.configurationFile
  system property. Converting to a file: URI ensures correct resolution on all platforms.
@cla-bot
Copy link
Copy Markdown

cla-bot Bot commented Apr 29, 2026

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @ndw on file. In order for us to review and merge your code, please contact the project maintainers to get yourself added.

@codacy-production
Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 3 critical · 91 medium · 6 minor

Alerts:
⚠ 100 issues (≤ 0 issues of at least minor severity)

Results:
100 new issues

Category Results
BestPractice 22 medium
Security 3 critical
CodeStyle 6 minor
Complexity 69 medium

View in Codacy

🟢 Metrics 12208 complexity · 285 duplication

Metric Results
Complexity 12208
Duplication 285

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@adamretter
Copy link
Copy Markdown
Member

Superseded by #202

@adamretter adamretter closed this Apr 29, 2026
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.

3 participants