Better error message when vlfeat not found

[AbdealiLoKo]

Currently if I try to install cyvlfeat without vlfeat installed, I get:

cyvlfeat/sift/cysift.c:280:21: fatal error: vl/host.h: No such file or directory
  compilation terminated.

This makes sense, because vl/host.h cannot be found, as vlfeat is not installed.
But, it would be nicer to mention something like "Could not find vlfeat http://www.vlfeat.org/. Please install it and then install cyvlfeat"

NOTE: This is again, when I use this with pip. The readme says vlfeat is auto installed in conda (I don't normally use conda so have not tried with conda)

You can see the whole error message at https://s3.amazonaws.com/archive.travis-ci.org/jobs/132164694/log.txt

[patricksnape]

Yes this is currently expected behaviour as pip installing is not recommended not fully supported at the moment as you must satisfy the vlfeat dependency yourself. This is essentially the same problem as #14. This can be fixed but requires some work on the setup.py as detailed in #14.

[AbdealiLoKo]

What I was thinking when creating this issue was that a better error
message should be given. I understand that vlfeat cannot be installed with
pypi.
But something like saying "Hey there, please install vlfeat and try again"
instead of the error message it gives right now

On Mon, May 23, 2016 at 7:31 PM, Patrick Snape notifications@github.com
wrote:

Yes this is currently expected behaviour as pip installing is not
recommended not fully supported at the moment as you must satisfy the
vlfeat dependency yourself. This is essentially the same problem as #14
#14.

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#15 (comment)

[AbdealiLoKo]

I thought they were separate issues (As this is simply giving a better
error message) and this would be simpler. But if this gets solved by #14
#14, then please do close this :)

On Mon, May 23, 2016 at 7:39 PM, Abdeali Kothari abdealikothari@gmail.com
wrote:

What I was thinking when creating this issue was that a better error
message should be given. I understand that vlfeat cannot be installed with
pypi.
But something like saying "Hey there, please install vlfeat and try again"
instead of the error message it gives right now

On Mon, May 23, 2016 at 7:31 PM, Patrick Snape notifications@github.com
wrote:

Yes this is currently expected behaviour as pip installing is not
recommended not fully supported at the moment as you must satisfy the
vlfeat dependency yourself. This is essentially the same problem as #14
#14.

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#15 (comment)

[patricksnape]

We could sort this separately but it would certainly be part of #14.