Skip to content

Readme cleanup#305

Merged
AnthonyGiorgio merged 3 commits intomainfrom
readme-cleanup
May 22, 2023
Merged

Readme cleanup#305
AnthonyGiorgio merged 3 commits intomainfrom
readme-cleanup

Conversation

@AnthonyGiorgio
Copy link
Copy Markdown
Collaborator

Fix up some markdown warnings, as well as grammar and punctuation.

Copy link
Copy Markdown
Collaborator

@MikeFultonDev MikeFultonDev left a comment

Choose a reason for hiding this comment

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

Thank you! Updates look good

Comment thread README.md Outdated
GNU make requires GNU m4, automake, autoconf, Perl, and a C compiler that is gcc compatible.

GNU make requires GNU m4, automake, autoconf, Perl, and a C compiler that is gcc compatible
Perl requires a c89 compiler.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We can probably remove this line since this is no longer the case

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What does it need? That was for a git build I think. It needs Perl for test right?

Comment thread README.md Outdated

Perl requires a c89 compiler
ncurses requires an ANSI C compiler
ncurses requires an ANSI C compiler.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Same with this

Comment thread README.md
First, you need to have some tools installed on your system:

### System Pre-reqs:
### System Prerequisites
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Might be better to just point to zopen-setup here

@AnthonyGiorgio AnthonyGiorgio merged commit 12f585e into main May 22, 2023
@AnthonyGiorgio AnthonyGiorgio deleted the readme-cleanup branch May 22, 2023 12:30
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.

3 participants