Ticket #20769 (closed defect: fixed)
mcrypt, config.inc.php issues in phpmyadmin
| Reported by: | ora.et.labora@… | Owned by: | alakazam@… |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 1.7.1 |
| Keywords: | phpmyadmin | Cc: | |
| Port: | phpmyadmin |
Description (last modified by ryandesign@…) (diff)
There are (at least) two issues whith phpmyadmin @3.2.0.1
- Extension mcrypt needs to be installed/enabled in PHP
- File config.inc.php is not generated.
I would expect that at least a warning message is shown during the installation that one needs to make sure that 'mcrypt' is enabled and how to do that (port install php5-mcryp for example).
The second issue about that the blowfish secret needs to be set and each and every offical documentation tells us to do so by editing config.inc.php. This file would be created during an installation either manually or automatically. This port really sucks in not following the official way - this file does not exist. If this port decides to follow an alternative route, then for heavens sake, please document this proper!!
Change History
comment:4 in reply to: ↑ description ; follow-up: ↓ 5 Changed 4 years ago by ryandesign@…
Replying to ora.et.labora@…:
I would expect that at least a warning message is shown during the installation that one needs to make sure that 'mcrypt' is enabled and how to do that (port install php5-mcryp for example).
The phpmyadmin port should simply declare a runtime dependency on the php5-mcrypt port, and the ports for any other modules it requires. Versions of the php5 port prior to 5.3.0_1 had mcrypt support always enabled, but it was recently moved into a separate module port along with many other modules. See #19091.
The second issue about that the blowfish secret needs to be set and each and every offical documentation tells us to do so by editing config.inc.php. This file would be created during an installation either manually or automatically. This port really sucks in not following the official way - this file does not exist. If this port decides to follow an alternative route, then for heavens sake, please document this proper!!
The port does nothing unusual. It simply copies the phpmyadmin files into the right place. It is the user's responsibility to set it up correctly, which includes creating and editing the config.inc.php. It wouldn't hurt if the port printed a message explaining to the user that this file must be created (if the user has not already done so).
comment:5 in reply to: ↑ 4 Changed 4 years ago by ryandesign@…
Replying to ryandesign@…:
The phpmyadmin port should simply declare a runtime dependency on the php5-mcrypt port, and the ports for any other modules it requires.
The patch attached to #20818 addresses this.