Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: packagerChrome.py

Issue 9259023: Use platform-dependent metadata files to allow multiple builds from the same repository (Closed)
Patch Set: Created Jan. 25, 2013, 7:13 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« build.py ('K') | « packager.py ('k') | packagerGecko.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: packagerChrome.py
===================================================================
--- a/packagerChrome.py
+++ b/packagerChrome.py
@@ -13,20 +13,27 @@
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
import sys, os, re, json, struct
from StringIO import StringIO
-from packager import getDefaultFileName, readMetadata, getBuildVersion, getTemplate, Files
+import packager
+from packager import getDefaultFileName, getBuildVersion, getTemplate, Files
defaultLocale = 'en_US'
+def getMetadataPath(baseDir):
Felix Dahlke 2013/01/25 07:49:43 This function wasn't previously defined or importe
Wladimir Palant 2013/01/25 08:23:10 You are right. I realized that geckoPackager.getMe
+ return packager.getMetadataPath(baseDir, 'chrome')
+
+def readMetadata(baseDir):
+ return packager.readMetadata(baseDir, 'chrome')
+
def getIgnoredFiles(params):
return set(('store.description',))
def getPackageFiles(params):
result = set(('_locales', 'icons', 'jquery-ui', 'lib', 'skin', 'ui',))
if params['devenv']:
result.add('qunit')
« build.py ('K') | « packager.py ('k') | packagerGecko.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld