Skip to content
This repository has been archived by the owner on Dec 3, 2019. It is now read-only.

Populate data base from upstream data #52

Open
shivanshuraj1333 opened this issue Mar 30, 2019 · 4 comments · May be fixed by #53
Open

Populate data base from upstream data #52

shivanshuraj1333 opened this issue Mar 30, 2019 · 4 comments · May be fixed by #53

Comments

@shivanshuraj1333
Copy link
Contributor

As mentioned by @zvr in his comment on Pull Request #48, a functionality is needed through which database can be populated without using .csv file (large if we want to solve #47 ).
Note: In this method internet connection is required to populate the database, advantage of merging PR #48 is that anyone can populate database without using internet.
@shivanshuraj1333 shivanshuraj1333 changed the title Populate data base without using .csv file(which is very large in case of complete license text) Populate data base without using .csv file (very large in case of complete license text) Mar 30, 2019
@zvr zvr changed the title Populate data base without using .csv file (very large in case of complete license text) Populate data base from upstream data Mar 30, 2019
@zvr
Copy link
Collaborator

zvr commented Mar 30, 2019

The issue with the csv (besides the size) is that this is not original data, so it has to be created somehow.
It would be better to use the data as published in https://github.com/spdx/license-list-data

@shivanshuraj1333
Copy link
Contributor Author

shivanshuraj1333 commented Mar 30, 2019 •
edited
Loading

The new csv file data in #48 is created from https://github.com/spdx/license-list-data. (Also contain license text)
Please review the updated commit on Pull Request #48

@zvr
Copy link
Collaborator

zvr commented Mar 30, 2019

If it was created, we need the software that created it, for review. But we shouldn't be transforming the data in other formats...

[and please, stop requesting actions by referencing me and adding comments; I see all activity and all it will be handled at some point]

@shivanshuraj1333
Copy link
Contributor Author

See this for script.

[and please, stop requesting actions by referencing me and adding comments; I see all activity and all it will be handled at some point]

Sorry, I will keep this in mind.

This was referenced Mar 31, 2019
Open
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Modified populate database method to directly fetch upstream data. shivanshuraj1333/clio
2 participants
@zvr @shivanshuraj1333