Mailinglist Archive: yast-devel (144 mails)

< Previous Next >
Re: [yast-devel] Unify yast-yast2 directory structure
On Tue, 27 Aug 2013 14:54:56 +0200
Martin Vidner <mvidner@xxxxxxx> wrote:

On Wed, Aug 21, 2013 at 10:06:13AM +0200, Josef Reidinger wrote:
now yast-yast2 contain library directory that is separated to
sub-directories [1]. I would like to have there only one src
directory that respect Y2DIR structure.
Main benefit from this move is that it is much easier to test code,
because now there interdependencies between directories, so you
need to know what subdirs you must use and it means you must create
a lot of effort to detect it or test only on installed system
( both is bad from my POV ).
The biggest drawback is that we lost logical separation between
these modules, but we can improve documentation, so it is not so
big ( like links in documentation of related modules ).

The resulting yast2.rpm has 114 files under
/usr/share/YaST2/modules. That is quite a lot for a source
directory. And that is not even counting the test case files.

I agree that running tests must be easy. But can't that be solved by
a wildcard?

DIRS=$(echo $(pwd)/library/*/src); echo "Y2DIR=${DIRS// /:}"

OK, valid complain. I would like to search usage of such modules and if
it is no longer used, then I plan to drop it. I also hope that some
modules can be replaced by ruby calls.
Testcase files is not issue, as it can be structured how we need.
Tricky part is src directory, that need to follow Y2DIR.
You proposal is interesting but it not much follow KISS and it is not
so easy to do it in ruby like we have it now. Not mention that then it
won't be easy to share test helpers specific for Yast2.

OK, I start with checking which modules we actually use and which can
be easily dropped.

< Previous Next >