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

[wiki] Install information confusing / out-of-date regarding Mcrypt (I think) #5780

Closed
seanm opened this Issue Jun 1, 2017 · 2 comments

Comments

Projects
None yet
2 participants
@seanm
Contributor

seanm commented Jun 1, 2017

I'm not sure, but some searching seems to suggest that Mcrypt is no longer used by current roundcube (1.2.5).

The install requirements don't mention it:
https://github.com/roundcube/roundcubemail/wiki/Install-Requirements

but the wiki install instructions do:
https://github.com/roundcube/roundcubemail/wiki/Installation

They say to install php5-mcrypt.

Mcrypt is of course obsolete, unmaintained, and deprecated:
http://php.net/manual/en/intro.mcrypt.php

So hopefully roundcube doesn't actually use it, and if so there's a few lingering places to un-document it.

@alecpl

This comment has been minimized.

Show comment
Hide comment
@alecpl

alecpl Jun 1, 2017

Member

Roundcube >= 1.2 does not use it, indeed. These install scripts in the wiki are outdated. I'd remove them.

Member

alecpl commented Jun 1, 2017

Roundcube >= 1.2 does not use it, indeed. These install scripts in the wiki are outdated. I'd remove them.

@alecpl alecpl changed the title from Install information confusing / out-of-date regarding Mcrypt (I think) to [wiki] Install information confusing / out-of-date regarding Mcrypt (I think) Jun 1, 2017

@alecpl alecpl added this to the 1.3.1 milestone Jun 14, 2017

@alecpl

This comment has been minimized.

Show comment
Hide comment
@alecpl

alecpl Jun 28, 2017

Member

Fixed wiki page.

Member

alecpl commented Jun 28, 2017

Fixed wiki page.

@alecpl alecpl closed this Jun 28, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment