search

pivotal / LicenseFinder

Find licenses for your project's dependencies.
MIT License
1.73k stars 340 forks source link

LicenseFinder hangs / stuck with Electron and Yarn #1007

Open MassimoDeLuca opened 1 year ago

MassimoDeLuca commented 1 year ago

Hi,

I have a node repository setup with Yarn and have various packages installed.

Everything works as expected with Electron versions 23, 24, 25 although when updated to version 26 or 27 LicenseFinder will hang indefinitely. It doesn't look like Electron has modified their license in 3 years and I'm unsure to the cause.

Command I'm running: license_finder report --format csv --columns=name version licenses homepage package_manager --save licenses.csv Output:

LicenseFinder::NPM: is active
LicenseFinder::Yarn: is active
**hangs**

With the --debug arg:

image

This also occurs with some of the basic commands in the Readme.

Running yarn licenses list outputs fine, even when using the --recursive --json like LicenseFinder does (AFAIK). Tried both latest LicenseFinder, and one of the prior releases to no avail. Also tried both Ruby 2.7.4 and 3.0.2

LMK if theres anymore info I can provide to help debug!

cf-gitbot commented 1 year ago

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

MassimoDeLuca commented 1 year ago

@xtreme-shane-lattanzio Is this repo still maintained? Noticing a handful of issues in the past few weeks haven't been looked at.

xtreme-shane-lattanzio commented 10 months ago

@MassimoDeLuca This repo is very lightly maintained at the moment as our priorities have shifted. I try to get to things when I have some time which is not as often nowadays.

As for the issue, I honestly have no idea what Electron even is so I am not even sure how to reproduce this or what to change about it but if you manage to get a PR up for this for us to look at, I'd be happy to merge in a fix