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 Skyland gba rom #30

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

evanbowman
Copy link
Contributor

The last one was a couple of years old and I've changed a lot of stuff since then

The last one was two years old and I've changed a lot of stuff since then
},
"playable": true
}
],
"license": "MIT",
"license": "Custom",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm surprised this passes the schema validator, to be honest :/
Anyway, if I understood correctly now the game is basically "source-available" but not open source anymore so we can simply remove the "license field"? Or maybe ad a "source-available" option for it..

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I unfortunately had to change the license after dealing with issues with pirated bootleg cartridges, and licensing complaints from Steam, who do not like open source licenses that much. The MIT license, which applied to the jam version of the game, was later superseded by the GPL license, which I used until valve rejected my game submission. I would be fine with either removing the license field or labeling it as source available, let me know which you'd prefer and I'll fix my PR

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shit, that sucks. I'd say, for the sake of keeping this PR in scope, removing the Open Source tag and the license field should be okay and we can merge it quickly. I can in parallel open an issue regarding changing the Schema to accomodate and expose "source-available" entries faithfully.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While it's no longer open source, the current license allows modification and redistribution, the only limitation is on selling copies of the ROM. The license is still compatible with publishing on homebrew hub :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, sure! But the entry shouldn't be listed there as "Open Source" anymore, no ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah correct! I tried to update the checksums, the license field, and the other stuff in the game.json, please let me know if there's anything I can do to fix my PR, I am happy to make any changes that you suggest!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe by github ui is glitched? I see the license field still there (which we can remove) and the "Open Source" tag still in the tags?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(I'd love to have "source available" / "custom" as possible values in "license" but the schema is not ready for it and pushing the entry as "custom" license would break the frontends, which atm expect SPDX identifiers..)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants