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

Remove bogus compile instruction in Linux/README #96

Closed
wants to merge 1 commit into from
Closed

Remove bogus compile instruction in Linux/README #96

wants to merge 1 commit into from

Conversation

artagnon
Copy link

There is no directory called libusb-1.0 in the source tree, and the
first instruction already asks us to get the distribution's version of
libusb.

There is no directory called libusb-1.0 in the source tree, and the
first instruction already asks us to get the distribution's version of
libusb.
@Benjamin-Dobell
Copy link
Owner

Looks like I stuffed up a merge somewhere. The fix you've reported isn't quite what I originally intended, but I'll sort it out. Thanks for letting me know!

@jwmh
Copy link

jwmh commented Feb 3, 2013

Also, the instructions in the Linux README differ from the one on this page:
https://github.com/Benjamin-Dobell/Heimdall/wiki/Bulding-from-Source

Those instructions say to run:
$ ./autogen.sh
$ ./configure
$ make

(whereas the Linux README doesn't mention running ./autogen.sh )

@Benjamin-Dobell
Copy link
Owner

Wow, I didn't even realise anyone had written anything on the wiki!

Running:

./autogen.sh

is unnecessary as I run autogen.sh before I push to Github. Running it again won't cause any harm as long as the necessary tools are installed on your system. However, to avoid confusion I've updated the wiki and removed the line.

@Benjamin-Dobell
Copy link
Owner

I believe this is addressed in the 1.4.0 release (https://github.com/Benjamin-Dobell/Heimdall/tree/v1.4.0).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants