<div dir="ltr"><div>Control: tags -1 patch</div><div><br></div>Hi once more.<div><br></div><div>@Paolo: There is a question for you:<br></div><div>Could you imagine (no pun intended) to include the change in the imagemagick version of the dh_doxygen script into the version in the doxygen package, possibly behind an option? It replaces known (currently only jquery) .js files by a symlink to the relevant known location of the (here:) jquery file and creates a substvar with the required dependency/dependencies.</div><div>This would eventually eliminate the need to keep a script in the imagemagick sources in sync with the doxygen package.</div><div>The replacement is desirable from a security standpoint, as it reduces the places that need patching if another jquery vulnerability surfaces.</div><div><br></div><div>I attached two things to this email: The full updated dh_doxygen script as it would need to be in the imagemagick package (compatible with both older and newer doxygen versions) and the patch to the one currently in the imagemagick package.</div><div><br></div><div>Below are my findings from my investigation:</div><div><br></div><div>So: Looking at the imagemagick-6-doc package, replacing the full debian/scripts/dh_doxygen call (including all parameters) with _just_ "/usr/bin/dh_doxygen" (without any parameters) causes the following differences in the imagemagick-6-doc package:</div><div><br></div><div>(+ from the contents for the newly built package, - for the original package currently in the Debian archive):</div><div><br></div><div>--rw-r--r-- root/root 2020-07-27 03:13 ./usr/share/doc/imagemagick-6-common/html/www/api/MagickCore/doxygen.png</div><div>+-rw-r--r-- root/root 2020-07-27 03:13 ./usr/share/doc/imagemagick-6-common/html/www/api/MagickCore/doxygen.svg<br></div><div>The above seems correct.</div><div><br>-lrwxrwxrwx root/root 2020-07-27 03:13 ./usr/share/doc/imagemagick-6-common/html/www/api/MagickCore/jquery.js -> ../../../../../../javascript/jquery/jquery.js<br>+-rw-r--r-- root/root 2020-07-27 03:13 ./usr/share/doc/imagemagick-6-common/html/www/api/MagickCore/jquery.js<br>-lrwxrwxrwx root/root 2020-07-27 03:13 ./usr/share/doc/imagemagick-6-common/html/www/api/MagickWand/doxygen.png -> ../MagickCore/doxygen.png<br>+lrwxrwxrwx root/root 2020-07-27 03:13 ./usr/share/doc/imagemagick-6-common/html/www/api/MagickWand/doxygen.svg -> ../MagickCore/doxygen.svg<br>-lrwxrwxrwx root/root 2020-07-27 03:13 ./usr/share/doc/imagemagick-6-common/html/www/api/MagickWand/jquery.js -> ../../../../../../javascript/jquery/jquery.js<br></div><div>+lrwxrwxrwx root/root 2020-07-27 03:13 ./usr/share/doc/imagemagick-6-common/html/www/api/MagickWand/jquery.js -> ../MagickCore/jquery.js<br></div><div><br></div><div>There is a problem though: The original dh_doxygen script in the imagemagick source replaces .js files with appropriate symlinks and generates some substvars. Maybe this is functionality that should get added to the original dh_doxygen script? CC'ing Paolo for this, see intro.</div><div><br></div><div>Cheers,</div><div>Sven</div><div><br></div><div>Attached is a patch to the included dh_doxygen </div></div>