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 prerequisites.md #1293

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

Conversation

WarWolflll
Copy link

Add a new line after COMMIT to avoid "Bad argument" error.

Add a new line after COMMIT to avoid "Bad argument" error.
@vercel
Copy link

vercel bot commented Sep 3, 2023

@WarWolflll is attempting to deploy a commit to the Mastodon Team on Vercel.

A member of the Team first needs to authorize it.

@vmstan
Copy link
Contributor

vmstan commented Nov 24, 2023

It looks like there is a mismatch of single tick/quotes

`COMMIT'

@@ -62,7 +62,7 @@ First, install iptables-persistent. During installation it will ask you if you w
apt install -y iptables-persistent
```

Edit `/etc/iptables/rules.v4` and put this inside:
Edit `/etc/iptables/rules.v4` and put this inside: (There must to be a new line after COMMIT, or it will get an error message ```Bad argument `COMMIT'``` at next step.)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Edit `/etc/iptables/rules.v4` and put this inside: (There must to be a new line after COMMIT, or it will get an error message ```Bad argument `COMMIT'``` at next step.)
Edit `/etc/iptables/rules.v4` and put this inside:

@@ -99,6 +99,7 @@ Edit `/etc/iptables/rules.v4` and put this inside:
-A FORWARD -j REJECT

COMMIT

```

Copy link
Contributor

@vmstan vmstan Dec 5, 2023

Choose a reason for hiding this comment

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

Suggested change
Please note there must to be a new line after COMMIT, or you will receive the error message `Bad argument COMMIT`.

@vmstan vmstan added the install-guide Installing a Mastodon instance label Dec 5, 2023
@andypiper andypiper self-assigned this Dec 10, 2023
@vmstan
Copy link
Contributor

vmstan commented Dec 11, 2023

@WarWolflll are you available to review?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
install-guide Installing a Mastodon instance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants