Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-27910

Cyrillic symbols and file removal

    XMLWordPrintable

Details

    Description

      Java exception on trying to remove some path with cyrillic symbols. For example if you choose Workspace -> Wipe Out Current Workspace.

      Attachments

        Issue Links

          Activity

            danielbeck Daniel Beck added a comment -

            Please make sure the `file.encoding` system property and the locale related env vars have sane (UTF-8ish) values on the slave.

            danielbeck Daniel Beck added a comment - Please make sure the `file.encoding` system property and the locale related env vars have sane (UTF-8ish) values on the slave.
            markewaite Mark Waite added a comment -

            Can you provide the specific characters which are in the problem file name? There are already automated tests in the code for various interesting characters, and it may be that we need to add more tests with the specific characters you have identified.

            Also, can you confirm (as danielbeck noted) that the Java configuration of the slave running that process is using UTF-8 settings for its locale?

            markewaite Mark Waite added a comment - Can you provide the specific characters which are in the problem file name? There are already automated tests in the code for various interesting characters, and it may be that we need to add more tests with the specific characters you have identified. Also, can you confirm (as danielbeck noted) that the Java configuration of the slave running that process is using UTF-8 settings for its locale?

            Mark:
            Nothing specific in those filenames. I met this issue from time to time with different cyrillic filenames. For example лицо.png (face.png) or плавание.png (swim.png).

            Daniel:
            One build machine was configured to UTF-8 but another was not. I can not provoke bug now but if I did this I will comment this issue.

            Thanks a lot for you help.

            alexchernyy Alexander Chernyy added a comment - Mark: Nothing specific in those filenames. I met this issue from time to time with different cyrillic filenames. For example лицо.png (face.png) or плавание.png (swim.png). Daniel: One build machine was configured to UTF-8 but another was not. I can not provoke bug now but if I did this I will comment this issue. Thanks a lot for you help.

            Code changed in jenkins
            User: Mark Waite
            Path:
            src/test/java/org/jenkinsci/plugins/gitclient/GitAPITestCase.java
            http://jenkins-ci.org/commit/git-client-plugin/f6900736f255f7cf99c772b1002fb42b87371905
            Log:
            Add test cyrillic file names from JENKINS-27910

            This does not address the core java library refusal to delete files
            named with unmappable characters. That needs a different solution, and
            a solution which would be better done in Jenkins core than in the git
            client plugin.

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Mark Waite Path: src/test/java/org/jenkinsci/plugins/gitclient/GitAPITestCase.java http://jenkins-ci.org/commit/git-client-plugin/f6900736f255f7cf99c772b1002fb42b87371905 Log: Add test cyrillic file names from JENKINS-27910 This does not address the core java library refusal to delete files named with unmappable characters. That needs a different solution, and a solution which would be better done in Jenkins core than in the git client plugin.
            danielbeck Daniel Beck added a comment -

            alexchernyy Any news about this? Did this occur again after you fixed the slave?

            danielbeck Daniel Beck added a comment - alexchernyy Any news about this? Did this occur again after you fixed the slave?

            danielbeck It looks correct. There were no crashes after fix.

            alexchernyy Alexander Chernyy added a comment - danielbeck It looks correct. There were no crashes after fix.
            danielbeck Daniel Beck added a comment -

            Config issue (though we could probably improve the docs), so resolving as not a defect.

            danielbeck Daniel Beck added a comment - Config issue (though we could probably improve the docs), so resolving as not a defect.

            People

              Unassigned Unassigned
              alexchernyy Alexander Chernyy
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: