-
Notifications
You must be signed in to change notification settings - Fork 7
Add chusanhook for Chunithm New - [opened] #74
Comments
In GitLab by @rakisaionji on Nov 8, 2021, 22:06 marked the checklist item Chunithm New as completed |
In GitLab by @rakisaionji on Nov 8, 2021, 22:06 marked the checklist item Chunithm New as incomplete |
In GitLab by @rakisaionji on Nov 8, 2021, 22:07 marked the checklist item Chunithm New as completed |
In GitLab by @rakisaionji on Nov 8, 2021, 22:08 marked the checklist item Chunithm New as incomplete |
In GitLab by @rakisaionji on Nov 8, 2021, 22:09 Some folks said @felix has Chunithm New so maybe he could test it? |
In GitLab by @Mai on Nov 15, 2021, 09:07 I haven't been able to get your branch to work on either versions of Superstar or NEW when building from Linux with mingw-64, but the same compiled code works on nu1.1 versions of the game fine. The process will dump out
Event Viewers says it's an access violation, but without any knowledge of attaching a debugger to this, I'm afraid I'm out of my usefulness. |
In GitLab by @rakisaionji on Nov 16, 2021, 20:22 approved this merge request |
In GitLab by @rakisaionji on Nov 16, 2021, 20:30 Well, since Chunithm Crystal I believe this MR would not be merged in near future unless some certain "hackers" in a "virus factory" decided to "release" their unpacked stuff. However, I have a strong feeling that @felix has tested this thoroughly and trying to improve with more stuff. Btw some people said there is something wrong with |
In GitLab by @Mai on Nov 17, 2021, 04:18 Commented on Package.mk line 23 When building this solution, architecture suffixes are not present on build. These would return file not found errors |
In GitLab by @Mai on Nov 17, 2021, 04:30 There's honestly no real rush, if the MR is not up to a level of standard that is appropriate to what DJH expects in their repo they are free to close or improve off of it. I can't tell you not to spit at them in the thread here but it won't earn you any respect from anyone |
In GitLab by @rakisaionji on Nov 17, 2021, 06:07 Tbh I have no problem having DJH improving this MR. I welcome every positive commits to improve this MR. Otherwise, I don't care about useless comments and such. If it's working for me then it should work for you, that's enough for me. If you can fix |
In GitLab by @rakisaionji on Nov 17, 2021, 06:10 Commented on Package.mk line 23 I have realized it already when a patron reported the bug to me couple days ago. Any suggestion? |
In GitLab by @rakisaionji on Nov 17, 2021, 06:13 I have no problem having DJH improving this MR and also their segatools. Just their gatekeeping scheme is getting more sketchy by giving n-0 to useless leakers rather than people who can actually do shit. |
In GitLab by @Mai on Nov 17, 2021, 09:02 Commented on Package.mk line 23 this is a blatant hack because I looked at the meson documentation for five-ten minutes instead of, ironically, wanting to go out to the arcade. I keep iterating over this implementation, granted how reducing work for existing methods are with a focus on x86 builds. to my dismay, it's either at the start of the filename, or as the extension (forcing a period).
and this does generate some good files, note that the x86_64 is consistant with their reference tables but we can further extend this by using ternary operators. This allows you to keep the consistency of not having a output with an architecture label on 32 bit versions, but specifying them for the 64
I guess I learned a lot today, but It's probably just easier with cp commands |
In GitLab by @rakisaionji on Feb 5, 2022, 17:07 marked the checklist item Chunithm New as completed |
In GitLab by @rakisaionji on Feb 5, 2022, 17:11 Somehow Many thanks to Mai and Athene for their willing to support, I am very appreciated for your suggestions and contributions. I will take everything into account and reflect it into my private build. |
In GitLab by @rakisaionji on Nov 8, 2021, 22:05
Merges chusan -> master
New Feature
Summary
Just like the title, this feature is to add support for Chunithm on ALLS platform such as Super Star and New.
Description
Originally named
chunewhook
from my private fork ofsegatools
.To maintain compatibility with custom DLLs for old Chunithm, I will keep the same
chuniio
andchuni-dll
.That means we can use older
segatools.ini
fromchunihook
forchusanhook
with some proper modifications.Related Issue
There were 3 failed attempts (!46 !47 !48), pathetically.
How Has This Been Tested?
Originally built with VS2019 using MSVC backend, works well on my ALLS UX workbench.
Tested with Chunithm Super Star and Super Star Plus. They should be literally the same with New.
Checklist
The text was updated successfully, but these errors were encountered: