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

Support yaml format of FOSSLight Report #42

Merged
merged 3 commits into from
Aug 16, 2022
Merged

Support yaml format of FOSSLight Report #42

merged 3 commits into from
Aug 16, 2022

Conversation

dd-jy
Copy link
Contributor

@dd-jy dd-jy commented Jul 22, 2022

Signed-off-by: Jiyeong Seok jiyeong.seok@lge.com

Description

Support yaml format of FOSSLight Report

Type of change

Please insert 'x' one of the type of change.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation update
  • Refactoring, Maintenance
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

@dd-jy dd-jy added the enhancement [PR/Issue] New feature or request label Jul 22, 2022
@dd-jy dd-jy requested a review from soimkim July 22, 2022 05:36
@dd-jy dd-jy self-assigned this Jul 22, 2022
@soimkim
Copy link
Contributor

soimkim commented Jul 22, 2022

  1. If FL_Dependency.yaml is not created because there is nothing to analyze dependency, an error occurs.
Tool Info : fosslight_dependency v3.11.2
It cannot find the manifest file.
Dependency scanning terminated because the package manager was not found.
FL_Source.yaml
FL_Binary.yaml
FL_Dependency.yaml
Fail to generate a result file(/home/worker/git/fosslight_binary_scanner/result/fosslight-sbom-info_20220722_171837.yaml): [Errno 2] No such file or directory: '/home/worker/git/fosslight_binary_scanner/result/fosslight_raw_data_20220722_171837/FL_Dependency.yaml'
OS: Linux 5.4.0-59-generic
Python version: 3
Tool Info: fosslight_scanner v1.7.0
  1. The FL_Source.yaml, FL_Binary.yaml,FL_Dependency.yaml are output in red even though it is not an error.
    image
    • The output file is output to the log as "Output File:", so the above files should not be output.

Signed-off-by: Jiyeong Seok <jiyeong.seok@lge.com>
@dd-jy
Copy link
Contributor Author

dd-jy commented Jul 28, 2022

@soimkim
I fixed all! Please review it again.

and add compared result summary log

Signed-off-by: Jiyeong Seok <jiyeong.seok@lge.com>
@dd-jy dd-jy merged commit 3eb1b51 into main Aug 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement [PR/Issue] New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants