New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Separate namespace for modules providing web interface #200
Conversation
…parameters aren't overridable by robot context.
src/sbin/task_manager.pl.in
Outdated
. $label . '.' | ||
$spool_task . '/' | ||
. $date . '.' | ||
. $label . '.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This hunk introduces trailing white spaces.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe a bug of perltidy. I fixed. Thanks.
Totally agree with you. This certainly clarifies the usage of these modules. |
Hm, I think the namespace "Sympa::WWW" would be better. Just my two cents. |
I think naming method is matter of preference. The method getting more support would be carried out. |
So let's discuss the new namespace on the (developers) mailing list. I also think it is a good idea and it is a good time to this before sympa-7.0 branch starts to take off. |
There seems no objection to moving. Rest of issue is how we may move (rename). Before we move to dev list, we would be better to present rationale by each. My favorite is new top-level namespace Sympa/
Sympa.pm
WWSympa/
WWSympa.pm (`Conf.pm will be removed --- hopefully in this year, besides) Separate namespaces will make separating packages easy:
etc. |
My favourite is to keep all our modules within a single namespace Sympa - this makes more sense to me and doesn't look so ugly like "WWSympa", "LMTPSympa", e.g.
This doesn't affect the separation of the modules for packaging. Also, while we are at it, a separate namespace for 3rd party modules would be nice to, e.g. one of these:
|
Okey, let's continue on dev list. |
Following modules are referred only by web applications (wwsympa.fcgi and sympa_soap_server.fcgi), and they would be moved to separate namespace. Because: