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

Added Support for Debian Packages. #89

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

35C4n0r
Copy link

@35C4n0r 35C4n0r commented Jan 10, 2023

Added package registry support for Debian. Closes #49

Changes:

  1. src/fetchcode/init.py: The Response Class has a new success parameter, which will be true if and only if the status code is 200, false otherwise.
  2. src/fetchcode/ls.py: New file picked from minecode https://github.com/nexB/purldb/blob/main/minecode/ls.py, helps with parsing ls-lR data.
  3. src/fetchcode/package.py: Few functions have been moved to src/fetchcode/utils.py. Added support for Debian Packages.
  4. Added new tests.

Signed-off-by: Jay [email protected]

@35C4n0r
Copy link
Author

35C4n0r commented Jan 10, 2023

@AyanSinhaMahapatra
Copy link
Member

@35C4n0r I think you have to add it to https://github.com/nexB/fetchcode/blob/master/setup.cfg#L53 if you're importing it, this is what's used when installing. Requirements is only used for constraints https://github.com/nexB/fetchcode/blob/master/configure#L31

@35C4n0r
Copy link
Author

35C4n0r commented Jan 25, 2023

@AyanSinhaMahapatra any idea why the checks are not running for 33e78dc ?

@keshav-space keshav-space marked this pull request as draft May 1, 2024 09:57
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.

Add package registry support for Debian
2 participants