aep / pug-rs

pug.js reimplemented in rust for speed
27 stars 7 forks source link

Installation as cargo dependency fails #4

Closed iredelmeier closed 5 years ago

iredelmeier commented 5 years ago

Hi there!

It seems like installing the crate as a cargo dependency fails. I believe that this has something to do with the crate being available as a cydlib, although I'm not sure exactly. Oddly, cloning the repo and adding pug-rs as a path-based dependency doesn't cause the same problem - the rlib is correctly found then.

(This may instead be an upstream cargo and/or crates.io bug.)

aep commented 5 years ago

huh thats strange. whats the error message?

iredelmeier commented 5 years ago

Whoops, should've included that before...

error[E0463]: can't find crate for `pug`
 --> src/lib.rs:4:1
  |
4 | extern crate pug;
  | ^^^^^^^^^^^^^^^^^ can't find crate

As an experiment, I published a few different versions from my fork and they all worked as expected (i.e., they were usable as regular dependencies).

These are the things I tried successfully:

I'm wondering if there was a bug somewhere (e.g., crates.io) that's since been fixed upstream?

aep commented 5 years ago

huh that's strange. i can't reproduce it. but i also published a new version with your other changes, according to your analysis there is a chance it just works now

iredelmeier commented 5 years ago

Looks like the new version works correctly. Thanks!

Who knows what was up with the old one :)