kevva / kevva/bin-wrapper

dest() affects the success of run() finding a binary

Open
#72 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
152
Forks
65
PR merge metrics
No merged PRs in 30d

Description

dest()

Accepts a path which the files will be downloaded to.

run()

Runs the search for the binary. If no binary is found it will download the file using the URL provided in .src().

Oddly, the path given to dest() appears to affect the success of run()? I was trying to figure out why bin-wrapper was not working in the imagemin-cwebp/cwebp-bin package, cwebp -version works just like they test for, which cwebp points to /usr/bin, but when running the package install.js which uses bin-wrapper it fails to run the command.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Reproduce the failure through the install.js used by imagemin-cwebp/cwebp-bin, comparing bin-wrapper behavior with and without dest() while checking the cwebp command and the path from which it is found. Read the dest() and run() entry points to identify why the destination changes binary detection. Done means the package installation finds and runs the available cwebp binary regardless of the destination path, with a regression test for the reported case.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
tooling
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.