• Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Minor Minor
    • core
    • None
    • 2.358

      Property files for localization requires to be encoded in ISO-8859-1.
      It results difficulties in reviewing localizing pull requires such as https://github.com/jenkinsci/maven-plugin/pull/86.
      It would be nice if property files in source directories can be written in UTF-8 letters.

      There are two approaches:

      1. Encode property files when compiling.
        • maven-resources-plugin copies files in the resources directory into the target directory. It doesn't provide a feature to encode property files.
        • Though native2ascii-maven-plugin encodes property files, it is provided only as beta for long time.
          • This might mean encoding property files in compilation time isn't a standard approach.
        • maven-antrun-plugin can run native2ascii (But it feels strange approach to me).
        • This approach would cause problems when testing plugins with "mvn hpi:run".
          • As property files aren't compiled and read from source directories directly at that time.
      2. Make libraries reading property files accept UTF-8 letters.

      Actually, I'm not sure this is a good idea.
      I created this ticket and will create pull requests to request comments about this idea.

          [JENKINS-41729] Using utf-8 letters in property files

          ikedam created issue -
          ikedam made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          ikedam made changes -
          Status Original: In Progress [ 3 ] New: Open [ 1 ]
          ikedam made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          ikedam made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Jesse Glick made changes -
          Assignee Original: ikedam [ ikedam ] New: Tim Jacomb [ timja ]
          Tim Jacomb made changes -
          Remote Link New: This issue links to "https://github.com/jenkinsci/jenkins-test-harness/pull/433 (Web Link)" [ 27741 ]
          Tim Jacomb made changes -
          Remote Link New: This issue links to "https://github.com/jenkinsci/jenkins/pull/6550 (Web Link)" [ 27742 ]
          Tim Jacomb made changes -
          Remote Link New: This issue links to "https://github.com/jenkinsci/stapler/pull/357 (Web Link)" [ 27743 ]
          Tim Jacomb made changes -
          Remote Link New: This issue links to "https://github.com/jenkinsci/localization-support-plugin/pull/6 (Web Link)" [ 27744 ]
          Alexander Brandes made changes -
          Released As New: 2.358
          Resolution New: Fixed [ 1 ]
          Status Original: In Review [ 10005 ] New: Closed [ 6 ]

            timja Tim Jacomb
            ikedam ikedam
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: