A.L.E.C wrote:
On 11/19/2012 02:30 PM, Thomas Bruederli wrote:
What about moving framework classes/files to program/framework and keep the Roundcubemail app related classes and files in program/include?
I prefer lib/Roundcube because this allow as to re-use rcube_autoload(). It's better to have /usr/share/php/Roundcube than /usr/share/php/framework.
I was not talking about the final install location on a target system but about our git repository.
Installing files to /usr/share/php/roundcube-framework and /usr/share/php/roundcubemail would be good suggestion but that's up to packagers or sysadmins.
For our complete webmail package, we should pack everything into the program directory in order to keep the update procedure as it currently is.
I think both, the framework and the include directory, should have an iniset.php (you can rename it to bootstrap.php if you want) which does the necessary setup to use either the framework or the Roundcube application. program/include/iniset.php would most likely include program/framework/iniset.php to effectively load the framework to be used within the application.
Not a bad idea, but we still have 4 files in program/include, that's why I thought about getting rid of this directory.
Only 4? I assumed that some classes like rcube_output*, rcube_session and rcube_user remain "application-only" and would not become part of the framework.
But even if we end up with the 4 non-class include files, I think the directory "include" is the appropriate location for them.
~Thomas