-
Notifications
You must be signed in to change notification settings - Fork 2k
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
pv1000.xml: Added sixteen working homebrew cartridges #12801
base: master
Are you sure you want to change the base?
Conversation
New working software list additions: ---------------------------------------- Aerial [Inufuto] Antair [Inufuto] Ascend [Inufuto] Battlot [Inufuto] Bootskell [Inufuto] Cacorm [Inufuto] Cavit [Inufuto] Cracky [Inufuto] Guntus [Inufuto] Hopman [Inufuto] Lift [Inufuto] Mazy [Inufuto] Neuras [Inufuto] Osotos [Inufuto] Ruptus [Inufuto] Yewdow [Inufuto]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Please capitalise the titles the way Inufuto does on his web site: http://inufuto.web.fc2.com/8bit/
- Don’t bother with the
offset
attribute when all the cartridges in the software list only have a single ROM (and if you do need it, put it next to the size to make reading the XML easier).
hash/pv1000.xml
Outdated
<software name="antair"> | ||
<description>Antair</description> | ||
<year>2024</year> | ||
<publisher>Inufuto</publisher> | ||
<part name="cart" interface="pv1000_cart"> | ||
<dataarea name="rom" size = "0x4000"> | ||
<rom name="antiair.rom" size="0x4000" crc="1478db1a" sha1="b0c640776dcdb79db8a7cc8b2387a443182e1036" offset="0"/> | ||
</dataarea> | ||
</part> | ||
</software> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is supposed to be AntiAir: http://inufuto.web.fc2.com/8bit/antiair/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
romset ID should also be antiair
, for consistency with coleco_homebrew.xml
Line 72 in bfb41f8
<software name="antiair"> |
One Question: on Inufuto's website Aerial is labeled as AERIAL, but in coleco_homebrew.xml it's Aerial, which one should be used? |
I'm a little confused about this request to be honest. I thought we decided long ago that we weren't calling things like Pac-Man PAC-MAN just because that's what it shows. |
Also remove unnecessary offset attribute
New working software list additions:
Aerial [Inufuto]
AntiAir [Inufuto]
Ascend [Inufuto]
Battlot [Inufuto]
Bootskell [Inufuto]
Cacorm [Inufuto]
Cavit [Inufuto]
Cracky [Inufuto]
Guntus [Inufuto]
Hopman [Inufuto]
Lift [Inufuto]
Mazy [Inufuto]
Neuras [Inufuto]
Osotos [Inufuto]
Ruptus [Inufuto]
Yewdow [Inufuto]