maven-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mark (JIRA)" <>
Subject [jira] [Comment Edited] (SUREFIRE-951) Better handling of file.encoding system property
Date Thu, 16 Jun 2016 10:54:05 GMT


Mark edited comment on SUREFIRE-951 at 6/16/16 10:53 AM:

still not working reliably as of surefire 2.19.1. Netbeans is starting maven tests with "mvn
-Dfile.encoding=...", and that seems to trigger the warning.

was (Author: gwdfl59u):
still not working reliably as of surefire 2.19.1.

> Better handling of file.encoding system property
> ------------------------------------------------
>                 Key: SUREFIRE-951
>                 URL:
>             Project: Maven Surefire
>          Issue Type: Bug
>          Components: Maven Surefire Plugin, process forking
>    Affects Versions: 2.13
>         Environment: Any environment in which the file encoding is distinct from ${}.
>            Reporter: Stephan Schroevers
>            Assignee: Kristian Rosenvold
>             Fix For: 2.15
>         Attachments: file-encoding-example.tbz
> It appears that Surefire doesn't (correctly) set {{-Dfile.encoding=$\{\}}}
prior to running the tests. As a result the JVM will derive {{file.encoding}} from the environment's
file encoding. This affects the return value of {{java.nio.charset.Charset#defaultCharset()}},
which reads the {{file.encoding}} system property exactly once, and is invoked very early
on. Concretely this means that the unit tests are unnecessarily platform dependent.
> Thus I have two requests:
> # Make {{-Dfile.encoding=$\{\}}} the default. That is, the
following configuration setting should be implied:
> {noformat}
> <argLine>-Dfile.encoding=${}</argLine>
> {noformat}
> # Extend the method {{org.apache.maven.plugin.surefire.SurefireProperties#verifyLegalSystemProperties(org.apache.maven.plugin.logging.Log)}}
such that is warns about users attempting to set the {{file.encoding}} property through the
{{systemPropertyVariables}} configuration setting. Like with {{java.library.path}}, this does
_not_ work.
> I have [attached|^file-encoding-example.tbz] a sample project that demonstrates the issue
(simply run {{`mvn test`}}). Please have a look at the comments I added to the POM. I have
tested this sample project only on Linux, but a colleague has confirmed that an instance of
this issue can also be recreated on Windows.
> TIA for looking into this!

This message was sent by Atlassian JIRA

View raw message