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.

Sign up for GitHub

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

Jump to bottom

Update README.md #544

Merged
merged 1 commit into from
Jun 22, 2021
Merged

Update README.md #544

merged 1 commit into from
Jun 22, 2021

Conversation

evrial
Copy link
Contributor

@evrial evrial commented May 2, 2021

No description provided.

@albarrentine albarrentine merged commit 273d0e0 into openvenues:master Jun 22, 2021
@missinglink
Copy link
Contributor

heya, any idea why this line was added? I'm about to remove it because it's causing issues for users.

the problem with it is that the Makefile isn't available at this stage since configure hasn't been run yet so it fails.

it's also not clear to me why a distclean is required after a fresh clone of the repo?

@evrial
Copy link
Contributor Author

evrial commented Oct 30, 2022

yea that's my mistake I didn't know that distclean isn't available at fresh install.
The line was added because many people faced the issue with lots of comments of incorrect installation or wrong datadir.
I think it's reasonable to add a comment to skip it on fresh install

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@evrial @missinglink @albarrentine