Fix issue #38

[laxa]

trafficstars

This small commit fixes issue #38 when installing current sources.

[c3c]

Thanks for the PR! Before I accept, we should add a minimum version requirement for bloodhound in the setup.py, do you want to add that?

---

From: Julien EGLOFF @.> Sent: Monday, December 18, 2023 6:19:27 PM To: c3c/ADExplorerSnapshot.py @.> Cc: Subscribed @.***> Subject: [c3c/ADExplorerSnapshot.py] Fix issue #38 (PR #39)

This small commit fixes issue #38https://github.com/c3c/ADExplorerSnapshot.py/issues/38 when installing current sources.

---

You can view, comment on, or merge this pull request online at:

https://github.com/c3c/ADExplorerSnapshot.py/pull/39

Commit Summary

• 7313979https://github.com/c3c/ADExplorerSnapshot.py/pull/39/commits/7313979619ea386d69f470eb8b1508ef42d611b7 Fix issue #38

File Changes

(1 filehttps://github.com/c3c/ADExplorerSnapshot.py/pull/39/files)

• M adexpsnapshot/init.pyhttps://github.com/c3c/ADExplorerSnapshot.py/pull/39/files#diff-921044e0048a35f62ca97595f22e7cbdd6c1ce05b481b618f024c4bdac1cf32f (2)

Patch Links:

• https://github.com/c3c/ADExplorerSnapshot.py/pull/39.patch
• https://github.com/c3c/ADExplorerSnapshot.py/pull/39.diff

— Reply to this email directly, view it on GitHubhttps://github.com/c3c/ADExplorerSnapshot.py/pull/39, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AARYDIO25AA2WESOXQHV5ZTYKB3J7AVCNFSM6AAAAABAZ3WFDCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA2DOMJRGAZTENY. You are receiving this because you are subscribed to this thread.Message ID: @.***>

[laxa]

I updated the PR to require at least the 1.7.0 version where this change was introduced in bloodhound.py.

[dirkjanm]

note that this PR is no longer needed since I've removed the extra parameter which broke this in bloodhound.py 1.7.2

[laxa]

I updated the PR to only require version 1.7.2 then.

[c3c]

Gracias :)