Closed Bug 978013 Opened 10 years ago Closed 10 years ago

Add-on SDK doesn't handle non-ascii filename

Categories

(Add-on SDK Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: slash, Unassigned)

Details

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:27.0) Gecko/20100101 Firefox/27.0 Iceweasel/27.0.1 (Beta/Release)
Build ID: 20140222041426

Steps to reproduce:

(addon-sdk)$ cfx init
(addon-sdk)$ touch data/あ
(addon-sdk)$ cfx run


Actual results:

Traceback (most recent call last):
  File ".../addon-sdk/bin/cfx", line 33, in <module>
    cuddlefish.run()
  File ".../addon-sdk/python-lib/cuddlefish/__init__.py", line 745, in run
    loader_modules)
  File ".../addon-sdk/python-lib/cuddlefish/manifest.py", line 661, in build_manifest
    mxt.build(scan_tests, test_filter_re)
  File ".../addon-sdk/python-lib/cuddlefish/manifest.py", line 202, in build
    self.datamaps[self.target_cfg.name] = DataMap(self.target_cfg)
  File ".../addon-sdk/python-lib/cuddlefish/manifest.py", line 144, in __init__
    zipname = datafile_zipname(pkg.name, dataname)
  File ".../addon-sdk/python-lib/cuddlefish/manifest.py", line 23, in datafile_zipname
    return "%s-data/%s" % (packagename, datapath)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xa4 in position 0: ordinal not in range(128)


Expected results:

runs normally
We're going to be deprecating and removing cfx fairly soon in favor of a new JS-based tool. 

I'm going to wontfix this for now, since we're not going to be doing any new work on cfx at this point, but feel free to reopen this or file a new bug if the problem's still around once the new tools come out.
Status: UNCONFIRMED → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.