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

[test] Get rid of mockldap #3894

Merged
merged 1 commit into from
Apr 27, 2023
Merged

Conversation

bruntib
Copy link
Contributor

@bruntib bruntib commented Apr 26, 2023

Mockldap dependency is not supported for a long time. It can't be
installed to the latest Ubuntu versions, due to some of its old version
implicit dependencies (funcparserlib). For this reason the usage of
mockldap module has been eliminated and a simplified mock class has been
introduced instead with hardcoded return values.

This error could be reproduced by "make venv_dev" command which emitted
this error message:
"error in funcparserlib setup command: use_2to3 is invalid."

Fixes #3892

@bruntib bruntib requested a review from Szelethus April 26, 2023 12:00
@bruntib bruntib requested a review from vodorok as a code owner April 26, 2023 12:00
@Szelethus Szelethus added the dependencies 📦 Pull requests that update a dependency file label Apr 27, 2023
@Szelethus Szelethus added this to the release 6.23.0 milestone Apr 27, 2023
Copy link
Collaborator

@Szelethus Szelethus left a comment

Choose a reason for hiding this comment

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

Please link this issue in the summary:
#3892

Folks that try to build CodeChecker and encounter the error, will google this:

"error in funcparserlib setup command: use_2to3 is invalid."

It would make sense to copy-paste this specific error message into the summary as well, so they would find this PR and know to simply upgrade their version.

Mockldap dependency is not supported for a long time. It can't be
installed to the latest Ubuntu versions, due to some of its old version
implicit dependencies (funcparserlib). For this reason the usage of
mockldap module has been eliminated and a simplified mock class has been
introduced instead with hardcoded return values.

This error could be reproduced by "make venv_dev" command which emitted
this error message:
"error in funcparserlib setup command: use_2to3 is invalid."

Fixes Ericsson#3892
Copy link
Collaborator

@Szelethus Szelethus left a comment

Choose a reason for hiding this comment

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

LGTM

@bruntib bruntib merged commit 1f3ea94 into Ericsson:master Apr 27, 2023
13 checks passed
@bruntib bruntib deleted the eliminate_mockldap branch April 27, 2023 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies 📦 Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Actions fail due outdated funcparserlib
2 participants