Modify

Opened 16 years ago

Closed 11 years ago

#2181 closed defect (wontfix)

Patch fails with 0.10-stable rev 6059

Reported by: anonymous Owned by: jmt4b04d4v
Priority: low Component: TracSpanishTranslation
Severity: major Keywords: UnicodeDecodeError, Win32
Cc: Trac Release: 0.10

Description (last modified by Ryan J Ollos)

Hello, I made a checkout of both the most recent 0.10-stable and your patch. After the patch was applied, I tried my sample project and got this.

Traceback (most recent call last):
  File "C:\Python24\Lib\site-packages\trac\web\main.py", line 406, in dispatch_request
    dispatcher.dispatch(req)
  File "C:\Python24\Lib\site-packages\trac\web\main.py", line 208, in dispatch
    chrome.populate_hdf(req, chosen_handler)
  File "C:\Python24\Lib\site-packages\trac\web\chrome.py", line 319, in populate_hdf
    for category, name, text in contributor.get_navigation_items(req):
  File "C:\Python24\Lib\site-packages\trac\Settings.py", line 35, in get_navigation_items
    yield ('metanav', 'settings',
UnicodeDecodeError: 'utf8' codec can't decode bytes in position 11-12: unexpected end of data

Attachments (0)

Change History (2)

comment:1 Changed 16 years ago by jmt4b04d4v

Keywords: UnicodeDecodeError Win32 added
Priority: normallow
Severity: normalmajor
Status: newassigned

Sorry for delay, I've started to research this issue today.

My first conclusions:

  • This encoding issue in Win32 was not predicted (it's not an EOL-style issue).
  • Patching can be realized in Win32 without hunk inconsistencies (it wasn't planned though).
  • Patching procedure in Win32 was reproduced, both with Eclipse + Subclipse and TortoiseSVN. Certainly some files doesn't keep UTF-8 formatting, some do?.
  • Patch files were produced with UTF-8 encoding. Revised today again.

Conclusion: Issue needs more attention and research to get solved.

As long as patching works in *nix, these are the suggested OS platforms where to realize patching procedure (with working copies obtained in there). Then, someone can certainly package patched sources to realize installation in any other OS platform.

Due to there is a workaround, I'm changing priority and severity.

comment:2 Changed 11 years ago by Ryan J Ollos

Description: modified (diff)
Resolution: wontfix
Status: assignedclosed

Deprecated. See Trac 0.12 if you need a translation.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain jmt4b04d4v.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.