FileDetectionRuleSets icon indicating copy to clipboard operation
FileDetectionRuleSets copied to clipboard

Update Pico8

Open MichaelHinrichs opened this issue 2 years ago • 14 comments

SteamDB app page links to a few games using this

https://steamdb.info/app/1494910/

Brief explanation of the change

The PICO-8 game pemsa uses files with a "p8" file type.

MichaelHinrichs avatar Jun 23 '23 16:06 MichaelHinrichs

This only matches the literal file .p8.

xPaw avatar Jun 23 '23 18:06 xPaw

Missing a description as well.

gibbed avatar Jun 23 '23 18:06 gibbed

This only matches the literal file .p8.

Then why does it work in all of these files? https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/tests/types/Engine.ChromeEngine.txt https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/tests/types/Engine.GameMaker.txt https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/tests/types/Engine.RPGMaker.txt https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/tests/types/Engine.RenPy.txt

MichaelHinrichs avatar Jun 26 '23 15:06 MichaelHinrichs

Missing a description as well.

PICO-8 already has a description. Is there something else i should have described? https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/descriptions/Engine.Pico8.md

MichaelHinrichs avatar Jun 26 '23 16:06 MichaelHinrichs

PICO-8 already has a description. Is there something else i should have described? https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/descriptions/Engine.Pico8.md

Sorry, that's my bad-- I incorrectly assumed you were adding a new entry for Pico8 entirely.

gibbed avatar Jun 26 '23 16:06 gibbed

This only matches the literal file .p8.

Then why does it work in all of these files? https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/tests/types/Engine.ChromeEngine.txt https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/tests/types/Engine.GameMaker.txt https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/tests/types/Engine.RPGMaker.txt https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/tests/types/Engine.RenPy.txt

Did you take a look at these engines rules ?

(?:^|/).p8$ =>.p8 \.rpack$ => .rpack \.gm(?:spr|bck)$ => .gmbck | .gmspr \.(?:rgssad|rvproj2|rgss3a|rgss2a)$ => .rgss2a | .rgss3a | .rgssad |.rvproj2 \.rpyb$ => .rpyb

Lifeismana avatar Jun 26 '23 16:06 Lifeismana

(?:^|/) matches start of string or after a slash. It should be \.p8$

xPaw avatar Jun 26 '23 21:06 xPaw

This only matches the literal file .p8.

Then why does it work in all of these files? https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/tests/types/Engine.ChromeEngine.txt https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/tests/types/Engine.GameMaker.txt https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/tests/types/Engine.RPGMaker.txt https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/tests/types/Engine.RenPy.txt

Did you take a look at these engines rules ?

(?:^|/).p8$ =>.p8 \.rpack$ => .rpack \.gm(?:spr|bck)$ => .gmbck | .gmspr \.(?:rgssad|rvproj2|rgss3a|rgss2a)$ => .rgss2a | .rgss3a | .rgssad |.rvproj2 \.rpyb$ => .rpyb

Where the heck does it say any of that?

MichaelHinrichs avatar Jul 07 '23 14:07 MichaelHinrichs

You might want to read (or reread) https://github.com/SteamDatabase/FileDetectionRuleSets/blob/main/CONTRIBUTING.md

(?:^|/).p8$ from rules.ini => .p8 from the test file

Lifeismana avatar Jul 07 '23 14:07 Lifeismana

Is that better?

MichaelHinrichs avatar Jul 09 '23 15:07 MichaelHinrichs

This is the only game that uses this extension, how come?

xPaw avatar Jul 11 '23 07:07 xPaw

I don't know, does it matter? It's a pico-8 game that's not detected as pico-8! Add it to the list.

MichaelHinrichs avatar Aug 10 '23 07:08 MichaelHinrichs

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

Changed technologies: Engine.Pico8

These apps now match: (29) 642020 917140 1275070 1282710 1296550 1354460 1448220 1448720 1451860 1472810 1494910 1519610 1749310 1857970 1953060 2216120 2318570 2425010 2447920 2448440 2552900 2587030 2658990 2675390 2675400 2684450 2707300 2737990 2766260

Please confirm that all these changed apps are correct.

SteamDB-Tracker avatar Mar 26 '24 13:03 SteamDB-Tracker

False positive. https://steamdb.info/depot/1857971/

MichaelHinrichs avatar Mar 31 '24 16:03 MichaelHinrichs