Comment 1 for bug 368157

Revision history for this message
Ara Pulido (ara) wrote :

Hello Markus,

Thanks for your efforts!

I generally like the new structure, but I have some comments to make:

* Why do we have two folders for ubuntu-desktop-testing script?
  bin/ubuntu-desktop-testing
  cmd/cmd.py

  Is it completely necessary for buildout? I would prefer to have it under bin

* desktoptesting_testsuites:
I think this folder name is a bit long. I would prefer testsuites. If we are afraid that this can be confused with the desktoptesting tests themselves, we just can add an explanation on folder structure on the HACKING file. Mmm, now that I check the setup.py file, there, you call this folder just "testsuites" so I guess it was a late minute change ;-)