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



_______________________________________________
Roundcube Development discussion mailing list
[email protected]
http://lists.roundcube.net/mailman/listinfo/dev

Reply via email to