-
Notifications
You must be signed in to change notification settings - Fork 353
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
Remove indirect dependencies #4873
Comments
cryptography should be no longer there, idna and cffi we would need to check. |
log2timeline/plaso/log2timeline#4873 WIP Bug: log2timeline#4873 Reported-by: Kalin KOZHUHAROV <[email protected]> Signed-off-by: Kalin KOZHUHAROV <[email protected]>
log2timeline/plaso/log2timeline#4873 WIP Bug: log2timeline#4873 Reported-by: Kalin KOZHUHAROV <[email protected]> Signed-off-by: Kalin KOZHUHAROV <[email protected]>
log2timeline/plaso/log2timeline#4873 WIP Bug: log2timeline#4873 Reported-by: Kalin KOZHUHAROV <[email protected]> Signed-off-by: Kalin KOZHUHAROV <[email protected]>
It looks like it is relying on a hard-coded output that just happens to have those module names? log2timeline/plaso/log2timeline#4873 WIP Bug: log2timeline#4873 Reported-by: Kalin KOZHUHAROV <[email protected]> Signed-off-by: Kalin KOZHUHAROV <[email protected]>
looks like idna was a requirement of cryptography and requests, cryptography is no longer used, but let me check requests |
Looks like idna still is a dependency of requests https://github.com/psf/requests/blob/main/setup.cfg#L6 . I would need to look why it was explicitly added instead of it being an implicit dependency. I recommend we keep that one for now. |
And cffi is a dependency of xattr https://github.com/xattr/xattr/blob/main/pyproject.toml#L15 |
I would need to assess first, the reason for these indirect dependencies were added. Likely due to one of the packaging/environments failing to include them. |
Looks like cffi was added for cryptography but this predates the inclusion of xattr 221a298 |
Idna was explicitly added for requests 8871cd9 |
I'll work on this a bit more, but here are my thoughts:
|
Unfortunately the "should" here is one of those theory versus reality arguments. They were added because they were needed.
tests can always be improved |
I am digging around and so far I don't see any direct use of
(Disclaimer: The above two ways are the only standard ways to use a Python module, AFAIK) While |
Python has many "standard" ways of using modules. I'll have a closer looks when time permits. Maybe this was msi or Pyinstaller related. |
While working on packaging latest
plaso
for Gentoo, I think I see two python modules (idna
,cffi
) which are not referenced in any code, yet they are installed by some of the scripts. I suggest to have them removed fromdependencies.ini
andrequirements.txt
.Those are only indirect dependencies as follows:
This concerns
plaso-20240409
on Gentoo installed from my pkalin overlay, as there is no officialplaso
support on Gentoo yet.NOTE: Description was edited as follows:
The text was updated successfully, but these errors were encountered: