Last modified: 2010-05-15 16:03:06 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T18661, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 16661 - 1.13.3 Patch is Missing IEContentAnalyzer.php
1.13.3 Patch is Missing IEContentAnalyzer.php
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Uploading (Other open bugs)
1.13.x
All All
: Normal critical (vote)
: ---
Assigned To: Tim Starling
:
: 17013 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-12-16 01:51 UTC by Joel Sherrill
Modified: 2010-05-15 16:03 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Joel Sherrill 2008-12-16 01:51:21 UTC
The patch was cut using diff -ur and you should have added a -N to pick up the added file.
Applying the patch to an existing installation breaks Image Uploads.

This is the line in the patch file.

Only in mediawiki-1.13.3/includes: IEContentAnalyzer.php

This is the error message in our httpd/error_log

[Mon Dec 15 19:38:37 2008] [error] [client 216.186.145.118] PHP Fatal error:  AutoLoader::require() [<a href='function.require'>function.require</a>]: Failed opening required '/home/johne/public_html/wiki/includes/IEContentAnalyzer.php' (include_path='/home/johne/public_html/wiki:/home/johne/public_html/wiki/includes:/home/johne/public_html/wiki/languages:.:/usr/share/pear:/usr/share/php') in /home/johne/public_html/wiki/includes/AutoLoader.php on line 508, referer: http://www.elviscostello.info/wiki/index.php/Special:Upload

You need to at least replace the patch file.  

I really feel your pain.  I am the RTEMS maintainer and I always worry about making a mistake cutting a release.
Comment 1 Tim Starling 2008-12-16 03:03:46 UTC
I'm just glad I left the checksums out of the email this time around. Patches (and signatures) updated.
Comment 2 Joel Sherrill 2008-12-16 12:40:05 UTC
Maybe you are doing more than I think but if you only update the patch, doesn't that mean that anyone who installed the patch during the 12 or so hours while it did not include this file will have broken uploads?

Anyone impacted will not be likely to reread the release notes or reinstall the patch. How about putting this file beside the new patch on the site and make an announcement that the original patch was missing this file and if you don't have it in your include directory, grab it from the ftp site and install it.
Comment 3 Chad H. 2009-02-07 05:15:18 UTC
*** Bug 17013 has been marked as a duplicate of this bug. ***

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


Navigation
Links