Mailinglist Archive: yast-devel (129 mails)

< Previous Next >
[yast-devel] Theme: foo(bar) is deprecated, [please fix]
  • From: Lukas Ocilka <lukas.ocilka@xxxxxxx>
  • Date: Fri, 09 Nov 2007 10:25:31 +0100
  • Message-id: <4734278B.8000902@xxxxxxx>

We just had a discussion about deprecated YCP code builtins. You've
definitely already mentioned these warnings in YaST log or when building
a YaST package:

[Parser] Warning: 'select ()' is deprecated
[Parser] Warning: find(...) is deprecated, please fix
[Parser] Warning: tolist(...) is deprecated, please fix
[Parser] Warning: 'lookup()' is deprecated

My proposal was to disable these deprecated builtins in openSUSE 11.0
but Martin Vidner objected that we should leave them for a backward
compatibility and just document them better. In fact, these builtins
don't make YCP core slower so pros and cons of backward compatibility
sound really good.

The current generated documentation doesn't document these deprecated
builtins because someone has removed the documentation from source code
:) So it's also sometimes hard to understand what the current code does
and how to replace it with new syntax/builtin.

So, how to improve the current situation?
1.) Unify those warnings (minor).
2.a.) Write a hint how to replace the deprecated builtin by a new one
(in that warning).
2.b.) Write a page at openSUSE WIKI about replacing deprecated builtins
with new ones (and write that URL in the warning).

Any better ideas?



Lukas Ocilka, YaST Developer (xn--luk-gla45d)
SUSE LINUX, s. r. o., Lihovarska 1060/12, Praha 9, Czech Republic

< Previous Next >
This Thread
  • No further messages