Core ReactOS
  1. Core ReactOS
  2. CORE-12614

regression of r73486 [SERVICES] empty folder "Documents and settings" is created during ros setup in german localization

    Details

    • Type: Bug Bug
    • Status: Untriaged Untriaged
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: Services
    • Labels:
    • Guilty Revision:
      73,486

      Description

      r73484 is not affected
      r73486 is guilty and from that revision and newer
      initial ros setup will always create the empty folder "C:\Documents And Settings".
      Please note, that I am using german build here, where another folder with proper contents does already exist "C:\Dokumente und Einstellungen".

      I can delete the empty folder, it will not be recreated upon reboot.

        Issue Links

          Activity

          Hide
          reactosfanboy
          added a comment -

          cc HBelusca since you committed what you referred to as hack in r73486

          Show
          reactosfanboy
          added a comment - cc HBelusca since you committed what you referred to as hack in r73486
          Hide
          Mark Jansen
          added a comment -

          Maybe ScmApplyServiceAccountsHack is called before locale is set?

          Show
          Mark Jansen
          added a comment - Maybe ScmApplyServiceAccountsHack is called before locale is set?
          Hide
          HBelusca
          added a comment -

          That's indeed possible Mark Jansen. I will have a look at this. But technically I would expect it uses the locale chosen @ 1st stage installation...

          Show
          HBelusca
          added a comment - That's indeed possible Mark Jansen . I will have a look at this. But technically I would expect it uses the locale chosen @ 1st stage installation...

            People

            • Assignee:
              HBelusca
              Reporter:
              reactosfanboy
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated: