Maven 2.x Resources Plugin

use ${project.build.sourceEncoding} as default value for "encoding" parameter

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.2
  • Fix Version/s: 2.3
  • Component/s: None
  • Labels:
    None
  • Patch Submitted:
    Yes
  • Number of attachments :
    1

Description

The platform's default encoding may easily differ among machines/developers so relying on it breaks with the aim of reproducible builds. The encoding used should always be fixed for a particular plugin run, either explicitly by the user or implicitly by a known default value.

See also MCOMPILER-63.

Issue Links

is related to

Improvement - An improvement or enhancement to an existing feature or task. MNG-2216 Add default encodings section to POM

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Improvement - An improvement or enhancement to an existing feature or task. MRESOURCES-98 Set default encoding to UTF-8

  • Critical - Crashes, loss of data, severe memory leak.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
relates to

Improvement - An improvement or enhancement to an existing feature or task. MRESOURCES-62 instead of "default encoding" message, should be "platform encoding" and explicit about the actual encoding used

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Herve Boutemy added a comment -

see http://docs.codehaus.org/display/MAVENUSER/POM+Element+for+Source+File+Encoding

Show
Herve Boutemy added a comment - see http://docs.codehaus.org/display/MAVENUSER/POM+Element+for+Source+File+Encoding
Hide
Permalink
Herve Boutemy added a comment -

done in r649654

Show
Herve Boutemy added a comment - done in r649654
Hide
Permalink
Benjamin Bentmann added a comment -

Needs to be reverted to use platform encoding by default.

Show
Benjamin Bentmann added a comment - Needs to be reverted to use platform encoding by default.
Hide
Permalink
Benjamin Bentmann added a comment -

Done in r653078.

Show
Benjamin Bentmann added a comment - Done in r653078 .

People

  • Assignee:
    Herve Boutemy
    Reporter:
    Benjamin Bentmann
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: