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 abuse.sh #2450

Closed
wants to merge 1 commit into from
Closed

Update abuse.sh #2450

wants to merge 1 commit into from

Conversation

justnibio
Copy link

Change Description
Add Help
Change function configure_abuse()

Change Description
Add Help
Change function configure_abuse()
moveConfigDir "$home/.abuse" "$md_conf_root/abuse"
addPort "$md_id" "abuse" "Abuse" "pushd $md_inst; $md_inst/abuse; popd"
addPort "$md_id" "abuse" "Abuse port of run and gum game" "pushd $md_inst; $md_inst/abuse -datadir $romdir/ports/$md_id; popd"
Copy link
Member

Choose a reason for hiding this comment

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

the description "port of run and gum game" shouldn't be here (it's gun and not gum also).

@dankcushions
Copy link
Member

@@ -10,8 +10,9 @@
#

rp_module_id="abuse"
rp_module_desc="Abuse"
rp_module_desc="Abuse - port of run and gun game"
rp_module_license="GPL https://raw.githubusercontent.com/Xenoveritas/abuse/master/COPYING"
Copy link
Member

Choose a reason for hiding this comment

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

This isn't your fault, but if you do get around to updating the PR with Jools' other comments, it would be helpful if you fixed this typo as well. The function name is supposed to be the non-US spelling (rp_module_licence). Thanks.

@luizoti
Copy link

luizoti commented Jan 21, 2021

Is this module working in relation to the instalation? in the past, I tried and always failed.

@joolswills
Copy link
Member

The module is currently disabled. I just tested with adjusting the module to use manually installed game data and it wasn't working correctly in regards to joystick/keyboard input.

I will close this PR though as it no longer applies.

@joolswills joolswills closed this Jan 27, 2021
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.

5 participants