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

Extend builtin with /spawn chat command #1690

Closed
wants to merge 1 commit into from
Closed

Extend builtin with /spawn chat command #1690

wants to merge 1 commit into from

Conversation

SmallJoker
Copy link
Member

This adds a /spawn chat command if it has not been added after one second after game start.

@Amaz1
Copy link
Contributor

Amaz1 commented Sep 26, 2014

+1

@kaeza
Copy link
Contributor

kaeza commented Sep 26, 2014

Why not outright register it?

Games that have a different approach to implement the /spawn command can just re-register it, and games which don't want/need it can just minetest.chatcommands.spawn = nil.

@SmallJoker
Copy link
Member Author

@kaeza Thanks for this hint. Updated the commit.

@kahrl
Copy link
Contributor

kahrl commented Sep 27, 2014

This command should require the teleport privilege.

@SmallJoker
Copy link
Member Author

@kahrl Why? All spawn commands I've seen work without the "teleport" priv.

@kahrl
Copy link
Contributor

kahrl commented Sep 28, 2014

Because it is a teleport?

In survival this lets you get out of a sticky situation and teleport into a most likely safe area (like /kill as in #396, but without death penalties such as losing items). This makes it a cheat and therefore it should require a cheat privilege such as "teleport".

@Calinou
Copy link
Member

Calinou commented Sep 29, 2014

We already have /home and /sethome which require just the home privilege.

The solution is to create a spawn privilege in order to use /spawn.

@SmallJoker
Copy link
Member Author

@kahrl I see what you mean, a spawn privilege would be a better soulation in this case, as @Calinou said. But it's again a privilege with a single usage, a setting in minetest.conf would be a better soulation then.

static_spawn_command = true

@ShadowNinja
Copy link
Member

A /die command would be good. It wouldn't require any special privileges because you would receive any death penalties that the server imposed, such as losing items to bones.

@SmallJoker
Copy link
Member Author

Dying is not what the most spawn commands target to do. Also, that would be an other PR.

@BlockMen
Copy link
Contributor

BlockMen commented Oct 2, 2014

I agree with @ShadowNinja that a /die command would be better. Some more thought on that here:
http://irc.minetest.ru/minetest-dev/2014-10-02#i_3959299

@SmallJoker
Copy link
Member Author

At least 2 devs disagree. Closed.

The /die command is an other topic.

@SmallJoker SmallJoker closed this Oct 2, 2014
@SmallJoker SmallJoker deleted the static_spawn_ext branch November 8, 2014 17:56
@SmallJoker SmallJoker restored the static_spawn_ext branch November 13, 2014 18:15
@SmallJoker SmallJoker deleted the static_spawn_ext branch December 4, 2014 18:31
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.

7 participants