User "Hashar" posted a comment on MediaWiki.r101744. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/101744#c25423 Commit summary:
[JSTesting] partial rewrite of TestSwarmMWFetcher * Before rewrite (should've been a separate commit) -- Removing savedPath/chdir code (unused). We probably shouldn't be changing paths. If need to execute commands on paths outside our current working directory we can provide relative or absolute paths to those functions. -- Removing getFirstRev, using minRev directly. -- Fix bug in __construct (using || instead of && to verify required options) -- Fix bug in getNextFollowingRevId, distinguish between invalid id and no id at all. Return null if there is no next id (given id is >= HEAD) -- Fix bug in log(); Call fclose() on the handle, not the file path. Pass handle as first argument to fwrite(), instead of file path as second argument. * During/after ewrite: -- Creating separate class for the install process so we have a context for stuff like paths and revision ids (instead of passing around everything all the time). -- Removing getLogFile(). Instead putting the full path directly in getPaths() -- Changing log path from /log/r123/debug.log to /logs/r123.log -- Renaming variables and functions to make a clearer distinction between the local checkout and the remote repo (i.e. "getNextRev") -- Changing order of functions. Defining functions in order of usage (and before they are used - where possible). * Follows-up r101636 -- Right now it successfully does the checkout and installing of the wiki. @todo: - Append settings template to LocalSettings - Append require() for global settings to LocalSettings - TestSwarm job submitter (needs config for testswarm instance location) - After running preprod-mode, got error: Location: http://localhost/mw/trunk/tools/testswarm/scripts/_checkouts_symlink/r101591/ A database error has occurred. Did you forget to run maintenance/update.php after upgrading? See: http://www.mediawiki.org/wiki/Manual:Upgrading#Run_the_update_script Query: DELETE FROM msg_resource Function: MessageBlobStore::clear Error: 8 attempt to write a readonly database Comment: TIP TIP TIP for small projects use git as a front end to svn :-) Example: git svn clone <somerepo> <localpath> cd <localpath> ... Do a ton of changes like above ... git add -p # pick your hunks git commit # with a nice message git add -p # pick your hunks git commit # with a nice message ... git svn dcommit Et voila. This way you will be able to submit your messy working copy in separate commits :-D _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
