Edgewall Software
Modify

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#8724 closed defect (fixed)

trac-admin initenv --inherit: should not generate a ["ticket-workflow"] section

Reported by: typo3_community@… Owned by: Remy Blank
Priority: normal Milestone: 0.12
Component: admin/console Version: 0.11.5
Severity: normal Keywords: needinfo
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

When using the —inherit option a rudimentary trac.ini will be created, but is trac.ini do still have a "ticket-workflow" section which IS NOT NECESSARY AS A BASELINE SETTING due to the fact that it is defined in the global trac.ini (referenced by the "inherit" section).

Please change behavior of trac-admin initenv -inherit to generate a trac.ini like this:

[inherit]
file = /home/trac_shared/shared_workflow.trac.ini

[project]
name = customer_sandbox

[trac]
repository_dir = /home/svn/repos/customer_sandbox/

(I am using trac 0.11.5)

Attachments (0)

Change History (4)

comment:1 by Remy Blank, 15 years ago

Component: generaladmin/console
Owner: set to Remy Blank

Right, the check for an existing [ticket-workflow] section seems to be broken in ConfigurableTicketWorkflow.environment_created().

comment:2 by Remy Blank, 15 years ago

Keywords: needinfo added
Version: none0.11.5

After some testing, I actually cannot reproduce this issue with current 0.11-stable HEAD. I get a minimal trac.ini as you described (with additional database and repository_type entries).

One caveat, though: make sure that you specify an absolute path after --inherit=, otherwise trac-admin will not find the global configuration. It will not print an error message, and the resulting trac.ini will contain a [ticket-workflow] section. Could this have happened to you?

Maybe we should check that the path to the inherited config is absolute, and quit with an error message if this is not the case?

comment:3 by Remy Blank, 15 years ago

Resolution: fixed
Status: newclosed

Even better, make the path specified with --inherit= absolute in any case. Fixed in [8703].

comment:4 by Remy Blank, 15 years ago

Milestone: 0.11.60.12

Reverted [8703] in [8740], as it changed the current behavior of trac-admin $ENV intenv --inherit=PATH for relative paths.

Instead, the documentation for trac-admin $ENV initenv was improved in [8741] to mention the resolution of relative paths. The documentation could not be improved on 0.11-stable, because it only allows short descriptions in help texts.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Remy Blank.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Remy Blank to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.