Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#11731 closed defect (fixed)

Check on Trac version in

Reported by: Lele Gaifax Owned by: Jun Omae
Priority: normal Component: TracPygit2Plugin
Severity: normal Keywords:
Cc: Trac Release:


In the there is a check that prevent installation of the plugin if the Trac version is not higher than 0.12.

Since I usually install Trac instances with a buildout recipe, that check actually blocks the installation, since Trac may not be already there when the plugin gets installed.

Would it be possible to either remove the check, or at least postpone it when the plugin gets loaded by Trac?

Attachments (0)

Change History (5)

comment:1 Changed 4 years ago by Jun Omae

In 13907:

TracPygit2Plugin: removed check the version of installed Trac in (refs #11731)

comment:2 Changed 4 years ago by Jun Omae

Resolution: fixed
Status: newclosed

I've removed the check. However, if some plugins use i18n feature by Babel, the plugins require installed Trac on the its build. Therefore, I think the recipe should allow to install the plugins.

comment:3 Changed 4 years ago by Lele Gaifax

Thank you!

It's not clear to me what you mean: I do have i18n-aware plugins installed (one for all, AccountManagerPlugin), and everything run smooth, ie I correctly see the instance in my own language, italian. Do you have an example of a plugin that actually requires an already installed Trac package to allow its installation?

comment:4 Changed 4 years ago by Jun Omae

Trac core provides get_l10n_cmdclass and get_l10n_js_cmdclass in source:tags/trac-0.12.5/trac/,306,313#L291. The cmdclasss includes the following commands which are used in build stage of Trac core and plugins.

  • build: run compile_catalog and compile_catalog_js before build_py.
  • compile_catalog_js: compile catalog files for javascript i18n.
  • generate_messages_js: generate javascript i18n files from compiled catalog files.

Also i18n-aware plugins use one of the cmdclasss in, e.g. accountmanagerplugin/trunk/, tracdragdropplugin/0.12/

comment:5 Changed 4 years ago by Lele Gaifax

Oh, I see, thank you for the refs, they clarify the issue to me. These are mainly useful while developing though, to update existing .po files. My install procedure does its own compile of PO files to MO files itself anyway.

I do no see a clean way to pre-install a Trac package before executing the buildout recipe, other than going thru a virtualenv...

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain Jun Omae.
The resolution will be deleted.

Add Comment

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

Note: See TracTickets for help on using tickets.