configuration settings not recognised

Bug #622886 reported by Timmie
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
bzr-testrunner
Confirmed
High
Unassigned

Bug Description

I tried to add the setting

pre_change_branch_tip_test_command = nosetests --with-coverage --cover-erase

to
branch.conf
locations.conf

but the setting was not regocnised unless added to bazaar.conf

Revision history for this message
James Westby (james-w) wrote : Re: [Bug 622886] [NEW] configuration settings not recognised

On Mon, 23 Aug 2010 17:44:32 -0000, Tim <email address hidden> wrote:
> Public bug reported:
>
> I tried to add the setting
>
> pre_change_branch_tip_test_command = nosetests --with-coverage --cover-
> erase
>
> to
> branch.conf
> locations.conf
>
> but the setting was not regocnised unless added to bazaar.conf

Hi,

I definitely have this working from locations.conf, so I wonder what is
different in your environment.

Are you using checkouts of remote branches?

Can you show me how you are adding it to branch.conf and locations.conf
(i.e. with the section headings)?

Thanks,

James

Revision history for this message
Timmie (timmie) wrote :

Thanks for the replys.

I added it to the settings in this manner:

in locations.conf

[d:\devel\myworkdir]
pre_change_branch_tip_test_command = nosetests --with-coverage --cover-erase

in branch.conf

pre_change_branch_tip_test_command = nosetests --with-coverage --cover-erase

I would prever to have it in branch.conf or at least locations.conf

Revision history for this message
Timmie (timmie) wrote :

It works if I add it to the branch.conf like:

[DEFAULT]
pre_change_branch_tip_test_command = nosetests --with-coverage --cover-erase

But how do I set section headers in windiows where there is no slash?

According to
http://doc.bazaar.canonical.com/latest/en/user-reference/configuration-help.html#section-headers
a local section header should show:

[/home/jdoe/branches/]
pre_change_branch_tip_test_command = nosetests --with-coverage --cover-erase

It seems that

[d:\devel\myworkdir]
pre_change_branch_tip_test_command = nosetests --with-coverage --cover-erase

gives a porblem.

Revision history for this message
James Westby (james-w) wrote :

It appears that I have a modified version that does

    config = params.branch.get_config()
    command = config._get_location_config()._get_user_option("pre_change_branch_tip_test_command")

so it appears as though the branch.get_config() isn't looking at locations.conf
for some reason.

Thanks,

James

Changed in bzr-testrunner:
status: New → Confirmed
importance: Undecided → High
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.