win32 seflftest results for bzr.0.9
Alexander Belchenko
bialix at ukr.net
Sat Aug 5 08:56:12 BST 2006
Alexander Belchenko пишет:
> John Arbash Meinel пишет:
> ....
>> Actually, the _set_user_ignores is so that we don't accidentally include
>> that directory in other output.
>>
>> It just needs to be updated everywhere to be:
>>
>> ignores._set_user_ignores(['./.bazaar', './bazaar'])
>>
>> The point is for it to ignore itself, so that when the directory would
>> get created later, it will be ignored.
>>
>> I'll certainly approve a patch that updates it.
>
> Well, it's not so obvious changes as it seems.
>
> Too many tests have hardwired string '.bazaar' inside string literals.
> Furthermore blackbox.test_ls it's a pain in the ass to make it pass.
> It's all about '.bazaar' in almost every test. I start to rework it to
> more generalized way. But after hours of work I'm totally unsure that I
> do *right* changes.
>
> I can show partial diff of my changes to discuss: is it the right
> changes for selftest?
Here my not finished yet patch. I'm slightly worrying about changes in
test/blackbox/test_ls.py. More generalized way to make expected output
for different platform make this test looks less obvious.
I'm waiting for comments: is it right way to fix this platform difference?
--
Alexander
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: bazaar-dir-for-testing.diff
Url: https://lists.ubuntu.com/archives/bazaar/attachments/20060805/8c61c7d8/attachment.diff
More information about the bazaar
mailing list