Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Kex Engine #102

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Update Kex Engine #102

wants to merge 4 commits into from

Conversation

mastan-rus
Copy link
Contributor

Brief explanation of the change

Friendship Club and Toast Time have files config.kpf which make false positives for the Kex engine. This fix adds negative lookbehind to exclude those files.

@xPaw
Copy link
Member

xPaw commented Sep 21, 2021

There's also Friendship Club.app/Contents/Resources/data/data.kpf so that's not enough.

But even then, doing negative lookbehinds is not exactly something I'd want to do here.

@mastan-rus
Copy link
Contributor Author

Can filesize be checked somehow? In Friendship Club and Toast Time kpf files are very small, less than 1 KB. In Kex engine games they contain resources and are relatively big, 20 MB and more.

@Lifeismana Lifeismana changed the title Kex Engine - remove false positives Update Kex Engine Jan 7, 2024
@SteamTracker
Copy link
Member

After running changed rules in this pull request (this bot only tests rules.ini changes):

Changed technologies: Engine.Kex

These apps no longer match: (1) 316660

These apps now match: (20) 2310 2320 2330 2340 9030 9040 332760 405820 405830 410710 668980 1010750 1148590 1413870 1421490 1678420 1678430 1996770 2292260 2424570

Please confirm that all these changed apps are correct.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: To Triage
Development

Successfully merging this pull request may close these issues.

None yet

3 participants