RetroPie forum home
    • Recent
    • Tags
    • Popular
    • Home
    • Docs
    • Register
    • Login
    Please do not post a support request without first reading and following the advice in https://retropie.org.uk/forum/topic/3/read-this-first

    Problem run ROMS with comma character

    Scheduled Pinned Locked Moved Help and Support
    romsruncommandbug
    20 Posts 6 Posters 2.6k Views
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • G
      gleam2003
      last edited by gleam2003

      When launch roms like this
      Action Fighter (USA, Europe).7z

      System back to emulationstation and no log avviable in /dev/shm

      No problem with roms like
      Alex Kidd - The Lost Stars (World).7z

      Test with other roms with "," and the problem is the same.

      1 Reply Last reply Reply Quote 0
      • G
        gleam2003
        last edited by

        Help, I don't want to rename all my roms and all my gamelist.

        mediamogulM 1 Reply Last reply Reply Quote 0
        • mediamogulM
          mediamogul Global Moderator @gleam2003
          last edited by

          @gleam2003

          After launching one of those games, post the contents of /dev/shm/runcommand.log. Also, please post the information requested at https://retropie.org.uk/forum/topic/3/read-this-first.

          RetroPie v4.5 • RPi3 Model B • 5.1V 2.5A PSU • 16GB SanDisk microSD • 512GB External Drive

          1 Reply Last reply Reply Quote 0
          • G
            gleam2003
            last edited by

            No log was created in /dev/shm, already write it

            1 Reply Last reply Reply Quote 0
            • G
              gleam2003
              last edited by gleam2003

              Run EmulationStation from ssh terminal for watch the error, report:
              sh: 1: Syntax error: "(" unexpected

              Pi Model or other hardware: Pi3
              RetroPie Version Used (eg 3.6, 3.8.1, 4.1 - do not write latest): 4.3
              Built From: RetroPie 4.3 clean image
              USB Devices connected: none exept snes style controler
              Controller used: SNES style (type="joystick" deviceName="USB Gamepad " deviceGUID="03000000790000001100000010010000")
              Error messages received: sh: 1: Syntax error: "(" unexpected
              Log found in /dev/shm/runcommand.log (if relevant): No data logged! (work with working roms)
              Guide used: (Mention if you followed a guide)
              File: Action Fighter (USA, Europe).7z
              Emulator: all emulator
              How to replicate the problem: use roms with , OR (,) on name

              mituM 1 Reply Last reply Reply Quote 0
              • mituM
                mitu Global Moderator @gleam2003
                last edited by

                @gleam2003 I have no problem with ROMs having , in their names, either running or showing in Emulationstation. Can you try an update for all packages (from binary) from the RetroPie-Setup and then restart Emulationstation ?

                1 Reply Last reply Reply Quote 0
                • G
                  gleam2003
                  last edited by gleam2003

                  Update, same issue

                  I test with change the roms name to find when the problem occurs.

                  A space after "(" is the problem. My error is to suppose that the problem is the ","

                  Can you verify?

                  1 Reply Last reply Reply Quote 0
                  • G
                    gleam2003
                    last edited by gleam2003

                    @mediamogul @mitu
                    This is the command pass to system() command in runSystemCommand function in platform.cpp

                    /opt/retropie/supplementary/runcommand/runcommand.sh 0 SYS mastersystem /home/pi/RetroPie/roms/mastersystem/Alex\ Kidd\ in\ Miracle\ World\ (USA,\ Europe).7z

                    My EmulationStation is the development version and I think the problem is in StringUtil.cpp that substitute the strreplace function

                    1 Reply Last reply Reply Quote 0
                    • cyperghostC
                      cyperghost
                      last edited by cyperghost

                      This post is deleted!
                      1 Reply Last reply Reply Quote 0
                      • cyperghostC
                        cyperghost
                        last edited by

                        @gleam2003 Don't know what going wrong here I just renamed one mastersystem ROM to Alex Kidd in Miracle World (USA, Europe).sms and it worked without any flaws.
                        Does your 7zip file contain more than one ROM?

                        G 1 Reply Last reply Reply Quote 0
                        • G
                          gleam2003 @cyperghost
                          last edited by

                          @cyperghost
                          Probably problem is only in development version of emulationstation
                          Wait i'm going to recompile the stable branch to watch if is ok

                          1 Reply Last reply Reply Quote 0
                          • G
                            gleam2003
                            last edited by gleam2003

                            I can confirm, the problem is with development tree

                            with stable:
                            /opt/retropie/supplementary/runcommand/runcommand.sh 0 SYS mastersystem /home/pi/RetroPie/roms/mastersystem/Alex\ Kidd\ in\ Miracle\ World\ \(USA,\ Europe\).7z

                            with development:
                            /opt/retropie/supplementary/runcommand/runcommand.sh 0 SYS mastersystem /home/pi/RetroPie/roms/mastersystem/Alex\ Kidd\ in\ Miracle\ World\ (USA,\ Europe).7z

                            In stable ther's escapePath function, missing completely in development

                            1 Reply Last reply Reply Quote 0
                            • T
                              Tomaz
                              last edited by Tomaz

                              I fix this later today when I'm not at work.

                              There is still an escapePath function, it's just moved to a different location in the source.

                              Looking at the output, it seems it didn't escape the last ')' properly.

                              G 1 Reply Last reply Reply Quote 0
                              • G
                                gleam2003 @Tomaz
                                last edited by gleam2003

                                @tomaz
                                Thank you
                                Not only last ) but also ( and I think any other charachter escape in stable branch with escapePath() function

                                Edit: there's a space before (

                                T 2 Replies Last reply Reply Quote 0
                                • T
                                  Tomaz @gleam2003
                                  last edited by

                                  @gleam2003
                                  I see now that both ( and ) fails, but the new escapePath function definitely runs since it has escaped all the spaces, will know more when I'm back home later today.

                                  1 Reply Last reply Reply Quote 0
                                  • T
                                    Tomaz @gleam2003
                                    last edited by

                                    @gleam2003
                                    Fixed and submitted, just need to wait for it to be accepted and merged.

                                    G 1 Reply Last reply Reply Quote 0
                                    • G
                                      gleam2003 @Tomaz
                                      last edited by

                                      @tomaz
                                      Thank you, compiled and tested.

                                      T 1 Reply Last reply Reply Quote 0
                                      • T
                                        Tomaz @gleam2003
                                        last edited by

                                        @gleam2003
                                        Did the fix work for you? It should be merged in the dev branch now.

                                        G 1 Reply Last reply Reply Quote 0
                                        • G
                                          gleam2003 @Tomaz
                                          last edited by

                                          @tomaz
                                          Yes sorry I think you understand from my previous post

                                          1 Reply Last reply Reply Quote 0
                                          • DwarfboysimD
                                            Dwarfboysim
                                            last edited by

                                            Hi,
                                            There are free utilities that can search an entire folder and remove that one character from all file names. Pretty sure it’s called Bulk Rename Utility. I used it to tidy up all my file names

                                            Dwarfboysim

                                            1 Reply Last reply Reply Quote 0
                                            • First post
                                              Last post

                                            Contributions to the project are always appreciated, so if you would like to support us with a donation you can do so here.

                                            Hosting provided by Mythic-Beasts. See the Hosting Information page for more information.