From 814f4adf6c0ce382df2be14ef3e7b3611d5f4b99 Mon Sep 17 00:00:00 2001 From: vrana Date: Wed, 14 Mar 2012 22:42:30 -0700 Subject: [PATCH] Stabilize sort in celerity map Summary: Smaller diffs, better blame, less noise on a file which is usually ignored. Less conflicts in merge! Test Plan: php -l __celerity_resource_map__.php Reviewers: epriestley Reviewed By: epriestley CC: aran, epriestley Differential Revision: https://secure.phabricator.com/D1909 --- scripts/celerity_mapper.php | 2 +- src/__celerity_resource_map__.php | 236 +++++++++++++++--------------- 2 files changed, 119 insertions(+), 119 deletions(-) diff --git a/scripts/celerity_mapper.php b/scripts/celerity_mapper.php index a9f4313822..c6ee8322cc 100755 --- a/scripts/celerity_mapper.php +++ b/scripts/celerity_mapper.php @@ -276,7 +276,7 @@ $runtime_map = var_export($runtime_map, true); $runtime_map = preg_replace('/\s+$/m', '', $runtime_map); $runtime_map = preg_replace('/array \(/', 'array(', $runtime_map); -ksort($package_map['packages']); +$package_map['packages'] = isort($package_map['packages'], 'name'); ksort($package_map['reverse']); $package_map = var_export($package_map, true); $package_map = preg_replace('/\s+$/m', '', $package_map); diff --git a/src/__celerity_resource_map__.php b/src/__celerity_resource_map__.php index 42b8cbe9f4..06febdb093 100644 --- a/src/__celerity_resource_map__.php +++ b/src/__celerity_resource_map__.php @@ -1942,111 +1942,6 @@ celerity_register_resource_map(array( ), array( 'packages' => array( - '1cb6883c' => - array( - 'name' => 'differential.pkg.css', - 'symbols' => - array( - 0 => 'differential-core-view-css', - 1 => 'differential-changeset-view-css', - 2 => 'differential-revision-detail-css', - 3 => 'differential-revision-history-css', - 4 => 'differential-table-of-contents-css', - 5 => 'differential-revision-comment-css', - 6 => 'differential-revision-add-comment-css', - 7 => 'differential-revision-comment-list-css', - 8 => 'phabricator-object-selector-css', - 9 => 'aphront-headsup-action-list-view-css', - 10 => 'phabricator-content-source-view-css', - 11 => 'differential-local-commits-view-css', - ), - 'uri' => '/res/pkg/1cb6883c/differential.pkg.css', - 'type' => 'css', - ), - '2af849fb' => - array( - 'name' => 'typeahead.pkg.js', - 'symbols' => - array( - 0 => 'javelin-typeahead', - 1 => 'javelin-typeahead-normalizer', - 2 => 'javelin-typeahead-source', - 3 => 'javelin-typeahead-preloaded-source', - 4 => 'javelin-typeahead-ondemand-source', - 5 => 'javelin-tokenizer', - 6 => 'javelin-behavior-aphront-basic-tokenizer', - ), - 'uri' => '/res/pkg/2af849fb/typeahead.pkg.js', - 'type' => 'js', - ), - '31d4b993' => - array( - 'name' => 'differential.pkg.js', - 'symbols' => - array( - 0 => 'phabricator-drag-and-drop-file-upload', - 1 => 'phabricator-shaped-request', - 2 => 'javelin-behavior-differential-feedback-preview', - 3 => 'javelin-behavior-differential-edit-inline-comments', - 4 => 'javelin-behavior-differential-populate', - 5 => 'javelin-behavior-differential-show-more', - 6 => 'javelin-behavior-differential-diff-radios', - 7 => 'javelin-behavior-differential-accept-with-errors', - 8 => 'javelin-behavior-differential-comment-jump', - 9 => 'javelin-behavior-differential-add-reviewers-and-ccs', - 10 => 'javelin-behavior-differential-keyboard-navigation', - 11 => 'javelin-behavior-aphront-drag-and-drop', - 12 => 'javelin-behavior-aphront-drag-and-drop-textarea', - 13 => 'javelin-behavior-phabricator-object-selector', - 14 => 'differential-inline-comment-editor', - 15 => 'javelin-behavior-differential-dropdown-menus', - 16 => 'javelin-behavior-buoyant', - ), - 'uri' => '/res/pkg/31d4b993/differential.pkg.js', - 'type' => 'js', - ), - '4fbae2af' => - array( - 'name' => 'javelin.pkg.js', - 'symbols' => - array( - 0 => 'javelin-util', - 1 => 'javelin-install', - 2 => 'javelin-event', - 3 => 'javelin-stratcom', - 4 => 'javelin-behavior', - 5 => 'javelin-request', - 6 => 'javelin-vector', - 7 => 'javelin-dom', - 8 => 'javelin-json', - 9 => 'javelin-uri', - ), - 'uri' => '/res/pkg/4fbae2af/javelin.pkg.js', - 'type' => 'js', - ), - '61f9d480' => - array( - 'name' => 'diffusion.pkg.css', - 'symbols' => - array( - 0 => 'diffusion-commit-view-css', - ), - 'uri' => '/res/pkg/61f9d480/diffusion.pkg.css', - 'type' => 'css', - ), - 31583232 => - array( - 'name' => 'maniphest.pkg.css', - 'symbols' => - array( - 0 => 'maniphest-task-summary-css', - 1 => 'maniphest-transaction-detail-css', - 2 => 'maniphest-task-detail-css', - 3 => 'aphront-attached-file-view-css', - ), - 'uri' => '/res/pkg/31583232/maniphest.pkg.css', - 'type' => 'css', - ), '78e8854e' => array( 'name' => 'core.pkg.css', @@ -2075,19 +1970,6 @@ celerity_register_resource_map(array( 'uri' => '/res/pkg/78e8854e/core.pkg.css', 'type' => 'css', ), - '86fc0b0c' => - array( - 'name' => 'maniphest.pkg.js', - 'symbols' => - array( - 0 => 'javelin-behavior-maniphest-batch-selector', - 1 => 'javelin-behavior-maniphest-transaction-controls', - 2 => 'javelin-behavior-maniphest-transaction-preview', - 3 => 'javelin-behavior-maniphest-transaction-expand', - ), - 'uri' => '/res/pkg/86fc0b0c/maniphest.pkg.js', - 'type' => 'js', - ), 'c18634d0' => array( 'name' => 'core.pkg.js', @@ -2110,6 +1992,124 @@ celerity_register_resource_map(array( 'uri' => '/res/pkg/c18634d0/core.pkg.js', 'type' => 'js', ), + '1cb6883c' => + array( + 'name' => 'differential.pkg.css', + 'symbols' => + array( + 0 => 'differential-core-view-css', + 1 => 'differential-changeset-view-css', + 2 => 'differential-revision-detail-css', + 3 => 'differential-revision-history-css', + 4 => 'differential-table-of-contents-css', + 5 => 'differential-revision-comment-css', + 6 => 'differential-revision-add-comment-css', + 7 => 'differential-revision-comment-list-css', + 8 => 'phabricator-object-selector-css', + 9 => 'aphront-headsup-action-list-view-css', + 10 => 'phabricator-content-source-view-css', + 11 => 'differential-local-commits-view-css', + ), + 'uri' => '/res/pkg/1cb6883c/differential.pkg.css', + 'type' => 'css', + ), + '31d4b993' => + array( + 'name' => 'differential.pkg.js', + 'symbols' => + array( + 0 => 'phabricator-drag-and-drop-file-upload', + 1 => 'phabricator-shaped-request', + 2 => 'javelin-behavior-differential-feedback-preview', + 3 => 'javelin-behavior-differential-edit-inline-comments', + 4 => 'javelin-behavior-differential-populate', + 5 => 'javelin-behavior-differential-show-more', + 6 => 'javelin-behavior-differential-diff-radios', + 7 => 'javelin-behavior-differential-accept-with-errors', + 8 => 'javelin-behavior-differential-comment-jump', + 9 => 'javelin-behavior-differential-add-reviewers-and-ccs', + 10 => 'javelin-behavior-differential-keyboard-navigation', + 11 => 'javelin-behavior-aphront-drag-and-drop', + 12 => 'javelin-behavior-aphront-drag-and-drop-textarea', + 13 => 'javelin-behavior-phabricator-object-selector', + 14 => 'differential-inline-comment-editor', + 15 => 'javelin-behavior-differential-dropdown-menus', + 16 => 'javelin-behavior-buoyant', + ), + 'uri' => '/res/pkg/31d4b993/differential.pkg.js', + 'type' => 'js', + ), + '61f9d480' => + array( + 'name' => 'diffusion.pkg.css', + 'symbols' => + array( + 0 => 'diffusion-commit-view-css', + ), + 'uri' => '/res/pkg/61f9d480/diffusion.pkg.css', + 'type' => 'css', + ), + '4fbae2af' => + array( + 'name' => 'javelin.pkg.js', + 'symbols' => + array( + 0 => 'javelin-util', + 1 => 'javelin-install', + 2 => 'javelin-event', + 3 => 'javelin-stratcom', + 4 => 'javelin-behavior', + 5 => 'javelin-request', + 6 => 'javelin-vector', + 7 => 'javelin-dom', + 8 => 'javelin-json', + 9 => 'javelin-uri', + ), + 'uri' => '/res/pkg/4fbae2af/javelin.pkg.js', + 'type' => 'js', + ), + 31583232 => + array( + 'name' => 'maniphest.pkg.css', + 'symbols' => + array( + 0 => 'maniphest-task-summary-css', + 1 => 'maniphest-transaction-detail-css', + 2 => 'maniphest-task-detail-css', + 3 => 'aphront-attached-file-view-css', + ), + 'uri' => '/res/pkg/31583232/maniphest.pkg.css', + 'type' => 'css', + ), + '86fc0b0c' => + array( + 'name' => 'maniphest.pkg.js', + 'symbols' => + array( + 0 => 'javelin-behavior-maniphest-batch-selector', + 1 => 'javelin-behavior-maniphest-transaction-controls', + 2 => 'javelin-behavior-maniphest-transaction-preview', + 3 => 'javelin-behavior-maniphest-transaction-expand', + ), + 'uri' => '/res/pkg/86fc0b0c/maniphest.pkg.js', + 'type' => 'js', + ), + '2af849fb' => + array( + 'name' => 'typeahead.pkg.js', + 'symbols' => + array( + 0 => 'javelin-typeahead', + 1 => 'javelin-typeahead-normalizer', + 2 => 'javelin-typeahead-source', + 3 => 'javelin-typeahead-preloaded-source', + 4 => 'javelin-typeahead-ondemand-source', + 5 => 'javelin-tokenizer', + 6 => 'javelin-behavior-aphront-basic-tokenizer', + ), + 'uri' => '/res/pkg/2af849fb/typeahead.pkg.js', + 'type' => 'js', + ), ), 'reverse' => array(