Bug 23762 - php-pecl-mcrypt should be named php-mcrypt
Summary: php-pecl-mcrypt should be named php-mcrypt
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Marc Krämer
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-10-26 18:35 CEST by David Walser
Modified: 2018-10-26 21:18 CEST (History)
0 users

See Also:
Source RPM: php-pecl-mcrypt-1.0.1-5.mga7.src.rpm
CVE:
Status comment:


Attachments

Description David Walser 2018-10-26 18:35:38 CEST
Marc just renamed the php-mcrypt package.  We need to keep consistency with our naming conventions, as being inconsistent makes it harder to determine if we have a package when looking at security issues.  The pecl thing is the Fedora convention, not ours.
Comment 1 Marc Krämer 2018-10-26 18:49:23 CEST
You're right. Corrected this issue in cauldron and mga6 backports.

Thanks.

Resolution: (none) => FIXED
Status: NEW => RESOLVED

Comment 2 Marc Krämer 2018-10-26 18:51:48 CEST
grr. Have to go eating, I have to rethink this thing. I might have to increase the epoch.

Resolution: FIXED => (none)
Status: RESOLVED => REOPENED

Comment 3 David Walser 2018-10-26 18:54:22 CEST
You shouldn't need an epoch.  In fact you shouldn't need to do anything special (other than temporarily obsoleting php-pecl-mcrypt to remove it from the repo, but you shouldn't need to keep the obsoletes in after that since it never shipped with that name in a stable release).
Comment 4 Marc Krämer 2018-10-26 19:45:23 CEST
nope, thats not the point.
php-mcrypt was a former internal package, which was built together with php, so the version is 5.6 in mga6. It is not officially supported anymore and was moved to pecl. Now the version is 1.0.1, so the version number is lower than before. And if I'm not missing anything, this is the reason epochs are made for.
Comment 5 David Walser 2018-10-26 20:06:26 CEST
Oh, I missed the part where it used to be an internal package, sorry.  I just saw the failed build attempt for php-mcrypt and forgot.  You're absolutely right.  Epoch is the way to go.
Comment 6 Marc Krämer 2018-10-26 21:18:01 CEST
thanks for your advice.

Status: REOPENED => RESOLVED
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.