search

remy / inliner

Node utility to inline images, CSS and JavaScript for a web page - useful for mobile sites
MIT License
1.1k stars 165 forks source link

path.existsSync is now called `fs.existsSync` #19

Closed ff6347 closed 9 years ago

ff6347 commented 11 years ago

Hi there,

I'm trying to use inliner command line tool locally to combine some files. But I get the following error message in the console. 

path.existsSync is now called `fs.existsSync`

So i went into /usr/local/lib/node_modules/inliner/bin/inliner and changed line 65 from: 

if (path.existsSync(url))

to 

if (fs.existsSync(url))

but I get still the same error message. Can you give me a hint what is wrong and how I can fix this?
There is already a question here @ stackoverflow but that didn't fix my problem. Or am I editing the wrong file?
Cheers

ff6347 commented 11 years ago

I got an output by using: 

    inliner -i http://fabiantheblind.info/coding.html > test2.html

seems like the image compression is making problems.
Also I got this warning during installation: 

npm WARN engine commander@0.5.1: wanted: {"node":">= 0.4.x < 0.7.0"} (current: {"node":"v0.8.12","npm":"1.1.68"})