kevva / exec-buffer

Run a Buffer through a child process

exec-buffer

Run a Buffer through a child process

Install

$ npm install exec-buffer

Usage

const fs = require('fs');
const execBuffer = require('exec-buffer');
const gifsicle = require('gifsicle').path;

execBuffer({
	input: fs.readFileSync('test.gif'),
	bin: gifsicle,
	args: ['-o', execBuffer.output, execBuffer.input]
}).then(data => {
	console.log(data);
	//=> <Buffer 47 49 46 38 37 61 ...>
});

API

execBuffer(options)

options

Type: Object

input

Type: Buffer

The Buffer to be ran through the child process.

bin

Type: string

Path to the binary.

args

Type: Array

Arguments to run the binary with.

inputPath

Type: string
Default: tempfile()

Where input will be written to. In most cases you don't need to set this.

outputPath

Type: string
Default: tempfile()

Where output file will be written to. In most cases you don't need to set this.

execBuffer.input

Returns a temporary path to where the input file will be written.

execBuffer.output

Returns a temporary path to where the output file will be written.

License

MIT © Kevin Mårtensson

exec-buffer's Issues

please update depencencies avoid error on build

I try to use imagemin and imagemin-webp on wepback project.
And then still trying to do something good, i get error about dependency exec-buffer. It's give me an error:

This dependency was not found:

* child_process in ./node_modules/exec-buffer/node_modules/cross-spawn/index.js, ./node_modules/exec-buffer/node_modules/execa/index.js 
and 1 other

Try to manage it my self but at the end it seems to be a dependencies error of depends packages of imagemin....

node v12.16.1
npm v6.14.4
yarn v1.22.4
windows 10

stderr output does not imply failure

this module, when used by imagemin-pngcrush, causes issues when pngcrush outputs warnings to stderr.

This module should be assuming success if the command finishes with a zero exit status. stderr is often used for debug, warning, or logging purposes when stdout can be used as the output of a process.

I can write a test and PR if this is something that will be merged quickly.

The tempfile dependency is outdated and show a deprecation message when installing dependencies

npm WARN deprecated [email protected]: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.

[email protected] -> [email protected] -> [email protected]

Concatenating file names with parameters

@kevva Nice library, thank you.

Trying to run jpegmini with it. The syntax I need:

jpegmini -f=input_file -o=outputfile

I tried:

• ['-f', execBuffer.input, '-o', execBuffer.output]
• ['-f=', execBuffer.input, '-o=', execBuffer.output]
• ['-f="', execBuffer.input, '" -o="', execBuffer.output, '"']
• ['"-f=', execBuffer.input, '" "-o=', execBuffer.output, '"']

None of the above works with jpegmini.

Given that execBuffer.input and execBuffer.output are Symbols, I can't just use args: ['-f=' + execBuffer.input, '-o=' + execBuffer.output]

How about adding execBuffer.inputFile() and execBuffer.outputFile() that would return strings (e.g. using random uuid) that would later be replaced with actual filenames in all parameters?

Or maybe you have some better idea how to address it... I can do PR.