search

CoderDojo-Content / sushi-tool

CoderDojo Sushi Cards tool.
MIT License
3 stars 2 forks source link

sushi-tool uses deprecated dependencies / tools using deprecated deps #2

Open sandrobonazzola opened 8 years ago

sandrobonazzola commented 8 years ago 
+ npm install sushi-tool
npm WARN deprecated pdfconcat@1.0.2: The development of this module has been discontinued
npm WARN optional dep failed, continuing fsevents@1.0.14
[cut]
npm WARN deprecated minimatch@0.3.0: Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated npmconf@2.1.2: this package has been reintegrated into npm and is now out of date with respect to npm
npm WARN deprecated cross-spawn-async@2.2.4: cross-spawn no longer requires a build toolchain, use it instead!
npm WARN deprecated minimatch@0.2.14: Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated lodash@1.0.2: lodash@<3.0.0 is no longer maintained. Upgrade to lodash@^4.0.0.
[cut]
npm WARN deprecated graceful-fs@1.2.3: graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to graceful-fs@^4.0.0 as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.

please check if new versions are available or different tools can be used, avoiding to rely on deprecated libraries / tools.

diegogd commented 8 years ago

Sandro thank you for your advice, I have to make some clean up of the dependencies.

By now there is a hard dependency on the use of pdfconcat to generate a single pdf at the end. If you have any other idea about other tool let me know.

sandrobonazzola commented 8 years ago

@diegogd there's pdf-merge https://www.npmjs.com/package/pdf-merge which has 1.103 downloads in the last month compared to the 52 of pdfconcat. Looking at the 2 libraries, pdf-merge is a wrapper for pdfTK which is unchanged since 2013 while pdfconcat is a wrapper for poppler, last version released 2 days ago. The pdfconcat author deprecated the module probably because he's not maintaining anymore the code to align with newer versions of poppler. I think you can use pdf-merge, looks stable enough and still maintained.

diegogd commented 8 years ago

pdf-merge was my first option, I didn't do this deep analysis but I was having problems to installa pdftk in mac, that was the reason I ended up using pdfconcat. I've seen some alternative ways of installing pdftk through brew in mac, so we could consider moving this implementation to use the other library. Thanks again.