wubzz / pdf-merge

Merge multiple PDF Files into a single PDF document
127 stars 32 forks source link

process.platform === 'Darwin' #2

Closed zimt28 closed 9 years ago

zimt28 commented 9 years ago

pdf-merge doesn't work on Mac, as the platform Darwin contains the string win. pdf-merge will ask for an absolute path for PDFtk.

zimt28 commented 9 years ago

Making sure process.platform/ os.platform() isn't Darwin should be enough (here).

wubzz commented 9 years ago

Thanks for reporting this. I've opted to go by os.type() instead, since the issue you described would also occur on other platforms.