TeamTofuShop/segatools
24
36
Fork 24
forked from Hay1tsme/segatools
Code Issues 4 Pull Requests 4 Actions Packages Releases 15 Wiki Activity

Vfs: Add ability to redirect arbitary paths (= serial ports) #62

Open
Haruka wants to merge 6 commits from Haruka/segatools:file_redirection into develop
pull from: Haruka/segatools:file_redirection
merge into: TeamTofuShop:develop
Conversation 5 Commits 6 Files Changed 6 +103 -3
Haruka commented 2025-03-09 11:42:15 +00:00
Owner
Copy Link

As someone running several games on the same cabinet that share most of the hardware (like KCA and FGO), it's a bit of a hassle switching the COM ports of the hardware around all the time.

This commit adds the ability to place arbitary path redirections in segatools.ini, which is mostly useful for hardcoded serial ports.
Technically this would also allow you to redirect arbitary paths, but I haven't tested that yet.

An example that I personally use is:

[vfs]
amfs=amfs
appdata=.
option=option

; aime reader
redirection0from=\\.\COM10
redirection0to=\\.\COM3
; rfid reader
redirection1from=\\.\COM12
redirection1to=\\.\COM2
; POP led begone
redirection2from=\\.\COM9
redirection2to=\\.\COM90
; partition to pop
redirection3from=\\.\COM11
redirection3to=\\.\COM9
As someone running several games on the same cabinet that share most of the hardware (like KCA and FGO), it's a bit of a hassle switching the COM ports of the hardware around all the time. This commit adds the ability to place arbitary path redirections in segatools.ini, which is mostly useful for hardcoded serial ports. Technically this would also allow you to redirect arbitary paths, but I haven't tested that yet. An example that I personally use is: ``` [vfs] amfs=amfs appdata=. option=option ; aime reader redirection0from=\\.\COM10 redirection0to=\\.\COM3 ; rfid reader redirection1from=\\.\COM12 redirection1to=\\.\COM2 ; POP led begone redirection2from=\\.\COM9 redirection2to=\\.\COM90 ; partition to pop redirection3from=\\.\COM11 redirection3to=\\.\COM9 ```
Haruka added 5 commits 2025-03-09 11:42:16 +00:00
fix msvc build again 9c0ca7d66c
add support for standalone MSVC compiler bf3ff76f0c
gitignore some newer meson file 2313113fd0
Merge remote-tracking branch 'fork-origin/develop' into develop 9b1f07e41b 
# Conflicts:
#	hooklib/meson.build
#	meson.build
#	platform/system.c
vfs: add file redirection feature 3772192c77
zaphkito commented 2025-03-09 12:54:00 +00:00
Contributor
Copy Link

is this work for chusan about

[vfs]
amfs=amfs
appdata=.
option=option

; slider
redirection0from=\\.\COM1
redirection0to=\\.\COM4
; aime reader
redirection1from=\\.\COM4
redirection1to=\\.\COM1

make COM1 is aime reader and COM4 is slider

is this work for chusan about ``` [vfs] amfs=amfs appdata=. option=option ; slider redirection0from=\\.\COM1 redirection0to=\\.\COM4 ; aime reader redirection1from=\\.\COM4 redirection1to=\\.\COM1 ``` make COM1 is aime reader and COM4 is slider
akanyan reviewed 2025-03-10 10:20:15 +00:00
platform/vfs.c Outdated
@ -544,0 +571,4 @@
return S_FALSE;
}
if (path_compare_w(src, from, wcslen(from)) != 0) {
akanyan commented 2025-03-10 10:20:15 +00:00
First-time contributor
Copy Link

You're calling strlen for each path access, multiplied by the number of redirections

You're calling strlen for each path access, multiplied by the number of redirections
Haruka commented 2025-03-10 11:51:45 +00:00
Author
Owner
Copy Link

corrected, thanks

corrected, thanks
Haruka marked this conversation as resolved
Haruka added 1 commit 2025-03-10 11:51:28 +00:00
vfs: strlen optimization ff40462779
Haruka commented 2025-03-10 11:52:37 +00:00
Author
Owner
Copy Link

@zaphkito IF chusan opens ports like that, it will work, but games can also open ports in different ways.

@zaphkito IF chusan opens ports like that, it will work, but games can also open ports in different ways.
Dniel97 commented 2025-03-10 17:45:12 +00:00
Owner
Copy Link

Thanks for the PR, I will try to review it and test it (or maybe @Hay1tsme?) but looks good so far :)

Thanks for the PR, I will try to review it and test it (or maybe @Hay1tsme?) but looks good so far :)
Dniel97 commented 2025-04-05 16:00:25 +00:00
Owner
Copy Link

So I tried the changes on FGO AC and I connected the VFD on COM7 (just for testing) and added the following redirect:

; vfd
redirection0from=\\.\COM1
redirection0to=\\.\COM7

but that didn't work.

So I tried the changes on FGO AC and I connected the VFD on COM7 (just for testing) and added the following redirect: ``` ; vfd redirection0from=\\.\COM1 redirection0to=\\.\COM7 ``` but that didn't work.
This pull request has changes conflicting with the target branch.
  • doc/config/common.md
  • meson.build
  • meson_options.txt
  • platform/config.c
  • platform/vfs.c
  • platform/vfs.h
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u file_redirection:Haruka-file_redirection
git checkout Haruka-file_redirection
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
question
More information is needed
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
Dniel97 Haruka Hay1tsme
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: TeamTofuShop/segatools#62
No description provided.
Delete Branch "Haruka/segatools:file_redirection"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?