Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LicenseFinder hangs / stuck with Electron and Yarn #1007

Open
MassimoDeLuca opened this issue Oct 10, 2023 · 3 comments
Open

LicenseFinder hangs / stuck with Electron and Yarn #1007

MassimoDeLuca opened this issue Oct 10, 2023 · 3 comments

Comments

@MassimoDeLuca
Copy link

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
Copy link
Collaborator

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
Copy link
Author

MassimoDeLuca commented Oct 30, 2023

@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
Copy link
Contributor

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants