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

Fix build instructions #383

Merged
merged 5 commits into from
May 19, 2024

Conversation

Weltraumschaf
Copy link
Contributor

The documented way with go get gives an error message:

go get github.com/zmap/zgrab2
go: go.mod file not found in current directory or any parent directory.
'go get' is no longer supported outside a module.
To build and install a command, use 'go install' with a version,
like 'go install example.com/cmd@latest'
For more information, see https://golang.org/doc/go-get-install-deprecation
or run 'go help get' or 'go help install'.

I'm not very experienced with go. Worked works for me: Clone repo and run make.

If you use the copy & paste feature from GH's rendered README.md this
$ sign is also copied and needs to be removed when pasted into a shell.

Removing the $ makes it possible to paste the command w/o manual
edit.

Signed-off-by: Sven Strittmatter <[email protected]>
The documented way to build seems to be outdated:

```
go get github.com/zmap/zgrab2
go: go.mod file not found in current directory or any parent directory.
        'go get' is no longer supported outside a module.
        To build and install a command, use 'go install' with a version,
        like 'go install example.com/cmd@latest'
        For more information, see https://golang.org/doc/go-get-install-deprecation
        or run 'go help get' or 'go help install'.
```

What works for me: Clone the repo and simply run `make` inside the repo
to build the binary.

Signed-off-by: Sven Strittmatter <[email protected]>
@Weltraumschaf
Copy link
Contributor Author

Hi, what's up?

@developStorm developStorm merged commit e6c519f into zmap:master May 19, 2024
1 check passed
@developStorm
Copy link
Member

Sorry for the delay. Thanks for helping us with the documentation!

@developStorm developStorm self-requested a review May 19, 2024 23:56
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