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

Add support for gzip-compressed database #23

Merged
merged 1 commit into from
Jan 22, 2021

Conversation

pearsonk
Copy link

For systems that store the database file in compressed form, directly
parse this instead of requiring an implementer to separately uncompress
to disk.

Signed-off-by: Kevin Pearson kevin.pearson@ortmanconsulting.com

For systems that store the database file in compressed form, directly
parse this instead of requiring an implementer to separately uncompress
to disk.

Signed-off-by: Kevin Pearson <kevin.pearson@ortmanconsulting.com>
@pearsonk
Copy link
Author

I work on a system OpenXT, based on OpenEmbedded Linux, that stores the pci.ids in compressed form. It is also stored in a location separate from the convention followed here, but I will submit a separate PR to add an option for specifying a direct path at the head of searchPaths

Copy link
Owner

@jaypipes jaypipes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @pearsonk this looks awesome, thanks very much for the patch! You will need to rebase, though, since I merged a quick err-catching fix from @martinkennelly this morning.

Thanks again!
-jay

@jaypipes
Copy link
Owner

Hi @pearsonk this looks awesome, thanks very much for the patch! You will need to rebase, though, since I merged a quick err-catching fix from @martinkennelly this morning.

Thanks again!
-jay

Never mind, looks like this is conflict-free. Merging...

@jaypipes jaypipes merged commit abb0521 into jaypipes:master Jan 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants