On Wed, Mar 28, 2012 at 01:53:20PM +0200, Jiri Suchomel wrote:
Dne St 28. března 2012 12:31:13, Gabriele Mohr napsal(a):
Am 26.03.2012 14:45, schrieb Lukas Ocilka:
On 03/26/2012 02:40 PM, Jiri Suchomel wrote:
Dne Po 26. března 2012 14:17:09, Arvin Schnell napsal(a): Not really related, but still: some packages currently do not build because of failing testsuites. This is (at least partly) because of Gabi's change in FileUtils::Exists which now returns true only when .target.stat returns non- empty map. Some of our testuites rely on default (nil) value of .target.stat and expect FileUtils::Exists to return true. So please check as well if this is not a case for your modules.
I have reverted my change in FileUtils::Exists(). It's unnessary to check the return of SCR::Read(.target.stat, target) for != nil because it will never return nil but an empty map if the file doesn't exist (should have had a closer look before doing the change). I have submitted yast2-2.22.5 to Factory (request id 111523) and with it the testsuites of affected modules should run again.
Greetings Gabi
Ouch! And I've already adapted my testsuites to your changes...
If you instruct the testsuite SCR to return a useful map for .target.stat instead of nil your adaptions do not hurt and seem to be the right approach anymay. The problem with Fileutils::Exists was that its behaviour has changed when SCR::Read(.target.stat, ...) returns nil, which it never does for the real SCR. But it happens with the fake testsuite SCR unless the return value for .target.stat is specified. Regards, Arvin -- Arvin Schnell, <aschnell@suse.de> Senior Software Engineer, Research & Development SUSE LINUX Products GmbH, GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB 16746 (AG Nürnberg) Maxfeldstraße 5 90409 Nürnberg Germany -- To unsubscribe, e-mail: yast-devel+unsubscribe@opensuse.org To contact the owner, e-mail: yast-devel+owner@opensuse.org