Vote #81331
完了Remove dependency on MimeMagic
0%
説明
In Redmine, MimeMagic is used to check that files with invalid formats are not passed to ImageMagick when generating thumbnail images with ImageMagick.
This patch replaces MimeMagic with "Marcel":https://github.com/rails/marcel, a gem bundled with Rails. The benefits of this change are as follows:
- Simplifies the installation process: Due to "a licensing issue":https://github.com/mimemagicrb/mimemagic/issues/97, the MIME database freedesktop.org.xml was removed from MimeMagic. As a result, you have to make sure that your system has shared-mime-info before you install Redmine. Marcel bundles another database "Apache Tika":https://tika.apache.org/, so you don't have to install MIME database separately
- Can reduce memory usage: By replacing MimeMagic with Marcel, we can reduce one gem dependency. Rails used to depend on MimeMagic, but in Rails 5.2.5 it no longer does. Now only Redmine itself depends on MimeMagic
journals
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
Committed the patches.
--------------------------------------------------------------------------------
I think it is better to merge the change to older versions for the following reason. Keep using MimeMagic may cause installation trouble in some environments.
Go MAEDA wrote:
> * Simplifies the installation process: Due to "a licensing issue":https://github.com/mimemagicrb/mimemagic/issues/97, the MIME database freedesktop.org.xml was removed from MimeMagic. As a result, you have to make sure that your system has shared-mime-info before you install Redmine. Marcel bundles another database "Apache Tika":https://tika.apache.org/, so you don't have to install MIME database separately
--------------------------------------------------------------------------------
Backported to 4.1-stable in r20877 and r20878.
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
Backported to 4.0-stable in r20953.
--------------------------------------------------------------------------------
related_issues
relates,Closed,34966,Update Rails to 5.2.5