Jorge Vargas wrote: > Hello, > > Please let me know if this isn't the place for this. > > I just installed trac multirepo branch + mercurial plugin, and I'm > loving it awesome work and I hope this becomes the main branch for the > upcomming 0.12 release. That said I think I found a bug, but I'm not > 100% sure. after the new i18n was added the following line was > introduced. > >
Did you actually face an installation issue? If yes, was it the same as http://trac.edgewall.org/ticket/7877? > locale_dir = pkg_resources.resource_filename(__name__,'../locale') > > I see two problems with this. > #1 __name__ is not supported in zipped eggs, which is the recommended > installation method for trac plugins. > I don't think Babel can deal with zipped eggs, so translated plugins should be marked as zip-unsafe for now (if that works at all, cf. problem above). > #2 I believe the ../locale syntax isn't supported by said function. > > ideally the first param should be the package name and the second only > "locale" but to be honest I don't know much of pkg_resources api to be > sure which will be the correct value. > Honestly, same goes for me, I simply picked that line from trac/utils/translations.py. As it worked fine for Trac itself, I figured it would for plugins as well (and indeed it does work for me). -- Christian --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Trac Development" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/trac-dev?hl=en -~----------~----~----~----~------~----~------~--~---
