Bazaar

test suite regression on OSX and FreeBSD

Bug #433120 reported by Vincent Ladeuil
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bazaar
Fix Released
Medium
Vincent Ladeuil
Bazaar 2.1.0b1 "While the cat is away"

Bug Description

No name to protect the innocents, but some new code forgot about some known issues:
- OSX silently redirect /tmp to /private/tmp
- OSX and FreeBSD doesn't raise the same exception than Linux when os.remove() is used for a dir

Related branches

lp://staging/~vila/bzr/433120-tiger-selftest-regressions
John A Meinel: Approve
Diff: None lines
lp://staging/~vila/bzr/integration
Vincent Ladeuil: Approve
Diff: 31 lines (+4/-4)
2 files modified
bzrlib/lazy_regex.py (+3/-3)
doc/en/release-notes/bzr-2.7.txt (+1/-1)
Vincent Ladeuil (vila)
Changed in bzr:
assignee: nobody → Vincent Ladeuil (vila)
importance: Undecided → Medium
status: New → In Progress
status: In Progress → Fix Committed
milestone: none → 2.1.0b1
Revision history for this message
Vincent Ladeuil (vila) wrote :

For historical reasons:

FreeBSD: FAILED (errors=2, known_failure_count=29)
http://babune.ladeuil.net:26862/builders/freebsd7/builds/7/steps/Non-regression%20tests/logs/stdio
http://babune.ladeuil.net:26862/builders/freebsd8/builds/7/steps/Non-regression%20tests/logs/stdio

OSX: FAILED (failures=84, errors=167, known_failure_count=66)
http://babune.ladeuil.net:26862/builders/tiger/builds/66/steps/Non-regression%20tests/logs/stdio

leopard: FAILED (failures=73, errors=167, known_failure_count=64)
http://babune.ladeuil.net:26862/builders/leopard/builds/75/steps/Non-regression%20tests/logs/stdio

Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 433120] Re: test suite regression on OSX and FreeBSD

On Sat, 2009-09-19 at 22:51 +0000, Vincent Ladeuil wrote:
> For historical reasons:
>
> FreeBSD: FAILED (errors=2, known_failure_count=29)
> http://babune.ladeuil.net:26862/builders/freebsd7/builds/7/steps/Non-regression%20tests/logs/stdio
> http://babune.ladeuil.net:26862/builders/freebsd8/builds/7/steps/Non-regression%20tests/logs/stdio

^^^^[log from
bzrlib.tests.test_script.TestRm.test_rm_dir]--------------------
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/home/babune/buildbot/bzr/slaves/freebsd8/tests/freebsd8/build/bzrlib/tests/test_script.py", line 404, in test_rm_dir
    """)
  File "/usr/home/babune/buildbot/bzr/slaves/freebsd8/tests/freebsd8/build/bzrlib/tests/script.py", line 405, in run_script
    return self.script_runner.run_script(script)
  File "/usr/home/babune/buildbot/bzr/slaves/freebsd8/tests/freebsd8/build/bzrlib/tests/script.py", line 169, in run_script
    self.run_command(cmd, input, output, error)
  File "/usr/home/babune/buildbot/bzr/slaves/freebsd8/tests/freebsd8/build/bzrlib/tests/script.py", line 219, in run_command
    retcode, actual_output, actual_error = method(str_input, args)
  File "/usr/home/babune/buildbot/bzr/slaves/freebsd8/tests/freebsd8/build/bzrlib/tests/script.py", line 350, in do_rm
    os.remove(p)
OSError: [Errno 1] Operation not permitted: 'dir'

Seems different :)

Revision history for this message
Vincent Ladeuil (vila) wrote :

>>>>> "robert" writes:

    robert> On Sat, 2009-09-19 at 22:51 +0000, Vincent Ladeuil wrote:
    >> For historical reasons:
    >>
    >> FreeBSD: FAILED (errors=2, known_failure_count=29)
    >> http://babune.ladeuil.net:26862/builders/freebsd7/builds/7/steps/Non-regression%20tests/logs/stdio
    >> http://babune.ladeuil.net:26862/builders/freebsd8/builds/7/steps/Non-regression%20tests/logs/stdio

    robert> ^^^^[log from
    robert> bzrlib.tests.test_script.TestRm.test_rm_dir]--------------------
    robert> ----------------------------------------------------------------------
    robert> Traceback (most recent call last):
    robert> File "/usr/home/babune/buildbot/bzr/slaves/freebsd8/tests/freebsd8/build/bzrlib/tests/test_script.py", line 404, in test_rm_dir
    robert> """)
    robert> File "/usr/home/babune/buildbot/bzr/slaves/freebsd8/tests/freebsd8/build/bzrlib/tests/script.py", line 405, in run_script
    robert> return self.script_runner.run_script(script)
    robert> File "/usr/home/babune/buildbot/bzr/slaves/freebsd8/tests/freebsd8/build/bzrlib/tests/script.py", line 169, in run_script
    robert> self.run_command(cmd, input, output, error)
    robert> File "/usr/home/babune/buildbot/bzr/slaves/freebsd8/tests/freebsd8/build/bzrlib/tests/script.py", line 219, in run_command
    robert> retcode, actual_output, actual_error = method(str_input, args)
    robert> File "/usr/home/babune/buildbot/bzr/slaves/freebsd8/tests/freebsd8/build/bzrlib/tests/script.py", line 350, in do_rm
    robert> os.remove(p)
    robert> OSError: [Errno 1] Operation not permitted: 'dir'

    robert> Seems different :)

If you want to look at the test isolation escapes, search the OSX
logs, the FreeBSD ones are only about remove raising EACCESS
where EISDIR was expected.

Vincent Ladeuil (vila)
Changed in bzr:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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